You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by "Jakub Vrána (JIRA)" <ji...@apache.org> on 2009/02/19 17:20:01 UTC

[jira] Created: (SHINDIG-930) Unable to autoload exceptions

Unable to autoload exceptions
-----------------------------

                 Key: SHINDIG-930
                 URL: https://issues.apache.org/jira/browse/SHINDIG-930
             Project: Shindig
          Issue Type: Bug
            Reporter: Jakub Vrána
            Priority: Minor


__autoload() function expects that classes are defined in files of the same name. However it is not true for exceptions. The problem is for example with php/src/common/sample/BasicSecurityTokenDecoder.php which throws GadgetException which is not found by __autoload.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Updated: (SHINDIG-930) Unable to autoload exceptions

Posted by "Jakub Vrána (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SHINDIG-930?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jakub Vrána updated SHINDIG-930:
--------------------------------

    Attachment: shindig-930.diff

This is only partial fix of the problem because not all exceptions are defined in files with "Exception" removed. For example GadgetSpecException is defined in GadgetSpecParser.php.

> Unable to autoload exceptions
> -----------------------------
>
>                 Key: SHINDIG-930
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-930
>             Project: Shindig
>          Issue Type: Bug
>            Reporter: Jakub Vrána
>            Priority: Minor
>         Attachments: shindig-930.diff
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> __autoload() function expects that classes are defined in files of the same name. However it is not true for exceptions. The problem is for example with php/src/common/sample/BasicSecurityTokenDecoder.php which throws GadgetException which is not found by __autoload.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Resolved: (SHINDIG-930) Unable to autoload exceptions

Posted by "Chris Chabot (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SHINDIG-930?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Chris Chabot resolved SHINDIG-930.
----------------------------------

    Resolution: Fixed
      Assignee: Chris Chabot

Thanks for the report! 

I was trying to cut down on the number of files included (since this tends to end up being dog slow in php) but indeed the gadget exception is used all over the place, so putting this in the gadget.php file was probably not the best solution.

I've separated this class back out into the GadgetException.php file, so the error should go away now.

ps Standard disclaimer: The trunk is currently known to be unstable, so while I'm very grateful for the fix (and please keep those coming!) if you want to use php-shindig in production, please use the stable release branch instead: http://svn.apache.org/repos/asf/incubator/shindig/branches/1.0.x-incubating/

> Unable to autoload exceptions
> -----------------------------
>
>                 Key: SHINDIG-930
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-930
>             Project: Shindig
>          Issue Type: Bug
>          Components: PHP
>    Affects Versions: trunk
>            Reporter: Jakub Vrána
>            Assignee: Chris Chabot
>            Priority: Minor
>             Fix For: trunk
>
>         Attachments: shindig-930.diff
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> __autoload() function expects that classes are defined in files of the same name. However it is not true for exceptions. The problem is for example with php/src/common/sample/BasicSecurityTokenDecoder.php which throws GadgetException which is not found by __autoload.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Updated: (SHINDIG-930) Unable to autoload exceptions

Posted by "Jakub Vrána (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SHINDIG-930?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jakub Vrána updated SHINDIG-930:
--------------------------------

          Component/s: PHP
        Fix Version/s: trunk
    Affects Version/s: trunk

> Unable to autoload exceptions
> -----------------------------
>
>                 Key: SHINDIG-930
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-930
>             Project: Shindig
>          Issue Type: Bug
>          Components: PHP
>    Affects Versions: trunk
>            Reporter: Jakub Vrána
>            Priority: Minor
>             Fix For: trunk
>
>         Attachments: shindig-930.diff
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> __autoload() function expects that classes are defined in files of the same name. However it is not true for exceptions. The problem is for example with php/src/common/sample/BasicSecurityTokenDecoder.php which throws GadgetException which is not found by __autoload.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.