[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