Problem/Motivation
============
Status report shows that Nivo Slider does not have the required jQuery version (>=1.7). However, jQuery 1.10.2 has been installed, and jQuery Update module has been configured accordingly.

Nivo Slider jQuery version jQuery 1.10.2
Nivo Slider requires jQuery 1.7 or greater. Configure jQuery Update.

Proposed Resolution
============

In nivo_slider.module, replace

36: // Check if the site is running >= jQuery 1.7
37: if (($library = drupal_get_library('system', 'jquery')) && $library['version'] >= 1.7) {

with

36: // Check if the site is running >= jQuery 1.7
37: if (($library = drupal_get_library('system', 'jquery')) && version_compare($library['version'], 1.7) >= 0) {

Remaining tasks
==========
The patch is ready for review.

CommentFileSizeAuthor
nivo_slider.module.patch606 bytessusukacang
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

susukacang created an issue. See original summary.

susukacang’s picture

Assigned: susukacang » Unassigned
kopanda’s picture

The patch file works. Thanks!

andrey.troeglazov’s picture

Assigned: Unassigned » andrey.troeglazov

andrey.troeglazov’s picture

Assigned: andrey.troeglazov » Unassigned
Status: Active » Fixed

Commited into dev branch. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

andrey.troeglazov’s picture