wikimedia/mediawiki-core

File BookletLayout.js has 361 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/* global moment */
( function () {

    /**
     * @classdesc Encapsulates the process of uploading a file

Function getErrorMessageForStateDetails has 71 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    mw.Upload.BookletLayout.prototype.getErrorMessageForStateDetails = function () {
        var state = this.upload.getState(),
            stateDetails = this.upload.getStateDetails(),
            warnings = stateDetails.upload && stateDetails.upload.warnings,
            $ul = $( '<ul>' ),

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

Open

    mw.Upload.BookletLayout.prototype.renderInfoForm = function () {
        var fieldset;

        this.filePreview = new OO.ui.Widget( {
            classes: [ 'mw-upload-bookletLayout-filePreview' ]

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

Open

    mw.Upload.BookletLayout.prototype.uploadFile = function () {
        var deferred = $.Deferred(),
            startTime = mw.now(),
            layout = this,
            file = this.getFile();

Function initialize has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    mw.Upload.BookletLayout.prototype.initialize = function () {
        var booklet = this;

        this.clear();
        this.upload = this.createUpload();

Function BookletLayout has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    mw.Upload.BookletLayout = function ( config ) {
        // Parent constructor
        mw.Upload.BookletLayout.super.call( this, config );

        this.$overlay = config.$overlay;

Avoid too many return statements within this function.

Open

                return $.Deferred().resolve( new OO.ui.Error(
                    $( '<p>' ).msg( 'filename-bad-prefix', warnings[ 'bad-prefix' ] ),
                    { recoverable: false }
                ) );

Avoid too many return statements within this function.

Open

                return $.Deferred().resolve( new OO.ui.Error(
                    $( '<p>' ).msg( 'file-deleted-duplicate', 'File:' + warnings[ 'duplicate-archive' ] ),
                    { recoverable: false }
                ) );

Avoid too many return statements within this function.

Open

                return $.Deferred().resolve( new OO.ui.Error(
                    $( '<p>' ).msg( 'file-exists-duplicate', warnings.duplicate.length ).append( $ul ),
                    { recoverable: false }
                ) );

Avoid too many return statements within this function.

Open

                return $.Deferred().resolve( new OO.ui.Error(
                    $( '<p>' ).msg( 'filewasdeleted', 'File:' + warnings[ 'was-deleted' ] ),
                    { recoverable: false }
                ) );

Avoid too many return statements within this function.

Open

                return $.Deferred().resolve( new OO.ui.Error(
                    $( '<p>' ).msg( 'badfilename', warnings.badfilename )
                ) );

Avoid too many return statements within this function.

Open

                return $.Deferred().resolve( new OO.ui.Error(
                    // Let's get all the help we can if we can't pin point the error
                    $( '<p>' ).msg( 'api-error-unknown-warning', JSON.stringify( stateDetails ) ),
                    { recoverable: false }
                ) );

Avoid too many return statements within this function.

Open

                return $.Deferred().resolve( new OO.ui.Error(
                    $( '<p>' ).msg( 'filename-thumb-name' ),
                    { recoverable: false }
                ) );

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

Open

            } else if ( warnings[ 'exists-normalized' ] !== undefined ) {
                return $.Deferred().resolve( new OO.ui.Error(
                    $( '<p>' ).msg( 'fileexists', 'File:' + warnings[ 'exists-normalized' ] ),
                    { recoverable: false }
                ) );

• Read up
  Read up

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

Open

            } else if ( warnings[ 'duplicate-archive' ] !== undefined ) {
                return $.Deferred().resolve( new OO.ui.Error(
                    $( '<p>' ).msg( 'file-deleted-duplicate', 'File:' + warnings[ 'duplicate-archive' ] ),
                    { recoverable: false }
                ) );

• Read up
  Read up

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

Open

            } else if ( warnings[ 'page-exists' ] !== undefined ) {
                return $.Deferred().resolve( new OO.ui.Error(
                    $( '<p>' ).msg( 'filepageexists', 'File:' + warnings[ 'page-exists' ] ),
                    { recoverable: false }
                ) );

• Read up
  Read up

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

Open

            } else if ( warnings[ 'was-deleted' ] !== undefined ) {
                return $.Deferred().resolve( new OO.ui.Error(
                    $( '<p>' ).msg( 'filewasdeleted', 'File:' + warnings[ 'was-deleted' ] ),
                    { recoverable: false }
                ) );

• Read up
  Read up

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

Open

            new OO.ui.FieldLayout( this.descriptionWidget, {
                label: mw.msg( 'upload-form-label-infoform-description' ),
                align: 'top',
                help: mw.msg( 'upload-form-label-infoform-description-tooltip' )
            } )

• Read up
  Read up

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

Open

            new OO.ui.FieldLayout( this.filenameWidget, {
                label: mw.msg( 'upload-form-label-infoform-name' ),
                align: 'top',
                help: mw.msg( 'upload-form-label-infoform-name-tooltip' )
            } ),

• Read up
  Read up

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

Open

    mw.Upload.BookletLayout.prototype.getText = function () {
        return this.descriptionWidget.getValue();
    };

• Read up
  Read up

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

Open

        this.on( 'fileUploadProgress', function ( progress ) {
            this.progressBarWidget.setProgress( progress * 100 );
        }.bind( this ) );

• Read up
  Read up