[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