Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Function List does not recognise PHP function name starting with 'f'... #4208

Closed
hnmcc opened this issue Feb 20, 2018 · 9 comments
Closed

Function List does not recognise PHP function name starting with 'f'... #4208

hnmcc opened this issue Feb 20, 2018 · 9 comments
Assignees

Comments

@hnmcc
Copy link

hnmcc commented Feb 20, 2018

Description of the Issue

The built-in function list does not recognise the first character of a PHP function name starting with 'f'. I have substituted numerous other characters for the 'f', and the new name has been picked up correctly by the function list. I have tried every permutation of closing/saving the file, closing NPP, rebooting, etc; the effect remains.

Debug Information

Notepad++ v7.5.4 (32-bit)
Build time : Jan 1 2018 - 01:47:45
Path : F:\PortableApps\Notepad++Portable\App\Notepad++\notepad++.exe
Admin mode : OFF
Local Conf mode : ON
OS : Windows 8.1 (64-bit)
Plugins : NppConverter.dll NppExport.dll ComparePlugin.dll PluginManager.dll NppTextFX.dll ClipboardHelper.dll mimeTools.dll NppColumnSort.dll MultiClipboard.dll DSpellCheck.dll

npp_function_list_bug

@sasumner
Copy link
Contributor

@hnmcc What is different about your "formAndControls" function and your "fetchAccountData" or "fetchServiceData", etc functions?--the latter ones also start with "f" but seem to be put in function list correctly...

@MAPJe71
Copy link
Contributor

MAPJe71 commented Feb 21, 2018

@hnmcc have a look here.

@hnmcc
Copy link
Author

hnmcc commented Mar 6, 2018 via email

@MAPJe71
Copy link
Contributor

MAPJe71 commented Mar 6, 2018

The problem is caused by the parser incorrectly checking for keywords as function/method names while allowing them to start with a keyword. So it's not related to the "f" alone but to "formAndControls" starting with the keyword "for".
Follow this link to the forum for an updated parser that solves this problem.

@MAPJe71
Copy link
Contributor

MAPJe71 commented Feb 4, 2019

@SuperUserNameMan no it's not (see my comment at #4845).

@SuperUserNameMan
Copy link

SuperUserNameMan commented Feb 5, 2019

@MAPJe71 : yep, sorry. I opened several issues related to the "Function List" and got confused.
I was looking for an issue related to functions starting with "for" not listed correctly into the function-list.
(i'm going to try to remove my previous posts to cleanup my mess edit : well, apparently, not possible to remove the "issue references" ... )

@hnmcc
Copy link
Author

hnmcc commented Mar 14, 2019

Sorry for not seeing earlier message. I encountered this problem again in the last few days. It now seems to be limited to function names beginning "format" - farmat, fermat, firmat, and furmat are all OK, as are fold, focus, and font. But formatXXX always appears in the list as ormatXXX.

Screenshots attached.
Base_setThresholdFormat
farmatThreshold
formatThreshold

@hnmcc
Copy link
Author

hnmcc commented Mar 14, 2019

This is using Npp 7.6.3 Portable on up-to-date Win 8 64-bit, btw.

@MAPJe71
Copy link
Contributor

MAPJe71 commented Mar 14, 2019

@hnmcc
If you updated the parser with the one I linked to (twice) you might have updated the wrong functionList.xml i.e. there could be more functionList.xml files on your system.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants