krafthaus/bauhaus

File plugin.js has 573 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/**
 * plugin.js
 *
 * Copyright, Moxiecode Systems AB
 * Released under LGPL License.

Function updateHtml has 125 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function updateHtml(html, data, updateAll) {
        var writer = new tinymce.html.Writer();
        var sourceCount = 0, hasImage;

        function setAttributes(attrs, updatedAttrs) {

Function showDialog has 123 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function showDialog() {
        var win, width, height, data;

        var generalFormItems = [
            {

Function dataToHtml has 82 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function dataToHtml(data) {
        var html = '';

        if (!data.source1) {
            tinymce.extend(data, htmlToData(data.embed));

Function start has 51 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            start: function(name, attrs, empty) {
                switch (name) {
                    case "video":
                    case "object":
                    case "embed":

Function htmlToData has 43 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function htmlToData(html) {
        var data = {};

        new tinymce.html.SaxParser({
            validate: false,

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

Open

            start: function(name, attrs) {
                if (!data.source1 && name == "param") {
                    data.source1 = attrs.map.movie;
                }

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

Open

            end: function(name) {
                if (name == "video" && updateAll) {
                    for (var index = 1; index <= 2; index++) {
                        if (data["source" + index]) {
                            var attrs = [];

Avoid deeply nested control flow statements.

Open

                if (editor.settings.video_template_callback) {
                    html = editor.settings.video_template_callback(data);
                } else {
                    html = (
                        '<video width="' + data.width + '" height="' + data.height + '"' + (data.poster ? ' poster="' + data.poster + '"' : '') + ' controls="controls">\n' +

Avoid deeply nested control flow statements.

Open

                            if (value) {
                                attrs.map[name] = value;
                                attr.value = value;
                            } else {
                                delete attrs.map[name];

Avoid too many return statements within this function.

Open

            return 'video/ogg';

Avoid too many return statements within this function.

Open

        return '';

Avoid too many return statements within this function.

Open

            return 'application/x-shockwave-flash';

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

Open

        if (editor.settings.media_poster !== false) {
            generalFormItems.push({name: 'poster', type: 'filepicker', filetype: 'image', size: 40, label: 'Poster'});
        }

• Read up
  Read up

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

Open

        if (editor.settings.media_alt_source !== false) {
            generalFormItems.push({name: 'source2', type: 'filepicker', filetype: 'media', size: 40, label: 'Alternative source'});
        }

• Read up
  Read up

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

Open

                        '<audio controls="controls" src="' + data.source1 + '">' +
                            (data.source2 ? '\n<source src="' + data.source2 + '"' + (data.source2mime ? ' type="' + data.source2mime + '"' : '') + ' />\n' : '') +

• Read up
  Read up

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

Open

                        '<video width="' + data.width + '" height="' + data.height + '"' + (data.poster ? ' poster="' + data.poster + '"' : '') + ' controls="controls">\n' +
                            '<source src="' + data.source1 + '"' + (data.source1mime ? ' type="' + data.source1mime + '"' : '') + ' />\n' +
                            (data.source2 ? '<source src="' + data.source2 + '"' + (data.source2mime ? ' type="' + data.source2mime + '"' : '') + ' />\n' : '') +

• Read up
  Read up

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

Open

            if (data.type == "iframe") {
                html += '<iframe src="' + data.source1 + '" width="' + data.width + '" height="' + data.height + '"></iframe>';
            } else if (data.source1mime == "application/x-shockwave-flash") {
                html += '<object data="' + data.source1 + '" width="' + data.width + '" height="' + data.height + '" type="application/x-shockwave-flash">';

• Read up
  Read up

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

Open

                if (data.poster) {
                    html += '<img src="' + data.poster + '" width="' + data.width + '" height="' + data.height + '" />';
                }

• Read up
  Read up

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

Open

                    tinymce.each(e.meta, function(value, key) {
                        win.find('#' + key).value(value);
                    });

• Read up
  Read up