Bruno17 / MIGX

MIGX for revo 2.2 and above
83 stars 78 forks source link

loadWin method properties #383

Closed Jako closed 2 years ago

Jako commented 2 years ago

Some method calls of this.loadWin use different method properties than the method itself is providing:

update: function(btn,e) {
      this.loadWin(btn,e,'u');
    },

vs

,loadWin: function(btn,e,index,action,tempParams) {

Maybe it is a local configuration issue.

Jako commented 2 years ago

At least the first result is an issue:

https://github.com/Bruno17/MIGX/search?q=update%3A+function%28btn%2Ce%29

Jako commented 2 years ago

This issue occured after upgrading MIGX from some ancient version and can be solved by changing the columnbuttons from update to edit_migx and so on. The previous developer of the site maybe created some wrong MIGX configuration or there was some breaking change in between.