[dev] API Interface Rewrite
Jan Schneider
jan at horde.org
Thu Aug 6 08:39:13 UTC 2009
Zitat von Michael M Slusarz <slusarz at horde.org>:
> Quoting Jan Schneider <jan at horde.org>:
>
>>>> Also, the $services and $types arrays should go away in favor of
>>>> well-documented and marked up PHP classes.
>>>
>>> Since I was never really sure what these were for, this makes
>>> complete sense. So API caching at logon will consist of loading
>>> an API file (if there is one), scanning for the list of public
>>> functions, and storing that data. API documentation will be
>>> created solely from phpdoc comments. This is a worthy goal in and
>>> of itself.
>>
>> We still need the types because those are necessary for translating
>> complex PHP types to SOAP structs.
>> But the service description should be build via Reflection. We
>> might have to add our own phpdoc tag for the type that we translate
>> to/from, is this supported by the Reflection API?
>
> I think this code already exists in the base Horde_Reflection class.
>
> But as far as the $types information - I can't find a single place
> in the code that uses it (doing a grep on
> Horde_Registry::listTypes() and Horde_Registry::getSignature() turn
> up absolutely nowhere in our code that we use it).
This is because the Horde_Rpc_Soap class is currently broken. See the
old Horde_RPC_soap::_setupDispatchMap().
> The code in the newapi branches works fine, albeit without any of
> this $types information. But if the $types information is not being
> used anywhere, I'm inclined to merge the changes to master and we
> can deal with re-adding $types info later.
Jan.
--
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digitale PGP-Unterschrift
URL: <http://lists.horde.org/archives/dev/attachments/20090806/0062f956/attachment.bin>
More information about the dev
mailing list