Bugzilla – Full Text Bug Listing
| Summary: | Variable type in phpDoc comment isn't recognized correctly, when default value for variable is array | ||
|---|---|---|---|
| Product: | z_Archived | Reporter: | Silver Zachara <silver.zachara> |
| Component: | PDT | Assignee: | Thierry BLIND <thierryblind> |
| Status: | CLOSED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | silviya, thierryblind, zulus |
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | Windows 7 | ||
| See Also: |
https://git.eclipse.org/r/52749 https://git.eclipse.org/c/pdt/org.eclipse.pdt.git/commit/?id=fc2084e325621fa874d8d87e4415165719bd2991 https://git.eclipse.org/r/52920 https://git.eclipse.org/c/pdt/org.eclipse.pdt.git/commit/?id=88b3a282951667274d461e3a1ebbb7070f47e479 |
||
| Whiteboard: | |||
New Gerrit change created: https://git.eclipse.org/r/52749 Hi,
patch is available.
There's only one case that is not fully handled by this patch, it's when field declarations are grouped together:
class Foo {
private $field1 = array(), $field2 = 5;
}
But on the other side, it also corrects function parameters handling:
function test($param = array()) {
}
I'll make a separate patch to handle the grouped field declarations, as there are non-trivial changes that I would discuss before merging ;)
Thierry.
Gerrit change https://git.eclipse.org/r/52749 was merged to [master]. Commit: http://git.eclipse.org/c/pdt/org.eclipse.pdt.git/commit/?id=fc2084e325621fa874d8d87e4415165719bd2991 First patched was merged. I leave this bug report open until grouped field declarations are correctly handled. Thierry. New Gerrit change created: https://git.eclipse.org/r/52920 Gerrit change https://git.eclipse.org/r/52920 was merged to [master]. Commit: http://git.eclipse.org/c/pdt/org.eclipse.pdt.git/commit/?id=88b3a282951667274d461e3a1ebbb7070f47e479 fixed. Verified. Closing |
Hi, when I press ctrl+shift+j and my cursor is at $_tmp variable @class foo { /** * Enter description here ... * @var unknown_type */ private $_tmp = array(); function __construct() { ; } }@ and should be: @class foo { /** * Enter description here ... * @var array */ private $_tmp = array(); function __construct() { ; } } @