mattvanhorn/BJJLife

Function prettyPhoto has 590 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 615 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.4

Function open has 130 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;
                if($.browser.msie && $.browser.version == 6) settings.theme = "light_square"; // Fallback to a supported theme for IE6
                pp_images = $.makeArray(arguments[0]);

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));

Function _getDimensions has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        function _getDimensions(width,height){
            width = parseFloat(width);
            height = parseFloat(height);
            
            // Get the details height, to do so, I need to clone it since it's invisible

Consider simplifying this complex logical expression.

Open

            if(isSet && settings.overlay_gallery && _getFileType(pp_images[set_position])=="image" && (settings.ie6_fallback && !($.browser.msie && parseInt($.browser.version) == 6))) {
                itemWidth = 52+5; // 52 beign the thumb width, 5 being the right margin.
                navWidth = (settings.theme == "facebook" || settings.theme == "pp_default") ? 50 : 30; // Define the arrow width depending on the theme
                
                itemsPerPage = Math.floor((pp_dimensions['containerWidth'] - 100 - navWidth) / itemWidth);

Avoid deeply nested control flow statements.

Open

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

Avoid too many return statements within this function.

Open

                return 'iframe';

Avoid too many return statements within this function.

Open

                return 'inline';

Avoid too many return statements within this function.

Open

                return 'custom';

Avoid too many return statements within this function.

Open

                return 'image';

Avoid too many return statements within this function.

Open

                return 'ajax';

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_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_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.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.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.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

            if($.browser.msie && $.browser.version == 6) $('select').css('visibility','hidden'); // To fix the bug with IE select boxes

• Read up
  Read up

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

Open

                if($.browser.msie && $.browser.version == 6) $('select').css('visibility','visible'); // To fix the bug with IE select boxes

• 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