XoopsModules25x/smallworld

Function galleriffic has 596 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    xoops_smallworld.fn.galleriffic = function (settings) {
        //  Extend Gallery Object
        xoops_smallworld.extend(this, {
            // Returns the version of the script
            version: xoops_smallworld.galleriffic.version,

File jquery.galleriffic.js has 663 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/**
 * jQuery Galleriffic plugin
 *
 * Copyright (c) 2008 Trent Foley (http://trentacular.com)
 * Licensed under the MIT License:

Function refresh has 58 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            refresh: function () {
                var imageData = this.currentImage;
                if (!imageData)
                    return this;

Function addImage has 47 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            addImage: function (listItem, thumbExists, insert, position) {
                var xoops_smallworldli = (typeof listItem === "string") ? xoops_smallworld(listItem) : listItem;
                var xoops_smallworldaThumb = xoops_smallworldli.find('a.thumb');
                var slideUrl = xoops_smallworldaThumb.attr('href');
                var title = xoops_smallworldaThumb.attr('title');

Function buildPager has 44 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            buildPager: function (pager) {
                var gallery = this;
                var numPages = this.getNumPages();
                var page = this.getCurrentPage();
                var startIndex = page * this.numThumbs;

Function rebuildThumbs has 36 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            rebuildThumbs: function () {
                var needsPagination = this.data.length > this.numThumbs;

                // Rebuild top pager
                if (this.enableTopPager) {

Function buildImage has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            buildImage: function (imageData, isSync) {
                var gallery = this;
                var nextIndex = this.getNextIndex(imageData.index);

                // Construct new hidden span for the image

Function preloadRecursive has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            preloadRecursive: function (startIndex, currentIndex) {
                // Check if startIndex has been relocated
                if (startIndex != this.preloadStartIndex) {
                    var nextIndex = this.getNextIndex(this.preloadStartIndex);
                    return this.preloadRecursive(this.preloadStartIndex, nextIndex);

Avoid too many return statements within this function.

Open

                return this;

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                if (this.enableBottomPager) {
                    var xoops_smallworldbottomPager = this.find('div.bottom');
                    if (xoops_smallworldbottomPager.length == 0)
                        xoops_smallworldbottomPager = this.append('<div class="bottom pagination"></div>').find('div.bottom');
                    else

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                if (this.enableTopPager) {
                    var xoops_smallworldtopPager = this.find('div.top');
                    if (xoops_smallworldtopPager.length == 0)
                        xoops_smallworldtopPager = this.prepend('<div class="top pagination"></div>').find('div.top');
                    else

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                if (this.xoops_smallworldcontrolsContainer) {
                    this.xoops_smallworldcontrolsContainer
                        .find('div.ss-controls a').removeClass().addClass('play')
                        .attr('title', this.playLinkText)
                        .attr('href', '#play')

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                if (this.xoops_smallworldcontrolsContainer) {
                    this.xoops_smallworldcontrolsContainer
                        .find('div.ss-controls a').removeClass().addClass('pause')
                        .attr('title', this.pauseLinkText)
                        .attr('href', '#pause')

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                    pager.append('<a rel="history" href="#' + this.data[prevPage].hash + '" title="' + this.prevPageLinkText + '">' + this.prevPageLinkText + '</a>');

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                    pager.append('<a rel="history" href="#' + this.data[nextPage].hash + '" title="' + this.nextPageLinkText + '">' + this.nextPageLinkText + '</a>');

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            next: function (dontPause, bypassHistory) {
                this.gotoIndex(this.getNextIndex(this.currentImage.index), dontPause, bypassHistory);
                return this;
            },

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            previous: function (dontPause, bypassHistory) {
                this.gotoIndex(this.getPrevIndex(this.currentImage.index), dontPause, bypassHistory);
                return this;
            },

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                if (page > 0) {
                    var startIndex = page * this.numThumbs;
                    var prevPage = startIndex - this.numThumbs;
                    this.gotoIndex(prevPage, dontPause, bypassHistory);
                }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                if (page < lastPage) {
                    var startIndex = page * this.numThumbs;
                    var nextPage = startIndex + this.numThumbs;
                    this.gotoIndex(nextPage, dontPause, bypassHistory);
                }

• Read up
  Read up