Changeset 14511


Ignore:
Timestamp:
Apr 23, 2015, 7:30:40 PM (6 years ago)
Author:
mikedld
Message:

#5894: Get rid of eval() in JS code (patch by Ancient; 2nd part)

Location:
trunk/web/javascript
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/web/javascript/dialog.js

    r13439 r14511  
    2525                this._cancel_button = $('#dialog_cancel_button');
    2626                this._confirm_button = $('#dialog_confirm_button');
    27                 this._callback_function = '';
    28                 this._callback_data = null;
     27                this._callback = null;
    2928               
    3029                // Observe the buttons
     
    5958        {
    6059                var dialog = event.data.dialog;
    61                 eval(dialog._callback_function + "(dialog._callback_data)");
     60                dialog._callback();
    6261                dialog.hideDialog();
    6362        },
     
    7372         */
    7473        confirm: function(dialog_heading, dialog_message, confirm_button_label,
    75                           callback_function, callback_data, cancel_button_label)
     74                          callback, cancel_button_label)
    7675        {
    7776                if (!isMobileDevice)
     
    8281                setTextContent(this._confirm_button[0], confirm_button_label);
    8382                this._confirm_button.show();
    84                 this._callback_function = callback_function;
    85                 this._callback_data = callback_data;
     83                this._callback = callback;
    8684                $('body').addClass('dialog_showing');
    8785                this._container.show();
  • trunk/web/javascript/remote.js

    r14210 r14511  
    6363                        'Could not connect to the server. You may need to reload the page to reconnect.',
    6464                        'Details',
    65                         'alert(remote._error);',
    66                         null,
     65                        function() {
     66                                alert(remote._error);
     67                        },
    6768                        'Dismiss');
    6869                remote._controller.togglePeriodicSessionRefresh(false);
  • trunk/web/javascript/transmission.js

    r14257 r14511  
    10351035                            header = 'Remove ' + torrent.getName() + '?',
    10361036                            message = 'Once removed, continuing the transfer will require the torrent file. Are you sure you want to remove it?';
    1037                         dialog.confirm(header, message, 'Remove', 'transmission.removeTorrents', torrents);
     1037                        dialog.confirm(header, message, 'Remove', function() {
     1038                                transmission.removeTorrents(torrents);
     1039                        });
    10381040                }
    10391041                else
     
    10411043                        var header = 'Remove ' + torrents.length + ' transfers?',
    10421044                            message = 'Once removed, continuing the transfers will require the torrent files. Are you sure you want to remove them?';
    1043                         dialog.confirm(header, message, 'Remove', 'transmission.removeTorrents', torrents);
     1045                        dialog.confirm(header, message, 'Remove', function() {
     1046                                transmission.removeTorrents(torrents);
     1047                        });
    10441048                }
    10451049        },
     
    10521056                            header = 'Remove ' + torrent.getName() + ' and delete data?',
    10531057                            message = 'All data downloaded for this torrent will be deleted. Are you sure you want to remove it?';
    1054                         dialog.confirm(header, message, 'Remove', 'transmission.removeTorrentsAndData', torrents);
     1058                        dialog.confirm(header, message, 'Remove', function() {
     1059                                transmission.removeTorrentsAndData(torrents);
     1060                        });
    10551061                }
    10561062                else
     
    10581064                        var header = 'Remove ' + torrents.length + ' transfers and delete data?',
    10591065                            message = 'All data downloaded for these torrents will be deleted. Are you sure you want to remove them?';
    1060                         dialog.confirm(header, message, 'Remove', 'transmission.removeTorrentsAndData', torrents);
     1066                        dialog.confirm(header, message, 'Remove', function() {
     1067                                transmission.removeTorrentsAndData(torrents);
     1068                        });
    10611069                }
    10621070        },
Note: See TracChangeset for help on using the changeset viewer.