Implement NPPN_GLOBALMODIFIED callback

not sure if it was necessary, but now it's done
molsonkiko · Feb 24, 2024 · d7edad1 · d7edad1
1 parent 4dbb720
commit d7edad1
Show file tree

Hide file tree

Showing 5 changed files with 65 additions and 45 deletions.
diff --git a/JsonToolsNppPlugin/Forms/TreeViewer.cs b/JsonToolsNppPlugin/Forms/TreeViewer.cs
@@ -53,7 +53,7 @@ public partial class TreeViewer : Form
         public bool documentTypeIndexChangeWasAutomatic;
 
         /// <summary>
-        /// If the user performs an undo or redo action,
+        /// If the user modifies the buffer belonging to this treeview,
         /// this will be set to true so that the next time the user performs a RemesPath query,
         /// the treeview is reset beforehand.
         /// </summary>

diff --git a/JsonToolsNppPlugin/Main.cs b/JsonToolsNppPlugin/Main.cs
@@ -350,11 +350,20 @@ public static void OnNotification(ScNotification notification)
             // if the user did nothing for a while (default 1 second) after editing,
             // re-parse the file and also perform validation if that's enabled.
             case (uint)SciMsg.SCN_MODIFIED:
-                // only turn on the flag if the user performed the modification
                 lastEditedTime = System.DateTime.UtcNow;
                 if (openTreeViewer != null)
                     openTreeViewer.shouldRefresh = true;
                 break;
+            // a find/replace action was performed.
+            // Tag the treeview of the modified file as needing to refresh, and indicate that the document was edited.
+            case (uint)NppMsg.NPPN_GLOBALMODIFIED:
+                IntPtr bufferModifiedId = notification.Header.hwndFrom;
+                string bufferModified = Npp.notepad.GetFilePath(bufferModifiedId);
+                if (bufferModified == activeFname)
+                    lastEditedTime = System.DateTime.UtcNow;
+                if (TryGetInfoForFile(bufferModified, out info) && !(info.tv is null) && !info.tv.IsDisposed)
+                    info.tv.shouldRefresh = true;
+                break;
                 //if (code > int.MaxValue) // windows messages
                 //{
                 //    int wm = -(int)code;

diff --git a/JsonToolsNppPlugin/PluginInfrastructure/Msgs_h.cs b/JsonToolsNppPlugin/PluginInfrastructure/Msgs_h.cs
@@ -834,6 +834,17 @@ public enum NppMsg : uint
         /// </summary>
         NPPN_FILEDELETED = NPPN_FIRST + 26,
 
+        /// <summary>
+        /// To notify plugins that the current document is just modified by Replace All action.<br></br>
+        /// For solving the performance issue (from v8.6.4), Notepad++ doesn't trigger SCN_MODIFIED during Replace All action anymore.<br></br>
+        /// As a result, the plugins which monitor SCN_MODIFIED should also monitor NPPN_GLOBALMODIFIED.<br></br>
+        /// <strong>This notification is implemented in Notepad++ v8.6.5.</strong><br></br>
+        /// scnNotification->nmhdr.code = NPPN_GLOBALMODIFIED;<br></br>
+        /// scnNotification->nmhdr.hwndFrom = BufferID;<br></br>
+        /// scnNotification->nmhdr.idFrom = 0; // preserved for the future use, must be zero
+        /// </summary>
+        NPPN_GLOBALMODIFIED = NPPN_FIRST + 30,
+
         /* --Autogenerated -- end of section automatically generated from notepad-plus-plus\PowerEditor\src\MISC\PluginsManager\Notepad_plus_msgs.h * */
     }
 

diff --git a/JsonToolsNppPlugin/Properties/AssemblyInfo.cs b/JsonToolsNppPlugin/Properties/AssemblyInfo.cs
@@ -28,5 +28,5 @@
 //      Build Number
 //      Revision
 //
-[assembly: AssemblyVersion("7.0.0.3")]
-[assembly: AssemblyFileVersion("7.0.0.3")]
+[assembly: AssemblyVersion("7.0.0.4")]
+[assembly: AssemblyFileVersion("7.0.0.4")]