sidharthkuruvila/phpparser

Automatically exported from code.google.com/p/phpparser

phpparser's Issues

Contact

I've a offer Please contact me 

Thanks

Original issue reported on code.google.com by [email protected] on 18 Sep 2014 at 8:43

Refine token names

Would you consider some minor refinements to the symbol names you are
using? This would clarify the generated code.
At issue: Your use of the word "Brace".  So far as I am aware, Brace always
refers to a curly bracket: {}, and *never* to a round bracket as you have
it in your symbols. See other grammars, wikipedia etc to see if you agree
(I also checked four PHP books, all refer to {} as braces.).
By contrast, "bracket" can apply to any of the ()[]{} or <>, with
adjectives: round, square, curly and angle.

Anyhow, I suggest the following:
LParen, RParen for left and right ( ) (Round parentheses)
LSquare, RSquare for [ ] (Square brackets)
LCurly, RCurly for { }   (Curly braces)

I *think* this should be unambiguous, and matches well with other grammars.

-- Graham

Original issue reported on code.google.com by [email protected] on 28 May 2009 at 2:12

final keword

Please check:

class C extends A {
final function X() {}
}

final keyword is not interpreted as a separate constant and it should.
right now you have to catch it from PhpLexer.UnquotedString constant.

Original issue reported on code.google.com by [email protected] on 12 Aug 2010 at 8:43

If..else problem

When trying to parse the code below:
    if ($this->getRequest()->isPost()) {
        $formData = $this->getRequest()->getPost();
        if ($form->isValid($formData)) {
            ...
        } else {
            ...
        }
    } else {
        $id = $this->_request->getParam('id', 0);
        if ($id > 0) {
            ...
        }
    }

It throws MismatchedTokenException 'else' at the line with second else.

Original issue reported on code.google.com by [email protected] on 23 Jan 2010 at 11:46

Optional Default of Switch not handled

The following code does not parse

switch ($i) {
    case 0:
        echo "i égal 0";
        break;
    case 1:
        echo "i égal 1";
        break;
    case 2:
        echo "i égal 2";
        break;
}

We fixed it by updating the cases rule:

cases 
    : casestatement*  defaultcase?
    ;

Original issue reported on code.google.com by [email protected] on 25 Sep 2012 at 3:36