Arbit - project tracking

PHP Depend

#61: Catch parser errors and continue processing

Issue revisions

  • new by Manuel Pichler at 2009-F-22 19:48
  • assigned by Manuel Pichler at 2009-F-22 19:48
  • assigned by Manuel Pichler at 2009-F-22 19:49
  • assigned by Manuel Pichler at 2009-F-22 23:02
  • assigned by Manuel Pichler at 2009-M-18 19:01
  • closed by Manuel Pichler at 2009-M-30 13:36
Type enhancement enhancement
State closed closed
Priority low low
Resolution implemented implemented
Assigned to Manuel Pichler
Scheduled for 0.9.5
Affected versions 0.9.4
Affected components Parser
Last change Saturday 30 May 2009 13:36:15 UTC by Manuel Pichler

The current behavior of PHP_Depend is, die when the parser detects a syntax error. This behavior should be changed, PHP_Depend should log those errors and complete the analysis phase.

  • Manuel Pichler at Sunday 22 February 2009 19:49:34 UTC

    Versions adjusted.

  • Manuel Pichler at Sunday 22 February 2009 23:02:03 UTC

    Typo in issue title fixed.

  • Manuel Pichler at Friday 20 March 2009 20:38:08 UTC

    The parser fails with an exception while analyzing the YII framework. This happens because the YII framework contains model templates that have a .php extension but contain invalid php code with placeholders.

    1 <?php
    2
    3 class {ClassName} extends CActiveRecord
    4 {
    5        /**
    6         * Returns the static model of the specified AR class.
    
    

    And this results in the following exception.

    Unexpected token: {, line: 3, col: 7, file: /yii/framework/cli/views/shell/model/model.php
    
    

    Strange, why do the add a .php extension instead of a meaningful extension for templates, php lint would also fail on for that code?!

  • Manuel Pichler at Monday 18 May 2009 19:01:05 UTC

    Implemented in svn revision #880.

  • Manuel Pichler at Saturday 30 May 2009 13:36:15 UTC

    Missing closed flag set.