woothemes/woocommerce

Function prettyPhoto has 585 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    $.fn.prettyPhoto = function(pp_settings) {
        pp_settings = jQuery.extend({
            hook: 'rel', /* the attribute tag to use for prettyPhoto hooks. default: 'rel'. For HTML5, use "data-rel" or similar. */
            animation_speed: 'fast', /* fast/slow/normal */
            ajaxcallback: function() {},

File jquery.prettyPhoto.js has 611 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/* ------------------------------------------------------------------------
    Class: prettyPhoto
    Use: Lightbox clone for jQuery
    Author: Stephane Caron (http://www.no-margin-for-errors.com)
    Version: 3.1.6

Function open has 128 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        $.prettyPhoto.open = function(event) {
            if(typeof settings == "undefined"){ // Means it's an API call, need to manually get the settings and set the variables
                settings = pp_settings;
                pp_images = $.makeArray(arguments[0]);
                pp_titles = (arguments[1]) ? $.makeArray(arguments[1]) : $.makeArray("");

Function _build_overlay has 91 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        function _build_overlay(caller){
            // Inject Social Tool markup into General markup
            if(settings.social_tools)
                facebook_like_link = settings.social_tools.replace('{location_href}', encodeURIComponent(location.href));

Function _fitToViewport has 31 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        function _fitToViewport(width,height){
            resized = false;

            _getDimensions(width,height);

Function _showContent has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        function _showContent(){
            $('.pp_loaderIcon').hide();

            // Calculate the opened top position of the pic holder
            projectedTop = scroll_pos['scrollTop'] + ((windowHeight/2) - (pp_dimensions['containerHeight']/2));

Avoid deeply nested control flow statements.

Open

                                if(!settings.modal)
                                $.prettyPhoto.close();

Avoid too many return statements within this function.

Open

                return 'ajax';

Avoid too many return statements within this function.

Open

                return 'custom';

Avoid too many return statements within this function.

Open

                return 'iframe';

Avoid too many return statements within this function.

Open

                return 'image';

Avoid too many return statements within this function.

Open

                return 'inline';

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

Open

            $pp_pic_holder.find('.pp_pause').unbind('click').removeClass('pp_pause').addClass('pp_play').click(function(){
                $.prettyPhoto.startSlideshow();
                return false;
            });

• Read up
  Read up

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

Open

                $pp_pic_holder.find('.pp_play').unbind('click').removeClass('pp_play').addClass('pp_pause').click(function(){
                    $.prettyPhoto.stopSlideshow();
                    return false;
                });

• Read up
  Read up

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

Open

            $pp_pic_holder.find('.pp_previous, .pp_nav .pp_arrow_previous').bind('click',function(){
                $.prettyPhoto.changePage('previous');
                $.prettyPhoto.stopSlideshow();
                return false;
            });

• Read up
  Read up

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

Open

            $pp_pic_holder.find('.pp_next, .pp_nav .pp_arrow_next').bind('click',function(){
                $.prettyPhoto.changePage('next');
                $.prettyPhoto.stopSlideshow();
                return false;
            });

• Read up
  Read up

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

Open

                $pp_gallery.find('.pp_arrow_next').click(function(){
                    $.prettyPhoto.changeGalleryPage('next');
                    $.prettyPhoto.stopSlideshow();
                    return false;
                });

• Read up
  Read up

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

Open

                $pp_gallery.find('.pp_arrow_previous').click(function(){
                    $.prettyPhoto.changeGalleryPage('previous');
                    $.prettyPhoto.stopSlideshow();
                    return false;
                });

• Read up
  Read up

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

Open

                        toInject = settings.quicktime_markup.replace(/{width}/g,pp_dimensions['width']).replace(/{height}/g,pp_dimensions['height']).replace(/{wmode}/g,settings.wmode).replace(/{path}/g,pp_images[set_position]).replace(/{autoplay}/g,settings.autoplay);

• Read up
  Read up

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

Open

                        toInject = settings.iframe_markup.replace(/{width}/g,pp_dimensions['width']).replace(/{height}/g,pp_dimensions['height']).replace(/{wmode}/g,settings.wmode).replace(/{path}/g,movie);

• Read up
  Read up

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

Open

                        toInject =  settings.flash_markup.replace(/{width}/g,pp_dimensions['width']).replace(/{height}/g,pp_dimensions['height']).replace(/{wmode}/g,settings.wmode).replace(/{path}/g,filename+'?'+flash_vars);

• Read up
  Read up

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

Open

            if(movie_width.indexOf('%') != -1) { movie_width = parseFloat(($(window).width() * parseFloat(movie_width) / 100) - 150); percentBased = true; }

• Read up
  Read up

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

Open

            if(movie_height.indexOf('%') != -1) { movie_height = parseFloat(($(window).height() * parseFloat(movie_height) / 100) - 150); percentBased = true; }

• Read up
  Read up

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

Open

            movie_height = ( parseFloat(getParam('height',pp_images[set_position])) ) ? getParam('height',pp_images[set_position]) : settings.default_height.toString();

• Read up
  Read up

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

Open

            movie_width = ( parseFloat(getParam('width',pp_images[set_position])) ) ? getParam('width',pp_images[set_position]) : settings.default_width.toString();

• Read up
  Read up