-
Notifications
You must be signed in to change notification settings - Fork 25
Ensure the parser runs properly on major PHP frameworks/libraries #148
Comments
These frameworks all need retesting once the issues marked against them are resolved. Update: re-ran, only Yii 1 is now failing to parse (due to memory leak) |
Deferring to 0.2.2 release |
Could we also add Zend Framework 1 to the list? 1.12.20 is the latest. Still failing to finishing parsing with it included as of Crane v0.2.1. EDIT: Re-ran the parser after it froze up and it worked! |
I've added Zend Framework 1, thanks for the suggestion! |
Is it possible to make Phalcon also work? I tried and tried with Phalcon stubs to get them working, but no luck so far. Am i wrong or not possible to do that yet? |
@mpotoc you mean you would like to parse files like these one : https://github.com/phalcon/cphalcon/blob/master/phalcon/acl.zep ? |
@ichiriac yes something like that, i am using phpstorm (university license until mid 2017) and in there you can add external libraries like Phalcon Dev Tools, which have stubs folder under IDE folder and you can link this in phpstorm to get full Phalcon namespace, etc. intellisense. Is something like this possible in VS Code with your extension, or will be anytime soon? Still have 8 month to decide stay on Storm and buy it or start using something else. Thanks |
@mpotoc Yes, we are planning to add a feature that lets you get intellisense for libraries that are outside the workspace folder. This change will also let you define filetypes other than |
@ichiriac perfect, will look for updates of your extension. Thanks a lot for the answer. |
Is it possible to add Magento 1.9.x and 2.x to the list? For Magento 1.9.x, it stops at 66 of 9748 / 0.7% |
@kiatng they are now in the php-parser token tests en ensure their parsing (bugfixing in progress) |
@ichiriac would that apply to Magento 2 as well, do you think? It also locks up having partially indexed. |
@scottsb already done, here the list of projects already passing : |
Is it possible to adding Wordpress to this list? |
hi @codewp, it's done & tests passes with the latest version - now we need to release crane in order to fix all these bugs |
As of Crane 0.3.0, all major libraries in this list now parse without any errors |
Frameworks (✓ means parsing successfully)
Profiler.php
- Issue(s) found: Suspected parse error for $this->_compile_{$section} #150php-parser undefined variable "c" (possible typo?) #151Tree builder/parser error parsing ellipsis #153Symfony PHP7 specific code #154, Parser crash during parse of TCPDF library #144Memory leak causing server crash when parsing large file [optimisation] #157Lexical error reject() error #159+ Incorrectly parsing inline HTML inside functions #158 (quick fix applied)Libraries
The text was updated successfully, but these errors were encountered: