diff --git a/client/galaxy/scripts/galaxy.pages.js b/client/galaxy/scripts/galaxy.pages.js index 2f3203f0d79..830a30e8e75 100644 --- a/client/galaxy/scripts/galaxy.pages.js +++ b/client/galaxy/scripts/galaxy.pages.js @@ -99,7 +99,7 @@ function get_item_info( dialog_type ) // Make an item importable. function make_item_importable( item_controller, item_id, item_type ) { - ajax_url = set_accessible_url.replace( "ITEM_CONTROLLER", item_controller ); + var ajax_url = set_accessible_url.replace( "ITEM_CONTROLLER", item_controller ); $.ajax({ type: "POST", url: ajax_url, @@ -273,9 +273,9 @@ WYMeditor.editor.prototype.dialog = function( dialogType, dialogFeatures, bodyHt newCaption.innerHTML = sCaption; //we create the rows and cells - for(x=0; x'; } - for (i in options.global_actions) { + for (var i in options.global_actions) { var action = options.global_actions[i]; tmpl += '
  • ' + '' + action.label + '' + @@ -158,7 +158,7 @@ return { } // Data columns - for (j in options.columns) { + for (var j in options.columns) { var column = options.columns[j]; if (column.visible) { // Nowrap @@ -242,6 +242,7 @@ return { // Set max page. var max_range = page_link_range + min_offset; var max_page = cur_page_num + max_range; + var max_offset; if (max_page <= num_pages) { // Max page is fine. max_offset = 0; @@ -307,7 +308,7 @@ return { 'For selected items: '; // configure buttons for operations - for (i in options.operations) { + for (var i in options.operations) { var operation = options.operations[i]; if (operation.allow_multiple) { tmpl += ' '; @@ -554,14 +555,14 @@ return { // add category filters var seperator = false; - for (cf_label in options.categorical_filters[column_key]) { + for (var cf_label in options.categorical_filters[column_key]) { // get category filter var cf = options.categorical_filters[column_key][cf_label]; // each filter will have only a single argument, so get that single argument var cf_key = ''; var cf_arg = ''; - for (key in cf) { + for (var key in cf) { cf_key = key; cf_arg = cf[key]; } diff --git a/client/galaxy/scripts/mvc/grid/grid-view.js b/client/galaxy/scripts/mvc/grid/grid-view.js index 6fae6060e62..eb837f7942c 100644 --- a/client/galaxy/scripts/mvc/grid/grid-view.js +++ b/client/galaxy/scripts/mvc/grid/grid-view.js @@ -583,7 +583,7 @@ return Backbone.View.extend({ this.grid.set('async', false); // get slide status - advanced_search = this.$el.find('#advanced-search').is(':visible'); + var advanced_search = this.$el.find('#advanced-search').is(':visible'); this.grid.set('advanced_search', advanced_search); // get default url diff --git a/client/galaxy/scripts/mvc/library/library-dataset-view.js b/client/galaxy/scripts/mvc/library/library-dataset-view.js index 751f862c18f..ddc4f556493 100644 --- a/client/galaxy/scripts/mvc/library/library-dataset-view.js +++ b/client/galaxy/scripts/mvc/library/library-dataset-view.js @@ -497,7 +497,7 @@ var LibraryDatasetView = Backbone.View.extend({ * Extract the role ids from Select2 elements's 'data' */ _extractIds: function(roles_list){ - ids_list = []; + var ids_list = []; for (var i = roles_list.length - 1; i >= 0; i--) { ids_list.push(roles_list[i].id); }; @@ -531,7 +531,7 @@ var LibraryDatasetView = Backbone.View.extend({ mod_utils.get({ url : Galaxy.root + "api/datatypes?extension_only=False", success : function( datatypes ) { - for (key in datatypes) { + for (var key in datatypes) { that.list_extensions.push({ id : datatypes[key].extension, text : datatypes[key].extension, @@ -548,7 +548,7 @@ var LibraryDatasetView = Backbone.View.extend({ mod_utils.get({ url : Galaxy.root + "api/genomes", success : function( genomes ) { - for ( key in genomes ) { + for (var key in genomes ) { that.list_genomes.push({ id : genomes[key][1], text : genomes[key][0] diff --git a/client/galaxy/scripts/mvc/library/library-folder-view.js b/client/galaxy/scripts/mvc/library/library-folder-view.js index ed3c30ed2b9..a66019ccbd7 100644 --- a/client/galaxy/scripts/mvc/library/library-folder-view.js +++ b/client/galaxy/scripts/mvc/library/library-folder-view.js @@ -181,7 +181,7 @@ var FolderView = Backbone.View.extend({ * Extract the role ids from Select2 elements's 'data' */ _extractIds: function(roles_list){ - ids_list = []; + var ids_list = []; for (var i = roles_list.length - 1; i >= 0; i--) { ids_list.push(roles_list[i].id); }; diff --git a/client/galaxy/scripts/mvc/library/library-folderlist-view.js b/client/galaxy/scripts/mvc/library/library-folderlist-view.js index 644f9fdcd38..df4467b5e84 100644 --- a/client/galaxy/scripts/mvc/library/library-folderlist-view.js +++ b/client/galaxy/scripts/mvc/library/library-folderlist-view.js @@ -112,7 +112,7 @@ var FolderListView = Backbone.View.extend({ // when dataset_id is present render its details too if ( this.options.dataset_id ){ - row = _.findWhere( that.rowViews, { id: this.options.dataset_id } ); + var row = _.findWhere( that.rowViews, { id: this.options.dataset_id } ); if ( row ) { row.showDatasetDetails(); } else { @@ -277,7 +277,7 @@ var FolderListView = Backbone.View.extend({ // Iterate each checkbox $(':checkbox', '#folder_list_body').each(function() { this.checked = selected; - $row = $(this.parentElement.parentElement); + var $row = $(this.parentElement.parentElement); // Change color of selected/unselected if (selected) { that.makeDarkRow($row); diff --git a/client/galaxy/scripts/mvc/library/library-folderrow-view.js b/client/galaxy/scripts/mvc/library/library-folderrow-view.js index ebb2d5e2822..138e487f180 100644 --- a/client/galaxy/scripts/mvc/library/library-folderrow-view.js +++ b/client/galaxy/scripts/mvc/library/library-folderrow-view.js @@ -66,7 +66,7 @@ var FolderRowView = Backbone.View.extend({ * the filling of the row template of a given folder. */ prepareButtons: function(folder){ - vis_config = this.options.visibility_config; + var vis_config = this.options.visibility_config; if (this.options.edit_mode === false){ vis_config.save_folder_btn = false; vis_config.cancel_folder_btn = false; diff --git a/client/galaxy/scripts/mvc/library/library-foldertoolbar-view.js b/client/galaxy/scripts/mvc/library/library-foldertoolbar-view.js index ef8ee193f0c..0c301fa1c1b 100644 --- a/client/galaxy/scripts/mvc/library/library-foldertoolbar-view.js +++ b/client/galaxy/scripts/mvc/library/library-foldertoolbar-view.js @@ -161,7 +161,8 @@ var FolderToolbarView = Backbone.View.extend({ var folderDetails = this.serialize_new_folder(); if (this.validate_new_folder(folderDetails)){ var folder = new mod_library_model.FolderAsModel(); - url_items = Backbone.history.fragment.split('/'); + var url_items = Backbone.history.fragment.split('/'), + current_folder_id; if(url_items.indexOf('page') > -1){ current_folder_id = url_items[url_items.length-3]; }else { @@ -317,7 +318,7 @@ var FolderToolbarView = Backbone.View.extend({ updateProgress: function(){ this.progress += this.progressStep; $( '.progress-bar-import' ).width( Math.round( this.progress ) + '%' ); - txt_representation = Math.round( this.progress ) + '% Complete'; + var txt_representation = Math.round( this.progress ) + '% Complete'; $( '.completion_span' ).text( txt_representation ); }, @@ -431,7 +432,7 @@ var FolderToolbarView = Backbone.View.extend({ url : Galaxy.root + "api/datatypes?extension_only=False", success : function( datatypes ) { that.list_extensions = []; - for (key in datatypes) { + for (var key in datatypes) { that.list_extensions.push({ id : datatypes[key].extension, text : datatypes[key].extension, @@ -450,7 +451,7 @@ var FolderToolbarView = Backbone.View.extend({ url : Galaxy.root + "api/genomes", success : function( genomes ) { that.list_genomes = []; - for ( key in genomes ) { + for (var key in genomes ) { that.list_genomes.push({ id : genomes[key][1], text : genomes[key][0] @@ -615,7 +616,7 @@ var FolderToolbarView = Backbone.View.extend({ this.modal.disableButton('Import'); paths = paths.split('\n'); for (var i = paths.length - 1; i >= 0; i--) { - trimmed = paths[i].trim(); + var trimmed = paths[i].trim(); if (trimmed.length!==0){ valid_paths.push(trimmed); } @@ -674,7 +675,7 @@ var FolderToolbarView = Backbone.View.extend({ importFromJstreePath: function ( that, options ){ var all_nodes = $( '#jstree_browser' ).jstree().get_selected( true ); // remove the disabled elements that could have been trigerred with the 'select all' - selected_nodes = _.filter(all_nodes, function(node){ return node.state.disabled == false; }) + var selected_nodes = _.filter(all_nodes, function(node){ return node.state.disabled == false; }) var preserve_dirs = this.modal.$el.find( '.preserve-checkbox' ).is( ':checked' ); var link_data = this.modal.$el.find( '.link-checkbox' ).is( ':checked' ); var file_type = this.select_extension.value(); @@ -752,7 +753,7 @@ var FolderToolbarView = Backbone.View.extend({ } }); for ( var i = history_item_ids.length - 1; i >= 0; i-- ) { - history_item_id = history_item_ids[i]; + var history_item_id = history_item_ids[i]; var folder_item = new mod_library_model.Item(); folder_item.url = Galaxy.root + 'api/folders/' + this.options.id + '/contents'; if (history_item_types[i] === 'collection') { diff --git a/client/galaxy/scripts/mvc/library/library-library-view.js b/client/galaxy/scripts/mvc/library/library-library-view.js index 42650e04cab..9d6d6b82d19 100644 --- a/client/galaxy/scripts/mvc/library/library-library-view.js +++ b/client/galaxy/scripts/mvc/library/library-library-view.js @@ -212,7 +212,7 @@ var LibraryView = Backbone.View.extend({ }, _extractIds: function(roles_list){ - ids_list = []; + var ids_list = []; for (var i = roles_list.length - 1; i >= 0; i--) { ids_list.push(roles_list[i].id); }; diff --git a/client/galaxy/scripts/mvc/library/library-libraryrow-view.js b/client/galaxy/scripts/mvc/library/library-libraryrow-view.js index e39cc2b0da5..757222170cb 100644 --- a/client/galaxy/scripts/mvc/library/library-libraryrow-view.js +++ b/client/galaxy/scripts/mvc/library/library-libraryrow-view.js @@ -64,7 +64,7 @@ var LibraryRowView = Backbone.View.extend({ * the filling of the row template of given library. */ prepareButtons: function(library){ - vis_config = this.element_visibility_config; + var vis_config = this.element_visibility_config; if (this.edit_mode === false){ vis_config.save_library_btn = false; diff --git a/client/galaxy/scripts/mvc/library/library-model.js b/client/galaxy/scripts/mvc/library/library-model.js index 91f00cd92ba..be62bef4a13 100644 --- a/client/galaxy/scripts/mvc/library/library-model.js +++ b/client/galaxy/scripts/mvc/library/library-model.js @@ -41,7 +41,7 @@ define([], function() { if (search_term == "") return this; var lowercase_term = search_term.toLowerCase(); return this.filter(function(data) { - lowercase_name = data.get("name").toLowerCase(); + var lowercase_name = data.get("name").toLowerCase(); return lowercase_name.indexOf(lowercase_term) !== -1; }); }, diff --git a/client/galaxy/scripts/mvc/tool/tool-form-composite.js b/client/galaxy/scripts/mvc/tool/tool-form-composite.js index 608109432c1..01144b1daa3 100644 --- a/client/galaxy/scripts/mvc/tool/tool-form-composite.js +++ b/client/galaxy/scripts/mvc/tool/tool-form-composite.js @@ -112,7 +112,7 @@ define([ 'utils/utils', 'utils/deferred', 'mvc/ui/ui-misc', 'mvc/form/form-view' var wp_count = 0; this.wp_inputs = {}; function _handleWorkflowParameter( value, callback ) { - var re = /\$\{(.+?)\}/g; + var re = /\$\{(.+?)\}/g, match; while ( match = re.exec( String( value ) ) ) { var wp_name = match[ 1 ]; callback( self.wp_inputs[ wp_name ] = self.wp_inputs[ wp_name ] || { @@ -336,7 +336,7 @@ define([ 'utils/utils', 'utils/deferred', 'mvc/ui/ui-misc', 'mvc/form/form-view' new_value = { values: [] }; _.each( input.step_linked, function( source_step ) { if ( self._isDataStep( source_step ) ) { - value = self.forms[ source_step.index ].data.create().input; + var value = self.forms[ source_step.index ].data.create().input; value && _.each( value.values, function( v ) { new_value.values.push( v ) } ); } }); @@ -345,7 +345,7 @@ define([ 'utils/utils', 'utils/deferred', 'mvc/ui/ui-misc', 'mvc/form/form-view' } } else if ( input.wp_linked ) { new_value = input.value; - var re = /\$\{(.+?)\}/g; + var re = /\$\{(.+?)\}/g, match; while ( match = re.exec( input.value ) ) { var wp_field = self.wp_form.field_list[ self.wp_form.data.match( match[ 1 ] ) ]; var wp_value = wp_field && wp_field.value(); @@ -514,7 +514,7 @@ define([ 'utils/utils', 'utils/deferred', 'mvc/ui/ui-misc', 'mvc/form/form-view' /** Is data input module/step */ _isDataStep: function( steps ) { - lst = $.isArray( steps ) ? steps : [ steps ] ; + var lst = $.isArray( steps ) ? steps : [ steps ] ; for ( var i = 0; i < lst.length; i++ ) { var step = lst[ i ]; if ( !step || !step.step_type || !step.step_type.startsWith( 'data' ) ) { diff --git a/client/galaxy/scripts/mvc/toolshed/categories-view.js b/client/galaxy/scripts/mvc/toolshed/categories-view.js index b914e55fe3a..b9226f8d5c4 100644 --- a/client/galaxy/scripts/mvc/toolshed/categories-view.js +++ b/client/galaxy/scripts/mvc/toolshed/categories-view.js @@ -39,7 +39,7 @@ define(['mvc/toolshed/toolshed-model', 'mvc/toolshed/util'], function(toolshed_m var params = {term: request.term, tool_shed_url: shed_url}; $.post(base_url, params, function(data) { console.log(data); - result_list = toolshed_util.shedParser(data); + var result_list = toolshed_util.shedParser(data); response(result_list); }); }, diff --git a/client/galaxy/scripts/mvc/toolshed/repo-status-view.js b/client/galaxy/scripts/mvc/toolshed/repo-status-view.js index 183de64a53d..0d8e381e87c 100644 --- a/client/galaxy/scripts/mvc/toolshed/repo-status-view.js +++ b/client/galaxy/scripts/mvc/toolshed/repo-status-view.js @@ -16,7 +16,7 @@ define(['mvc/toolshed/toolshed-model', 'mvc/toolshed/util'], function(toolshed_m var all_done = true; _.some(self.model.models, function(repository) { repo_id = repository.get('id'); - repo_status = repository.get('status').toLowerCase(); + var repo_status = repository.get('status').toLowerCase(); if (terminal_states.indexOf(repo_status) === -1) { all_done = false; return true; diff --git a/client/galaxy/scripts/mvc/toolshed/repositories-view.js b/client/galaxy/scripts/mvc/toolshed/repositories-view.js index 795adfa0471..acf7f2fa7f3 100644 --- a/client/galaxy/scripts/mvc/toolshed/repositories-view.js +++ b/client/galaxy/scripts/mvc/toolshed/repositories-view.js @@ -32,7 +32,7 @@ define(['mvc/toolshed/toolshed-model', 'mvc/toolshed/util'], function(toolshed_m var base_url = Galaxy.root + 'api/tool_shed/search'; var params = {term: request.term, tool_shed_url: shed_url}; $.post(base_url, params, function(data) { - result_list = toolshed_util.shedParser(data); + var result_list = toolshed_util.shedParser(data); response(result_list); }); }, diff --git a/client/galaxy/scripts/mvc/toolshed/repository-queue-view.js b/client/galaxy/scripts/mvc/toolshed/repository-queue-view.js index 24193a63d2b..509f0a869bb 100644 --- a/client/galaxy/scripts/mvc/toolshed/repository-queue-view.js +++ b/client/galaxy/scripts/mvc/toolshed/repository-queue-view.js @@ -63,7 +63,7 @@ define(['mvc/toolshed/toolshed-model', 'mvc/toolshed/util'], function(toolshed_m if (queue_key === undefined) { queue_key = toolshed_util.queueKey(repository_metadata); } - repository_queue = JSON.parse(localStorage.repositories); + var repository_queue = JSON.parse(localStorage.repositories); if (repository_queue.hasOwnProperty(queue_key)) { delete repository_queue[queue_key]; localStorage.repositories = JSON.stringify(repository_queue); @@ -82,7 +82,7 @@ define(['mvc/toolshed/toolshed-model', 'mvc/toolshed/util'], function(toolshed_m }, loadFromQueue: function(queue_key) { - repository_queue = JSON.parse(localStorage.repositories); + var repository_queue = JSON.parse(localStorage.repositories); if (repository_queue.hasOwnProperty(queue_key)) { return repository_queue[queue_key]; } diff --git a/client/galaxy/scripts/mvc/toolshed/repository-view.js b/client/galaxy/scripts/mvc/toolshed/repository-view.js index 27403c54d02..9192988f16c 100644 --- a/client/galaxy/scripts/mvc/toolshed/repository-view.js +++ b/client/galaxy/scripts/mvc/toolshed/repository-view.js @@ -19,7 +19,7 @@ define(['mvc/toolshed/toolshed-model', this.options = _.defaults(this.options || {}, this.defaults); this.model = new toolshed_model.RepositoryCollection(); this.listenTo(this.model, 'sync', this.render); - shed = params.tool_shed.replace(/\//g, '%2f'); + var shed = params.tool_shed.replace(/\//g, '%2f'); this.model.url += '?tool_shed_url=' + shed + '&repository_id=' + params.repository_id; this.model.tool_shed_url = params.tool_shed.replace(/%2f/g, '/'); this.model.tool_shed = shed; @@ -81,7 +81,7 @@ define(['mvc/toolshed/toolshed-model', params.tool_panel_section = JSON.stringify(that.panelSelect(params)); params.shed_tool_conf = $("select[name='shed_tool_conf']").find('option:selected').val() params.changeset = $('#changeset').find("option:selected").val(); - url = $('#repository_installation').attr('action'); + var url = $('#repository_installation').attr('action'); that.prepareInstall(params, url); }); $('#queue_install').on('click', function(ev) { @@ -107,8 +107,8 @@ define(['mvc/toolshed/toolshed-model', that.checkInstalled(repository_metadata); }); $('.tool_panel_section_picker').on('change', function() { - new_value = $(this).find('option:selected').val(); - default_tps = $('#tool_panel_section_select').find('option:selected').val(); + var new_value = $(this).find('option:selected').val(); + var default_tps = $('#tool_panel_section_select').find('option:selected').val(); if (new_value == default_tps) { $(this).attr('default', 'active'); } @@ -191,8 +191,8 @@ define(['mvc/toolshed/toolshed-model', params.new_tool_panel_section = $("#new_tool_panel_section").val(); } $('.tool_panel_section_picker').each(function() { - element_name = $(this).attr('name'); - tool_guid = $(this).attr('data-toolguid'); + var element_name = $(this).attr('name'); + var tool_guid = $(this).attr('data-toolguid'); if (element_name === 'tool_panel_section_id') { tool_panel_section[tool_guid] = { tool_panel_section: $(this).find("option:selected").val(), action: 'append' } } @@ -244,13 +244,13 @@ define(['mvc/toolshed/toolshed-model', prepareInstall: function(params, api_url) { var that = this; $.post(api_url, params, function(data) { - iri_parameters = JSON.parse(data); + var iri_parameters = JSON.parse(data); that.doInstall(iri_parameters); }); }, doInstall: function(params) { - controller_url = Galaxy.root + 'admin_toolshed/manage_repositories'; + var controller_url = Galaxy.root + 'admin_toolshed/manage_repositories'; var repositories = params.repositories; var new_route = 'status/r/' + repositories.join('|'); $.post(controller_url, params, function(data) { diff --git a/client/galaxy/scripts/mvc/toolshed/util.js b/client/galaxy/scripts/mvc/toolshed/util.js index ed052ecb8e8..143309f6df4 100644 --- a/client/galaxy/scripts/mvc/toolshed/util.js +++ b/client/galaxy/scripts/mvc/toolshed/util.js @@ -4,7 +4,7 @@ define([], function() { var shed_url = this.shed_url; var base_url = Galaxy.root + 'api/tool_shed/search'; $.get(base_url, {term: request.term, tool_shed_url: shed_url}, function(data) { - result_list = that.shedParser(data); + var result_list = that.shedParser(data); response(result_list); }); @@ -16,7 +16,7 @@ define([], function() { $.each(hits, function(hit) { var record = hits[hit]; var label = record.repository.name + ' by ' + record.repository.repo_owner_username + ': ' + record.repository.description; - result = {value: record.repository.id, label: label}; + var result = {value: record.repository.id, label: label}; results.push(result); }); return results; @@ -37,8 +37,8 @@ define([], function() { var queueLength = function() { if (localStorage.hasOwnProperty('repositories')) { - repo_queue = JSON.parse(localStorage.repositories); - queue_length = Object.keys(repo_queue).length; + var repo_queue = JSON.parse(localStorage.repositories); + var queue_length = Object.keys(repo_queue).length; return queue_length; } else { diff --git a/client/galaxy/scripts/mvc/toolshed/workflows-view.js b/client/galaxy/scripts/mvc/toolshed/workflows-view.js index bb47d9ba65a..45ea8faacb9 100644 --- a/client/galaxy/scripts/mvc/toolshed/workflows-view.js +++ b/client/galaxy/scripts/mvc/toolshed/workflows-view.js @@ -24,7 +24,7 @@ define(['mvc/toolshed/toolshed-model', 'mvc/toolshed/util'], function(toolshed_m }, bindEvents: function() { - var that = this; + var that = this, repository_id; $('.show_wf_repo').on('click', function() { var tool_ids = $(this).attr('data-toolids'); var toolshed = $(this).attr('data-shed'); diff --git a/client/galaxy/scripts/mvc/ui/ui-drilldown.js b/client/galaxy/scripts/mvc/ui/ui-drilldown.js index 71627f2e651..88f56957b5b 100644 --- a/client/galaxy/scripts/mvc/ui/ui-drilldown.js +++ b/client/galaxy/scripts/mvc/ui/ui-drilldown.js @@ -53,7 +53,7 @@ var View = Options.BaseIcons.extend({ // recursive function which iterates through options function iterate ( $tmpl, options, header ) { header = header || []; - for ( i in options ) { + for (var i in options ) { var level = options[ i ]; var has_options = level.options && level.options.length > 0; var new_header = header.slice( 0 ); diff --git a/client/galaxy/scripts/mvc/ui/ui-popover.js b/client/galaxy/scripts/mvc/ui/ui-popover.js index 3227a1c4408..7d8f013b419 100644 --- a/client/galaxy/scripts/mvc/ui/ui-popover.js +++ b/client/galaxy/scripts/mvc/ui/ui-popover.js @@ -116,7 +116,8 @@ var View = Backbone.View.extend({ var container_position = $container.position(); // get position - var top = left = 0; + var top, left; + top = left = 0; if ([ 'top', 'bottom' ].indexOf( placement ) != -1) { left = container_position.left - width + ( container_width + width ) / 2; switch ( placement ) { diff --git a/client/galaxy/scripts/mvc/ui/ui-select-default.js b/client/galaxy/scripts/mvc/ui/ui-select-default.js index 4d0671344a5..20caebcd3d5 100644 --- a/client/galaxy/scripts/mvc/ui/ui-select-default.js +++ b/client/galaxy/scripts/mvc/ui/ui-select-default.js @@ -315,7 +315,7 @@ var View = Backbone.View.extend({ } if ( this.model.get( 'searchable' ) ) { if ( $.isArray( new_value ) ) { - val = []; + var val = []; _.each( new_value, function( v ) { var d = _.findWhere( self.data2, { id: v } ); d && val.push( d ); diff --git a/client/galaxy/scripts/mvc/upload/upload-ftp.js b/client/galaxy/scripts/mvc/upload/upload-ftp.js index 4e36750f55f..b3547f82eee 100644 --- a/client/galaxy/scripts/mvc/upload/upload-ftp.js +++ b/client/galaxy/scripts/mvc/upload/upload-ftp.js @@ -27,7 +27,7 @@ define( [ 'utils/utils' ], function( Utils ) { if ( ftp_files && ftp_files.length > 0 ) { this.$( '.upload-ftp-content' ).html( $( this._templateTable() ) ); var size = 0; - for ( index in ftp_files ) { + for (var index in ftp_files ) { this.rows.push( this._add( ftp_files[ index ] ) ); size += ftp_files[ index ].size; } @@ -39,7 +39,7 @@ define( [ 'utils/utils' ], function( Utils ) { this.$select_all = this.$( '.upload-selectall' ).addClass( this.options.class_add ); this.$select_all.on( 'click', function() { var add = self.$select_all.hasClass( self.options.class_add ); - for ( index in ftp_files ) { + for (var index in ftp_files ) { var ftp_file = ftp_files[ index ]; var model_index = self._find( ftp_file ); if( !model_index && add || model_index && !add ) { diff --git a/client/galaxy/scripts/mvc/upload/upload-view.js b/client/galaxy/scripts/mvc/upload/upload-view.js index e6a8a2dc839..38e6b8329d1 100644 --- a/client/galaxy/scripts/mvc/upload/upload-view.js +++ b/client/galaxy/scripts/mvc/upload/upload-view.js @@ -48,7 +48,7 @@ function( Utils, Modal, Tabs, UploadButton, UploadViewDefault, UploadViewComposi Utils.get({ url : Galaxy.root + 'api/datatypes?extension_only=False', success : function( datatypes ) { - for ( key in datatypes ) { + for (var key in datatypes ) { self.list_extensions.push({ id : datatypes[ key ].extension, text : datatypes[ key ].extension, @@ -72,7 +72,7 @@ function( Utils, Modal, Tabs, UploadButton, UploadViewDefault, UploadViewComposi Utils.get({ url : Galaxy.root + 'api/genomes', success : function( genomes ) { - for ( key in genomes ) { + for (var key in genomes ) { self.list_genomes.push({ id : genomes[ key ][ 1 ], text : genomes[ key ][ 0 ] diff --git a/client/galaxy/scripts/mvc/workflow/workflow-forms.js b/client/galaxy/scripts/mvc/workflow/workflow-forms.js index 0eb059caa73..1acc2a910f2 100644 --- a/client/galaxy/scripts/mvc/workflow/workflow-forms.js +++ b/client/galaxy/scripts/mvc/workflow/workflow-forms.js @@ -110,7 +110,7 @@ define( [ 'utils/utils', 'mvc/form/form-view', 'mvc/tool/tool-form-base' ], func var self = this; var extensions = []; var input_terminal_names = []; - for ( key in datatypes ) { + for (var key in datatypes ) { extensions.push( { 0 : datatypes[ key ], 1 : datatypes[ key ] } ); } for ( key in this.node.input_terminals ){ @@ -131,6 +131,7 @@ define( [ 'utils/utils', 'mvc/form/form-view', 'mvc/tool/tool-form-base' ], func 0 : 'Leave unchanged', 1 : '__empty__' }); + var output; var input_config = { title : 'Configure Output: \'' + output_id + '\'', type : 'section', diff --git a/client/galaxy/scripts/mvc/workflow/workflow-manager.js b/client/galaxy/scripts/mvc/workflow/workflow-manager.js index 299c792a200..2b098f14463 100644 --- a/client/galaxy/scripts/mvc/workflow/workflow-manager.js +++ b/client/galaxy/scripts/mvc/workflow/workflow-manager.js @@ -199,7 +199,7 @@ function( Connector, Toastr ) { }, from_simple : function ( data, initialImport_ ) { var initialImport = (initialImport_ === undefined) ? true : initialImport_; - wf = this; + var wf = this; var offset = 0; if( initialImport ) { wf.name = data.name; @@ -264,7 +264,7 @@ function( Connector, Toastr ) { $.each(node.output_terminals, function(ot_id, ot){ if(node.post_job_actions['HideDatasetAction'+ot.name] === undefined){ node.addWorkflowOutput(ot.name); - callout = $(node.element).find('.callout.'+ot.name); + var callout = $(node.element).find('.callout.'+ot.name); callout.find('img').attr('src', Galaxy.root + 'static/images/fugue/asterisk-small.png'); wf.has_changes = true; } @@ -338,10 +338,10 @@ function( Connector, Toastr ) { }); }); // Assemble order, tracking levels - node_ids_by_level = []; + var node_ids_by_level = []; while ( true ) { // Everything without a predecessor - level_parents = []; + var level_parents = []; for ( var pred_k in n_pred ) { if ( n_pred[ pred_k ] == 0 ) { level_parents.push( pred_k ); @@ -395,7 +395,7 @@ function( Connector, Toastr ) { ymin = Infinity, ymax = -Infinity, p; $.each( this.nodes, function( id, node ) { - e = $(node.element); + var e = $(node.element); p = e.position(); xmin = Math.min( xmin, p.left ); xmax = Math.max( xmax, p.left + e.width() ); @@ -411,7 +411,7 @@ function( Connector, Toastr ) { } function fix_delta( x, n ) { if ( x < n|| x > 3*n ) { - new_pos = ( Math.ceil( ( ( x % n ) ) / n ) + 1 ) * n; + var new_pos = ( Math.ceil( ( ( x % n ) ) / n ) + 1 ) * n; return ( - ( x - new_pos ) ); } return 0; diff --git a/client/galaxy/scripts/mvc/workflow/workflow-terminals.js b/client/galaxy/scripts/mvc/workflow/workflow-terminals.js index 159068cacb5..15cebc012f4 100644 --- a/client/galaxy/scripts/mvc/workflow/workflow-terminals.js +++ b/client/galaxy/scripts/mvc/workflow/workflow-terminals.js @@ -4,6 +4,9 @@ define(['mvc/workflow/workflow-globals'], function( Globals ) { this.isCollection = true; this.rank = collectionType.split(":").length; } + + var ANY_COLLECTION_TYPE_DESCRIPTION, NULL_COLLECTION_TYPE_DESCRIPTION; + $.extend( CollectionTypeDescription.prototype, { append: function( otherCollectionTypeDescription ) { if( otherCollectionTypeDescription === NULL_COLLECTION_TYPE_DESCRIPTION ) { diff --git a/client/galaxy/scripts/mvc/workflow/workflow-view.js b/client/galaxy/scripts/mvc/workflow/workflow-view.js index 568fd357d2f..fa85f7841c5 100644 --- a/client/galaxy/scripts/mvc/workflow/workflow-view.js +++ b/client/galaxy/scripts/mvc/workflow/workflow-view.js @@ -64,7 +64,7 @@ define([ var close_editor = function() { self.workflow.check_changes_in_active_form(); if ( workflow && self.workflow.has_changes ) { - do_close = function() { + var do_close = function() { window.onbeforeunload = undefined; window.document.location = self.urls.workflow_index; }; @@ -227,7 +227,7 @@ define([ self.scroll_to_nodes(); self.canvas_manager.draw_overview(); // Determine if any parameters were 'upgraded' and provide message - upgrade_message = ""; + var upgrade_message = ""; _.each( data.steps, function( step, step_id ) { var details = ""; if ( step.errors ) { @@ -335,7 +335,7 @@ define([ } // On load, set the size to the pref stored in local storage if it exists - overview_size = $.jStorage.get("overview-size"); + var overview_size = $.jStorage.get("overview-size"); if (overview_size !== undefined) { $("#overview-border").css( { width: overview_size, @@ -468,7 +468,7 @@ define([ success: function( data ) { self.workflow.from_simple( data, false ); // Determine if any parameters were 'upgraded' and provide message - upgrade_message = ""; + var upgrade_message = ""; $.each( data.upgrade_messages, function( k, v ) { upgrade_message += ( "
  • Step " + ( parseInt(k, 10) + 1 ) + ": " + self.workflow.nodes[k].name + "