Autocomplete Items Order Issue

Valery Bogomolov

@michael-vincent Here you can find my php.xml file

https://github.com/CoreText/WordPress-Auto-complete-for-Notepad-Plus/blob/master/php.xml

that has all PHP functions that I generated using a parser + I added my custom items and I had no issues with the previous autocomplete functionality of Npp, except was not able to autocomplete items with spaces, so I added tab character using XML entity and it works for me.

I’ll read the FAQ and will try to describe the feature in the appropriate place.

Thanks.

PeterJones

@valery-bogomolov ,

Please understand that Autocomplete has been advertised as being able to auto-complete keywords that are made up of “word characters”: ASCII letters, ASCII numbers, and the _ underscore. If you try to extend its usage beyond that, you are moving into edge-case, non-guaranteed behavior. If it always worked that way for you before, you were lucky, but it was not guaranteed. You have now found the problem of relying on non-advertised/non-guaranteed functionality: it can change without notice.

For code snippets or templates, you should really use one of the plugins that @Michael-Vincent mentioned.

That said, I think that it shouldn’t have been sorted with the ?1000 (or if he was using that to keep the Fx ones separate from the word-completion ones, it should have been handled differently).

Michael Vincent

@lycan-thrope said in Autocomplete Items Order Issue:

Something has to have changed.

It did. I explained it above. The commit is here. I authored it.

Using non-word characters in custom autocomplete language files was not part of my testing. I would argue that the autocomplete files are for “functions” not “code snippets”:

function: print_r
code snippet: print_r(get_class_methods(get_class($var)));die();

Use a snippet plugin for the latter.

Cheers.

Michael Vincent

@peterjones said in Autocomplete Items Order Issue:

That said, I think that it shouldn’t have been sorted with the ?1000

That’s tough, I agree I’d like to add it later, but the code structure (as-is) really doesn’t allow that. As we loop through the “functions” from the autocomplete language file, we add the ?1000 since later those are merged with the word completions found in the current document. If we wait until the merge, I couldn’t see a way to identify the “functions” and add the icon identifier. Then, the whole merged list is sorted - which means the ?1000 are in the items during the sort. And I’m guessing ? comes after ( in the sort routine.

Cheers.

PeterJones

@michael-vincent ,

Left paren ( (0x28) comes before ? (0x3F) in ASCII order, so it really should be sorted before by any normal lexicographical sort function.

Michael Vincent

@peterjones said in Autocomplete Items Order Issue:

Left paren ( (0x28) comes before ? (0x3F) in ASCII order, so it really should be sorted before by any normal lexicographical sort function.

Pretty sure that’s the answer then.

Cheers.

PeterJones

@michael-vincent ,

Since you were involved in that old ticket, could you maybe suggest to Don that instead of ?1000, just use a low-ASCII control character (0x01-0x1F – obviously not 0x00 NUL, since that would end the string) – that way it would sort after the words found in the current document, but before any visible ASCII character when sorting the function-list keywords. That would fix the OP issue

Michael Vincent

@peterjones said in Autocomplete Items Order Issue:

maybe suggest to Don that instead of ?1000, just use a low-ASCII control character (0x01-0x1F

the ? is the default in Scintilla so that’s what I used. I didn’t bother trying a low control character for sort order since ? is a non-word character - who would’ve thought it could even be used in the Notepad++ autocomplete/ directory files?

Hmm… I guess OP?

You present a good idea. I can try some testing. If it works, it’s a simple enough change.

Cheers.

Lycan Thrope

@michael-vincent ,

I installed portable 8.3.1 to test whether what I know changed or not, and after copying my own UDL, FunctionList, Autocomplete and overridemap files over to the portable it works as it should.

Granted, by the way, who decided to include that fx glyph in the Autocomplete keywords and function list? As screenshot shows for me, the only thing different is that glyph, the list still sorts lexocographically.

That glyph is now a distraction. :)

Lee

Alan Kilborn

@lycan-thrope said in Autocomplete Items Order Issue:

who decided to include that fx glyph

That would be @michael-vincent
Or rather, ultimately, @don-ho

Valery Bogomolov

@michael-vincent If Notepad++ could autocomplete snippets - would be very handy to write faster things.

For example, this item

<KeyWord name="error_reporting(0);
ini_set('display_errors',0);
ini_set('display_startup_errors',0);
"/>

inserts each function on the new line.

Before:

After:

I used hexadecimal XML entities in the php.xml autocomplete file, except spaces   that broke the autocomplete box behaviour:

Hexadecimal Decimal Unicode Name
	 or 	 CHARACTER TABULATION

 or 
 LINE FEED (LF)
 or  CARRIAGE RETURN (CR)
  or   SPACE

It’s useful for me. It could be great if somebody could remain the same functionality.

My current Notepad++ v8.3.1 autocompletion behaves the same as in the previous version, except for the order of items in the autocomplete results box.

Cheers.

Michael Vincent

@valery-bogomolov

Yes, what you are describing is the EXACT use case for the code snippets plugins I mentioned and NOT the intended use of the Notepad++/autoComplete/ files.

But alas, @PeterJones had a good suggestion and ASCII record separator 30 (\x1E) seems to be a good alternative that is “lower” in sort order than all visible ASCII characters so should fix the sorting problem if I can get an issue and pull request accepted.

Cheers.

PeterJones

@michael-vincent said in Autocomplete Items Order Issue:

alas …

It’s not just for the snippet-abuse of auto-complete.

if I can get an issue and pull request accepted.

If you need additional justification: Perfectly valid autocomplete word characters (0-9) are sorted incorrectly because of the use of ? as the image separator character:

The following three keywords are sorted in lexicographical sort order. They only have valid word characters (\w) in their keyword. But they still sort incorrectly in the function list:

print_r
print_r0
print_rand

The shorter print_r should always come before anything longer that starts with the same sequence.

But because asc(0)=48=0x30 is less than asc(?)=63=0x3F, the auto-complete will list print_r0 before print_r. This is a bug in the current implementation.

<?xml version="1.0" encoding="UTF-8" ?>
<!--
@author	 Geoffray Warnants - http://www.geoffray.be
@version	1.35.20100625
-->
<NotepadPlus>
	<AutoComplete>
		<KeyWord name="print_r" func="yes">
			<Overload retVal="mixed">
				<Param name="mixed expression"/>
				<Param name="[bool return=false]"/>
			</Overload>
		</KeyWord>
		<KeyWord name="print_r0" func="yes">
			<Overload retVal="mixed">
				<Param name="argument"/>
			</Overload>
		</KeyWord>
		<KeyWord name="print_rand" func="yes"/>
	</AutoComplete>
</NotepadPlus>

Feel free to use this image and minimized auto-completion file in your issue report as justification.

Notepad++ v8.3.1 (64-bit)
Build time : Feb 13 2022 - 15:41:12
Path : C:\usr\local\apps\npp\npp.8.3.1.portable.x64\notepad++.exe
Command Line :
Admin mode : OFF
Local Conf mode : ON
Cloud Config : OFF
OS Name : Windows 10 Enterprise (64-bit)
OS Version : 2009
OS Build : 19042.1526
Current ANSI codepage : 1252
Plugins : mimeTools.dll NppConverter.dll NppExport.dll

But I definitely agree, @Valery-Bogomolov should learn how to use one of the code-snippet tools (I would personally recommend Michael’s fork of the QuickText plugin, as I found that the most intuitive of the snippet-plugins for my use-case), rather than continuing to abuse the auto-complete files for things it wasn’t intended to do.

Michael Vincent

@michael-vincent said in Autocomplete Items Order Issue:

get an issue and pull request accepted

Done and Done. Well, submitted at least, the accepted part will need to wait a bit.

Cheers.

Valery Bogomolov

@peterjones I’m sorry for the “snippet-abuse”, I just wanted to get the most from the Npp’s auto-complete interface. I experimented with autocomplete a lot before developing the parser that generates KeyWords for my favorite language PHP. I wrote about it here

Unexpectedly, I figured out that I was able, for example, to write faster SQL queries code rather than using any other editor for that.

FingerText is not compatible with my configured Notepad++, QuickText is not convenient for me to use, but I use it. NppSnippets is a great plugin, but I have to search for a specific item in a dialog to insert a snippet and it takes some time.

Regards.

Michael Vincent

@valery-bogomolov

You can try the pull request AppVeyor build by downloading, and replacing your current notepad++.exe with the appropriate downloaded, renamed version. Of course, save your current notepad++.exe (notepad++.exe.orig) so you can revert back when done testing.

Cheers.

Alan Kilborn

@valery-bogomolov

@michael-vincent said in Autocomplete Items Order Issue:

You can try the…

More details (if you need them) on exactly how you’d do that are HERE.

Valery Bogomolov

@michael-vincent Thank you for the great work, but personally, I don’t like the new UX of the autocomplete: the fx icon is really distracting and I have to look down a couple of lines to see the first result. I would prefer to have the old autocomplete if it could be possible, sorry

Michael Vincent

@valery-bogomolov said in Autocomplete Items Order Issue:

Thank you for the great work

Assuming that means is sorts as you expect now?

Valery Bogomolov

This post is deleted!