[Resolved] GP Premium and WP Migrate DB Pro conflict: admin.js attaches to .backup-options

Home Forums Support [Resolved] GP Premium and WP Migrate DB Pro conflict: admin.js attaches to .backup-options

Home Forums Support GP Premium and WP Migrate DB Pro conflict: admin.js attaches to .backup-options

Viewing 6 posts - 1 through 6 (of 6 total)
  • Author
    Posts
  • #743473
    Dragomir

    Hi,

    Just dug that out – in admin.js (329) you attach a JS function to the .backup-options selector. The trouble is there is a <div> element with the same class present in the WP Migrate DB Pro “Configure Profile” screen (when Pull or Push is selected).

    As a result, when I open the WPMDB screen and click on that div, there is a JS error in the console (“Uncaught TypeError: Cannot read property ‘slug’ of undefined”) and the div is set to display:none => so I cannot change the relevant options in that div (they refer to backing up the DB while doing the migration).

    I cannot write to the WPMDB dev since the license is not mine and the person who owns it is hard to find … long story πŸ™‚ Could you please check and contact him if there is a need to?

    Thank you in advance!

    #743647
    Dragomir

    Sorry to bump but I see other requests being answered and this one – no. Is there anything else I have to provide as information?

    Thank you.

    #743669
    David
    Staff
    Customer Support

    Hi there,

    sorry for the delay, i have passed this onto Tom as some of this is outside my scope. He lives in a different time zone to me so won’t be on for at least a couple of hours.

    #743980
    Tom
    Lead Developer
    Lead Developer

    Thanks for the heads up! Will get this fixed in the next version.

    For now, go to wp-content/plugins/gp-premium/sites/assets/js/admin.js and find this:

    $( '.backup-options' )

    And replace it with:

    $( '.site-box .backup-options' )

    #744005
    Dragomir

    Thanks, Tom! πŸ™‚

    #744243
    Tom
    Lead Developer
    Lead Developer

    No problem – thanks again for the bug report πŸ™‚

Viewing 6 posts - 1 through 6 (of 6 total)
  • You must be logged in to reply to this topic.