Hi

i think we should add Type Hinting as a *Good Practice *for all PHP Code now
onwards, and start adding that to actual code.

Ropu

On Thu, Jul 10, 2008 at 9:10 AM, Chris Chabot (JIRA) <[EMAIL PROTECTED]>
wrote:

>
>     [
> https://issues.apache.org/jira/browse/SHINDIG-439?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel]
>
> Chris Chabot resolved SHINDIG-439.
> ----------------------------------
>
>    Resolution: Fixed
>      Assignee: Chris Chabot
>
> Looks good, the other services had the type hinting too, so this was indeed
> an erroneous situation.
>
> Applied & Committed, thanks!
>
> > Missing type hinting regarding SecurityToken in abstract PeopleService
> and UserId classes
> >
> -----------------------------------------------------------------------------------------
> >
> >                 Key: SHINDIG-439
> >                 URL: https://issues.apache.org/jira/browse/SHINDIG-439
> >             Project: Shindig
> >          Issue Type: Bug
> >          Components: RESTful API (PHP)
> >            Reporter: Karsten Beyer
> >            Assignee: Chris Chabot
> >         Attachments: fix-issue-SHINDIG-439.patch
> >
> >
> > The ActivitiesService and AppDataService both require the $token to be of
> type SecurityToken. For consistency this should also be the case for the
> PeopleService abstract class and the UserId handler class as both of them
> indeed require the $token parameter to be of type SecurityToken.
>
> --
> This message is automatically generated by JIRA.
> -
> You can reply to this email to add a comment to the issue online.
>
>


-- 
.-. --- .--. ..-
R o p u

Reply via email to