Community
    • Login

    Autocomplete Items Order Issue

    Scheduled Pinned Locked Moved Help wanted · · · – – – · · ·
    35 Posts 6 Posters 4.2k Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • Valery BogomolovV
      Valery Bogomolov @Michael Vincent
      last edited by

      @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.

      1 Reply Last reply Reply Quote 1
      • PeterJonesP
        PeterJones @Valery Bogomolov
        last edited by

        @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 VincentM 1 Reply Last reply Reply Quote 3
        • Michael VincentM
          Michael Vincent @Lycan Thrope
          last edited by

          @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.

          Lycan ThropeL 1 Reply Last reply Reply Quote 1
          • Michael VincentM
            Michael Vincent @PeterJones
            last edited by

            @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.

            PeterJonesP 1 Reply Last reply Reply Quote 1
            • PeterJonesP
              PeterJones @Michael Vincent
              last edited by

              @michael-vincent ,

              4e71340c-3d57-4196-a5ae-fe34206f9bea-image.png

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

              Michael VincentM 1 Reply Last reply Reply Quote 3
              • Michael VincentM
                Michael Vincent @PeterJones
                last edited by

                @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.

                PeterJonesP Valery BogomolovV 2 Replies Last reply Reply Quote 1
                • PeterJonesP
                  PeterJones @Michael Vincent
                  last edited by

                  @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 VincentM 1 Reply Last reply Reply Quote 3
                  • Michael VincentM
                    Michael Vincent @PeterJones
                    last edited by

                    @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.

                    1 Reply Last reply Reply Quote 3
                    • Lycan ThropeL
                      Lycan Thrope @Michael Vincent
                      last edited by

                      @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.

                      NPP8_3_1portableAutoComplete.PNG

                      That glyph is now a distraction. :)

                      Lee

                      Alan KilbornA 1 Reply Last reply Reply Quote 2
                      • Alan KilbornA
                        Alan Kilborn @Lycan Thrope
                        last edited by

                        @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

                        1 Reply Last reply Reply Quote 1
                        • Valery BogomolovV
                          Valery Bogomolov @Michael Vincent
                          last edited by

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

                          For example, this item

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

                          inserts each function on the new line.

                          Before:
                          Npp_Autocomplete_Snippet.png

                          After:
                          Npp_Autocomplete_Snippet_Result.png

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

                          Hexadecimal Decimal Unicode Name
                          &#x09; or &#09; CHARACTER TABULATION
                          &#x0A; or &#10; LINE FEED (LF)
                          &#x0D; or &#13; CARRIAGE RETURN (CR)
                          &#x20; or &#32; 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 VincentM 1 Reply Last reply Reply Quote 0
                          • Michael VincentM
                            Michael Vincent @Valery Bogomolov
                            last edited by

                            @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.

                            PeterJonesP Michael VincentM 2 Replies Last reply Reply Quote 4
                            • PeterJonesP
                              PeterJones @Michael Vincent
                              last edited by

                              @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.

                              744c0861-2f0b-4be1-b58a-714c7e28f382-image.png

                              <?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.

                              Valery BogomolovV 1 Reply Last reply Reply Quote 4
                              • Michael VincentM
                                Michael Vincent @Michael Vincent
                                last edited by

                                @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.

                                1 Reply Last reply Reply Quote 4
                                • Valery BogomolovV
                                  Valery Bogomolov @PeterJones
                                  last edited by

                                  @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 VincentM Alan KilbornA 2 Replies Last reply Reply Quote 0
                                  • Michael VincentM
                                    Michael Vincent @Valery Bogomolov
                                    last edited by

                                    @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.

                                    Valery BogomolovV 1 Reply Last reply Reply Quote 3
                                    • Alan KilbornA
                                      Alan Kilborn @Valery Bogomolov
                                      last edited by

                                      @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.

                                      1 Reply Last reply Reply Quote 3
                                      • Valery BogomolovV
                                        Valery Bogomolov @Michael Vincent
                                        last edited by

                                        @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 VincentM 1 Reply Last reply Reply Quote 1
                                        • Michael VincentM
                                          Michael Vincent @Valery Bogomolov
                                          last edited by

                                          @valery-bogomolov said in Autocomplete Items Order Issue:

                                          Thank you for the great work

                                          Assuming that means is sorts as you expect now?

                                          Valery BogomolovV 2 Replies Last reply Reply Quote 1
                                          • Valery BogomolovV
                                            Valery Bogomolov @Michael Vincent
                                            last edited by Valery Bogomolov

                                            This post is deleted!
                                            1 Reply Last reply Reply Quote 0
                                            • First post
                                              Last post
                                            The Community of users of the Notepad++ text editor.
                                            Powered by NodeBB | Contributors