Agreed, as the saying goes "patches are welcome" :-)
-- Chris
On Jul 10, 2008, at 2:21 PM, Ropu wrote:
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