When I ckeck migrate and use local version on my site, jquery version is not inserted in the html of the page.
when I uncheck this option, jquery 1.9 (my version) is correctly inserted in my page.

CommentFileSizeAuthor
#6 migrate_option_broken-2604036-6.patch789 bytess_leu
migrate.png31.27 KBpixelpreview@gmail.com
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pixelpreview created an issue. See original summary.

pixelpreview@gmail.com’s picture

pixelpreview@gmail.com CreditAttribution: pixelpreview@gmail.com commented

I see some errors in the drupal's logs :
Notice: Undefined index: name in variable_children() (line 81 of /opt/drupal/xxx/sites/all/modules/contrib/variable/variable.module).

Notice: Undefined index: module in variable_type_include() (line 473 of /opt/drupal/xxx/sites/all/modules/contrib/variable/variable.module).

Referrer http://xxx/fr/admin/content/migrate
Notice: Undefined index: driver in Database::openConnection() (line 1681 of /opt/drupal/xxx/includes/database/database.inc).

hamuchen’s picture

hamuchen CreditAttribution: hamuchen commented

Same here. When I activate Migrate, jQuery is not loaded (checked HTML source), no matter if CDN or not.

Regards
hamu

hamuchen’s picture

hamuchen CreditAttribution: hamuchen commented

I have found a workaround: the Migrate option only breaks the site, if the "Default jQuery version" is set to "1.4 (Drupal Core)". I thought that option has no effect, because I set a version for the theme.

Regards
hamu

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Version: 7.x-3.0-alpha3 » 7.x-3.x-dev
s_leu’s picture

s_leu CreditAttribution: s_leu at MD Systems GmbH commented
Status: Active » Needs review
FileSize
migrate_option_broken-2604036-6.patch789 bytes

There was an error in hook_library_alter: The hook added a dependency to a library which didn't exist yet. Changed the code to actually provide that referred library and removed some obsolete code below.

Worked for me. Please review.

amme’s picture

amme CreditAttribution: amme commented
Parent issue: » #2852350: [jquery_update] 7.x-3.0 stable release
markhalliwell’s picture

markhalliwell
Primary language English
CreditAttribution: markhalliwell at Tag1 Consulting commented
Status: Needs review » Closed (duplicate)
Parent issue: #2852350: [jquery_update] 7.x-3.0 stable release »
Related issues: +#2692295: jQuery Migrate update to latest version

It was actually a little more complicated than the above patch (when dealing with multiple versions of jQuery Migrate). I went ahead and fixed this in the related issue.