JS error on mousemove, $.throttle

[DavidThomas48]

Describe the bug

js error message on screen

To Reproduce

can't

Expected behavior

Screenshots

If applicable, add screenshots to help explain your problem.

Server configuration

Win10 64 XAMPP

Client configuration

Chrome

Additional context

Add any other context about the problem here.

[Tithugues]

@DavidThomas48 , would it be please possible to get the screenshot of the whole page with the errors displayed?
Can you please confirm that you have the error on the home page?

[DavidThomas48]

Hi Hugues, If it happens again I will do that. I can tell you that it happens when I resize the Routines editor dialog. David

[DavidThomas48]

Hello Hugues , I just experienced the problem and it happened when I dragged the divider to the right to widen the left sidebar (database navigation tree). I had previously dragged it left to make the sidebar very narrow, then done some other things. My previous comment about it being related to the routines dialog may be incorrect. I think I may not have noticed the error for a moment, and started using the routine editor dialog in the meantime. I hope that comment has not been a distraction - if so I apologise. Hope that helps. David

[DavidThomas48]

Hi again,
Re "Can you please confirm that you have the error on the home page?" - no this is not the case. It happens when I am working in tables or routines. I am rarely on the home page.

[williamdes]

Reading the code it seems like js/vendor/jquery/jquery.debounce-1.0.5.js did throttle the call and then invoke it but forgetting the event data

[williamdes]

@DavidThomas48 does this patch solve this issue for you ?

diff --git a/js/navigation.js b/js/navigation.js
index 169d4e40592..43f104ce2f9 100644
--- a/js/navigation.js
+++ b/js/navigation.js
@@ -1298,10 +1298,12 @@ Navigation.ResizeHandler = function () {
      */
     this.mousemove = function (event) {
         event.preventDefault();
-        var pos = event.data.resize_handler.getPos(event);
-        event.data.resize_handler.setWidth(pos);
-        if ($('.sticky_columns').length !== 0) {
-            Sql.handleAllStickyColumns();
+        if (event.data.resize_handler) {
+            var pos = event.data.resize_handler.getPos(event);
+            event.data.resize_handler.setWidth(pos);
+            if ($('.sticky_columns').length !== 0) {
+                Sql.handleAllStickyColumns();
+            }
         }
     };
     /**

[DavidThomas48]

Hello William, I am working on a WordPress site that contains only one copy of navigation.js which is only 3.66Kb and does not contain Navigation.ResizeHandler = function () {… I should add that the error happens very rarely – I had not seen it for well over a month until the other day and I am working in pma every day. Regards David Thomas

[DavidThomas48]

Sorry William – senior moment – just making the patch now but I really won’t know if it has fixed the problem because of the rarity of the error. DT

[williamdes]

Hi

From what you commented I can understand you where probably no up to date :)
I am not sure that I will post my patch, can you let me know if you where using a 5.0 version?

[DavidThomas48]

I have installed the patch on my server. If the error hasn’t happened by August, it’s probably fixed! D

[DavidThomas48]

I just realised, I have been working online with a host site recently, and they are running pma 4.9.1. I am fairly sure that the error occurred on my machine (pma 5.0.2), but it is possible that the error occurred on that site. Sorry to be vague about this. Doesn’t the error report give you the pma version? David

[williamdes]

Indeed, I got confused but you where running 5.0.1 when you reported this issue, I will commit the patch ASAP then

William

[DavidThomas48]

Hi William, I just had the error again but this was definitely in pma 4.9.1. I noticed that the error report includes the pma version. Just to be sure, if you still have my earlier report, please check whether that error happened in 5.0.2 or 4.9.1 - I am working on both at the moment. All the best David

[williamdes]

Hi,
I committed 9b66cf1 to fix the issue you reported.
I hope it will be okay to fix this issue

Kind regards,
William