[Web UI] Changes in labels implementation

src/webui/webapplication.cpp

@@ -669,20 +669,20 @@ void WebApplication::action_command_recheck()
 void WebApplication::action_command_setLabel()
 {
     CHECK_URI(0);
-    CHECK_PARAMETERS("hash" << "label_obj");
+    CHECK_PARAMETERS("hashes" << "label");
 
-    QString hash = request().posts["hash"];
+    QStringList hashes = request().posts["hashes"].split("|");
-    QString label_obj = request().posts["label_obj"];
+    QString label = request().posts["label"].trimmed();
+    if (!Utils::Fs::isValidFileSystemName(label)) {
+        status(400, "Labels must not contain special characters");
+        return;
+    }
 
-    const QVariantMap m = json::fromJson(label_obj).toMap();
+    foreach (const QString &hash, hashes) {
-    if( m.contains("value") ) {
-        QString label = m["value"].toString();
-        if (!hash.isEmpty()) {
         BitTorrent::TorrentHandle *const torrent = BitTorrent::Session::instance()->findTorrent(hash);
         if (torrent)
             torrent->setLabel(label);
     }
-    }
 }
 
 bool WebApplication::isPublicScope()

src/webui/www/private/index.html

@@ -106,9 +106,8 @@
         <li><a href="#ForceStart"><img src="theme/media-seek-forward" alt="QBT_TR(Force Resume)QBT_TR"/> QBT_TR(Force Resume)QBT_TR</a></li>
         <li class="separator"><a href="#Delete"><img src="theme/list-remove" alt="QBT_TR(Delete)QBT_TR"/> QBT_TR(Delete)QBT_TR</a></li>
         <li class="separator">
-            <a href="#Label">QBT_TR(Label >)QBT_TR</a>
+            <a href="#Label" class="arrow-right"><img src="theme/view-categories" alt="QBT_TR(Label)QBT_TR"/> QBT_TR(Label)QBT_TR</a>
-            <ul id="contextLabelList">
+            <ul id="contextLabelList"></ul>
-            </ul>
         </li>
         <li id="queueingMenuItems" class="separator">
             <a href="#priority" class="arrow-right"><span style="display: inline-block; width:16px"></span> QBT_TR(Priority)QBT_TR</a>

src/webui/www/public/filters.html

@@ -1,5 +1,5 @@
-                QBT_TR(Torrents)QBT_TR
+QBT_TR(Torrents)QBT_TR
-                <ul class="filterList">
+<ul class="filterList">
     <li id="all_filter"><a href="#" onclick="setFilter('all');return false;"><img src="images/skin/filterall.png"/>QBT_TR(All)QBT_TR</a></li>
     <li id="downloading_filter"><a href="#" onclick="setFilter('downloading');return false;"><img src="images/skin/downloading.png"/>QBT_TR(Downloading)QBT_TR</a></li>
     <li id="seeding_filter"><a href="#" onclick="setFilter('seeding');return false;"><img src="images/skin/uploading.png"/>QBT_TR(Seeding)QBT_TR</a></li>
@@ -8,8 +8,8 @@
     <li id="paused_filter"><a href="#" onclick="setFilter('paused');return false;"><img src="images/skin/paused.png"/>QBT_TR(Paused)QBT_TR</a></li>
     <li id="active_filter"><a href="#" onclick="setFilter('active');return false;"><img src="images/skin/filteractive.png"/>QBT_TR(Active)QBT_TR</a></li>
     <li id="inactive_filter"><a href="#" onclick="setFilter('inactive');return false;"><img src="images/skin/filterinactive.png"/>QBT_TR(Inactive)QBT_TR</a></li>
-                </ul>
+</ul>
-                <br/>
+<br/>
-                QBT_TR(Labels)QBT_TR
+QBT_TR(Labels)QBT_TR
-                <ul id="filterLabelList" style="padding-left: 0px;">
+<ul id="filterLabelList" class="filterList">
-                </ul>
+</ul>

src/webui/www/public/newlabel.html

@@ -7,58 +7,41 @@
     <script type="text/javascript" src="scripts/mootools-1.2-core-yc.js" charset="utf-8"></script>
     <script type="text/javascript" src="scripts/mootools-1.2-more.js" charset="utf-8"></script>
     <script type="text/javascript">
-        window.addEvent('domready', function(){
+        window.addEvent('domready', function() {
             $('newLabel').focus();
-            $('newLabelButton').addEvent('click', function(e){
+            $('newLabelButton').addEvent('click', function(e) {
-                // check field
-                var label_str = $('newLabel').value;
-                if( label_str == null || label_str == "" ) {
-                    return false;
-                }
-                if( label_str.match("[\\\\/:?\"*<>|]") !== null ) {
-                    $('alert_div').style.display = "block";
-                    return false;
-                }
-
                 new Event(e).stop();
-                var hashes = new URI().getData('hashes');
+                // check field
-                var hashesList = hashes.split(',');
+                var labelName = $('newLabel').value.trim();
-                var label_json = JSON.stringify( { value : label_str } );
+                if (labelName == null || labelName == "")
-
+                    return false;
-                if (hashesList.length) {
+                if (labelName.match("[\\\\/:?\"*<>|]") !== null) {
-                    var counter = hashesList.length;
+                    alert("QBT_TR(Invalid label name:\nPlease do not use any special characters in the label name.)QBT_TR");
-                    hashesList.each(function(hash, index) {
+                    return false;
+                }
+                var hashesList = new URI().getData('hashes');
                 new Request({
                     url: 'command/setLabel',
                     method: 'post',
                     data: {
-                                hash: hash,
+                        hashes: hashesList,
-                                label_obj: label_json
+                        label: labelName
                     },
                     onComplete: function() {
-                                if( --counter === 0 ) {
+                        window.parent.closeWindows();
-                                    window.parent.document.getElementById('newLabelPage').parentNode.removeChild(window.parent.document.getElementById('newLabelPage'));
-                                }
                     }
                 }).send();
             });
-                }
-            });
         });
     </script>
 </head>
 <body>
-<center>
+    <div style="padding: 10px 10px 0px 10px;">
-    <h1 class="vcenter">QBT_TR(Label)QBT_TR:
+        <p style="font-weight: bold;">QBT_TR(Label)QBT_TR:</p>
-    <input type="text" name="label" id="newLabel" value="" maxlength="100" />
+        <input type="text" id="newLabel" value="" maxlength="100" style="width: 220px;"/>
-    </h1>
+        <div style="text-align: center;">
-    <div id="alert_div" style="border: 1px #ff0000 solid; display: none; color: #ff0000; background-color: #ffdee8; width: 95%;">
-        <b>QBT_TR(Invalid label name)QBT_TR</b>:
-        <br/>
-        QBT_TR(Please don't use any special characters in the label name.)QBT_TR
-        <br/>
-    </div>
             <input type="button" value="QBT_TR(Add)QBT_TR" id="newLabelButton"/>
-</center>
+        </div>
+    </div>
 </body>
 </html>

src/webui/www/public/scripts/client.js

@@ -95,10 +95,10 @@ window.addEvent('load', function () {
         resizeLimit : [100, 300]
     });
 
-    setLabelFilter = function( hash ) {
+    setLabelFilter = function(hash) {
         selected_label = hash;
         localStorage.setItem('selected_label', selected_label);
-        updateLabelList();
+        highlightSelectedLabel();
         if (typeof myTable.table != 'undefined')
             updateMainData();
     };
@@ -161,35 +161,33 @@ window.addEvent('load', function () {
     var syncMainDataLastResponseId = 0;
     var serverState = {};
 
-    var removeTorrentFromLabelList = function( hash )
+    var removeTorrentFromLabelList = function(hash) {
-    {
+        if (hash == null || hash == "")
-        if( hash == null || hash == "" ) return false;
+            return false;
-
         var removed = false;
-        Object.each( label_list, function( label ) {
+        Object.each(label_list, function(label) {
-            if( Object.contains( label.torrents, hash ) ) {
+            if (Object.contains(label.torrents, hash)) {
                 removed = true;
-                label.torrents.splice( label.torrents.indexOf( hash ), 1 );
+                label.torrents.splice(label.torrents.indexOf(hash), 1);
             }
         });
         return removed;
     };
 
-    var addTorrentToLabelList = function( torrent ) {
+    var addTorrentToLabelList = function(torrent) {
         var label = torrent['label'];
-        if( label == null || label.length === 0 ) {
+        if (label == null)
-            removeTorrentFromLabelList( torrent['hash'] );
             return false;
+        if (label.length === 0) { // Empty label
+            removeTorrentFromLabelList(torrent['hash']);
+            return true;
         }
-
+        var labelHash = genHash(label);
-        var labelHash = genHash( label );
+        if (label_list[labelHash] == null) // This should not happen
-        if( label_list[labelHash] == null ) {
+            label_list[labelHash] = {name: label, torrents: []};
-            console.log( "addTorrentToLabelList: warning, label not found. label=", label, " label_list=", label_list );
+        if (!Object.contains(label_list[labelHash].torrents, torrent['hash'])) {
-            label_list[labelHash] = { name: label, torrents: [] };
+            removeTorrentFromLabelList(torrent['hash']);
-        }
+            label_list[labelHash].torrents = label_list[labelHash].torrents.combine([torrent['hash']]);
-        if( !Object.contains(label_list[labelHash].torrents, torrent['hash'] ) ) {
-            removeTorrentFromLabelList( torrent['hash'] );
-            label_list[labelHash].torrents = label_list[labelHash].torrents.combine( [ torrent['hash'] ] );
             return true;
         }
         return false;
@@ -198,15 +196,20 @@ window.addEvent('load', function () {
     var updateContextMenu = function () {
         var labelList = $('contextLabelList');
         labelList.empty();
-        labelList.appendChild(new Element('li', {html: '<a href="javascript:newLabelFN();">QBT_TR(New...)QBT_TR</a>'}));
+        labelList.appendChild(new Element('li', {html: '<a href="javascript:newLabelFN();"><img src="theme/list-add" alt="QBT_TR(New...)QBT_TR"/> QBT_TR(New...)QBT_TR</a>'}));
-        labelList.appendChild(new Element('li', {html: '<a href="javascript:resetLabelFN();">QBT_TR(Reset)QBT_TR</a>'}));
+        labelList.appendChild(new Element('li', {html: '<a href="javascript:updateLabelFN(0);"><img src="theme/edit-clear" alt="QBT_TR(Reset)QBT_TR"/> QBT_TR(Reset)QBT_TR</a>'}));
+
+        var sortedLabels = []
+        Object.each(label_list, function(label) {
+            sortedLabels.push(label.name);
+        });
+        sortedLabels.sort();
 
         var first = true;
-        Object.each(label_list, function (label) {
+        Object.each(sortedLabels, function(labelName) {
-            var labelHash = genHash( label.name );
+            var labelHash = genHash(labelName);
-            var el = new Element('li', {html: '<a href="javascript:updateLabelFN(\'' + labelHash + '\');">' + label.name + '</a>'});
+            var el = new Element('li', {html: '<a href="javascript:updateLabelFN(\'' + labelHash + '\');"><img src="theme/inode-directory"/> ' + labelName + '</a>'});
             if (first) {
-                el.removeClass();
                 el.addClass('separator');
                 first = false;
             }
@@ -215,42 +218,54 @@ window.addEvent('load', function () {
     };
 
     var updateLabelList = function() {
-        var labelList = $( 'filterLabelList' );
+        var labelList = $('filterLabelList');
-        if( !labelList ) {
+        if (!labelList)
             return;
-        }
         labelList.empty();
 
-        var create_link = function( hash, text, count )
+        var create_link = function(hash, text, count) {
-        {
             var html = '<a href="#" onclick="setLabelFilter(' + hash + ');return false;">' +
-                '<img src="images/oxygen/folder-documents.png"/>' +
+                '<img src="theme/inode-directory"/>' +
-                text + '(' + count + ')' + '</a>';
+                text + ' (' + count + ')' + '</a>';
-
+            return new Element('li', {id: hash, html: html});
-            return new Element( 'li', { id: hash, html: html } );
         };
 
-        var allLabels = 0;
+        var all = myTable.getRowIds().length;
-        Object.each( label_list, function( label ) {
+        var unlabelled = 0;
-            allLabels += label.torrents.length;
+        Object.each(myTable.rows, function(row) {
+            if (row['full_data'].label.length === 0)
+            unlabelled += 1;
+        });
+        labelList.appendChild(create_link(LABELS_ALL, 'QBT_TR(All)QBT_TR', all));
+        labelList.appendChild(create_link(LABELS_UNLABELLED, 'QBT_TR(Unlabeled)QBT_TR', unlabelled));
+
+        var sortedLabels = []
+        Object.each(label_list, function(label) {
+            sortedLabels.push(label.name);
+        });
+        sortedLabels.sort();
+
+        Object.each(sortedLabels, function(labelName) {
+            var labelHash = genHash(labelName);
+            var labelCount = label_list[labelHash].torrents.length;
+            labelList.appendChild(create_link(labelHash, labelName, labelCount));
         });
 
-        var unlabelled = myTable.getRowIds().length - allLabels;
+        highlightSelectedLabel();
-        labelList.appendChild( create_link( LABELS_ALL, 'QBT_TR(All Labels)QBT_TR', allLabels ) );
+    };
-        labelList.appendChild( create_link( LABELS_UNLABELLED, 'QBT_TR(Unlabeled)QBT_TR', unlabelled ) );
-
-        Object.each( label_list, function( label ) {
-            var labelHash = genHash( label.name );
-            labelList.appendChild( create_link( labelHash, label.name, label.torrents.length ) );
-        } );
 
+    var highlightSelectedLabel = function() {
+        var labelList = $('filterLabelList');
+        if (!labelList)
+            return;
         var childrens = labelList.childNodes;
         for (var i in childrens) {
-            if( childrens[i].id == selected_label ) {
+            if (childrens[i].id == selected_label)
                 childrens[i].className = "selectedFilter";
+            else
+                childrens[i].className = "";
         }
     }
-    };
 
     var syncMainDataTimer;
     var syncMainData = function () {
@@ -268,30 +283,42 @@ window.addEvent('load', function () {
             onSuccess : function (response) {
                 $('error_div').set('html', '');
                 if (response) {
+                    var update_labels = false;
                     var full_update = (response['full_update'] == true);
                     if (full_update) {
                         myTable.rows.erase();
                         label_list = {};
-                        Object.each( response['labels'], function( label ) {
-                            var labelHash = genHash( label );
-                            label_list[ labelHash ] = { name: label, torrents: [] };
-                        } );
                     }
                     if (response['rid']) {
                         syncMainDataLastResponseId = response['rid'];
                     }
+                    if (response['labels']) {
+                        response['labels'].each(function(label) {
+                            var labelHash = genHash(label);
+                            label_list[labelHash] = {name: label, torrents: []};
+                        });
+                        update_labels = true;
+                    }
+                    if (response['labels_removed']) {
+                        response['labels_removed'].each(function(label) {
+                            var labelHash = genHash(label);
+                            delete label_list[labelHash];
+                        });
+                        update_labels = true;
+                    }
                     if (response['torrents']) {
                         for (var key in response['torrents']) {
                             response['torrents'][key]['hash'] = key;
-                            myTable.updateRowData( response['torrents'][key] );
+                            myTable.updateRowData(response['torrents'][key]);
-                            addTorrentToLabelList( response['torrents'][key] );
+                            if (addTorrentToLabelList(response['torrents'][key]))
+                                update_labels = true;
                         }
-                        updateLabelList();
-                        updateContextMenu();
                     }
                     if (response['torrents_removed'])
                         response['torrents_removed'].each(function (hash) {
                             myTable.removeRow(hash);
+                            removeTorrentFromLabelList(hash);
+                            update_labels = true; // Allways to update All label
                         });
                     myTable.updateTable(full_update);
                     myTable.altRow();
@@ -301,6 +328,10 @@ window.addEvent('load', function () {
                             serverState[key] = tmp[key];
                         processServerState();
                     }
+                    if (update_labels) {
+                        updateLabelList();
+                        updateContextMenu();
+                    }
                 }
                 clearTimeout(syncMainDataTimer);
                 syncMainDataTimer = syncMainData.delay(syncMainDataTimerPeriod);

src/webui/www/public/scripts/dynamicTable.js

@@ -280,7 +280,7 @@ var dynamicTable = new Class({
                     break;
             }
 
-            if (labelName == LABELS_ALL && row['full_data'].label.length > 0)
+            if (labelName == LABELS_ALL)
                 return true;
 
             if (labelName == LABELS_UNLABELLED && row['full_data'].label.length === 0)

src/webui/www/public/scripts/mocha-init.js

@@ -314,52 +314,34 @@ initializeWindows = function() {
         if (h.length) {
             new MochaUI.Window({
                 id: 'newLabelPage',
-                title: "QBT_TR(Torrent Label)QBT_TR",
+                title: "QBT_TR(New Label)QBT_TR",
                 loadMethod: 'iframe',
-                contentURL: 'newlabel.html?hashes=' + h.join(','),
+                contentURL: 'newlabel.html?hashes=' + h.join('|'),
                 scrollbars: false,
                 resizable: false,
                 maximizable: false,
                 paddingVertical: 0,
                 paddingHorizontal: 0,
-                width: 424,
+                width: 250,
-                height: 150
+                height: 100
             });
         }
     };
 
-    resetLabelFN = function () {
+    updateLabelFN = function (labelHash) {
+        var labelName = '';
+        if (labelHash != 0)
+            var labelName = label_list[labelHash].name;
         var h = myTable.selectedIds();
-        var label_json = JSON.stringify({value: ''});
         if (h.length) {
-            h.each(function (hash, index) {
             new Request({
                 url: 'command/setLabel',
                 method: 'post',
                 data: {
-                        hash: hash,
+                    hashes: h.join("|"),
-                        label_obj: label_json
+                    label: labelName
                 }
             }).send();
-            });
-        }
-    };
-
-    updateLabelFN = function (label_hash) {
-        var label = label_list[label_hash].name;
-        var label_json = JSON.stringify({value: label});
-        var h = myTable.selectedIds();
-        if (h.length) {
-            h.each(function (hash, index) {
-                new Request({
-                    url: 'command/setLabel',
-                    method: 'post',
-                    data: {
-                        hash: hash,
-                        label_obj: label_json
-                    }
-                }).send();
-            });
         }
     };