Add no-return-assign eslint rule and fix issues

.eslintrc.js

@@ -52,6 +52,7 @@ module.exports = {
         'no-multi-spaces': ['error'],
         'no-multiple-empty-lines': ['error', { 'max': 1 }],
         'no-restricted-globals': ['error'].concat(restrictedGlobals),
+        'no-return-assign': ['error'],
         'no-return-await': ['error'],
         'no-sequences': ['error', { 'allowInParentheses': false }],
         'no-trailing-spaces': ['error'],

src/components/channelMapper/channelMapper.js

@@ -87,7 +87,8 @@ export default class channelMapper {
             html += '</div>';
             html += '</div>';
             html += `<button class="btnMap autoSize" is="paper-icon-button-light" type="button" data-id="${channel.Id}" data-providerid="${channel.ProviderChannelId}"><span class="material-icons mode_edit" aria-hidden="true"></span></button>`;
-            return html += '</div>';
+            html += '</div>';
+            return html;
         }
 
         function getEditorHtml() {
@@ -100,7 +101,8 @@ export default class channelMapper {
             html += '</div>';
             html += '</form>';
             html += '</div>';
-            return html += '</div>';
+            html += '</div>';
+            return html;
         }
 
         function initEditor(dlg, options) {

src/components/tunerPicker.js

@@ -26,7 +26,8 @@ function getEditorHtml() {
     html += '<div is="emby-itemscontainer" class="results vertical-wrap">';
     html += '</div>';
     html += '</div>';
-    return html += '</div>';
+    html += '</div>';
+    return html;
 }
 
 function getDeviceHtml(device) {
@@ -61,7 +62,8 @@ function getDeviceHtml(device) {
     html += '</div>';
     html += '</div>';
     html += '</div>';
-    return html += '</button>';
+    html += '</button>';
+    return html;
 }
 
 function getTunerName(providerId) {

src/components/viewManager/viewManager.js

@@ -103,9 +103,9 @@ function getViewEventDetail(view, {state, url, options = {}}, isRestored) {
     const searchParams = new URLSearchParams(url.substring(index + 1));
     const params = {};
 
-    searchParams.forEach((value, key) =>
+    searchParams.forEach((value, key) => {
-        params[key] = value
+        params[key] = value;
-    );
+    });
 
     return {
         detail: {

src/controllers/dashboard/apikeys.js

@@ -37,7 +37,8 @@ import { pageIdOn } from '../../utils/dashboard';
             const date = datetime.parseISO8601Date(item.DateCreated, true);
             html += datetime.toLocaleDateString(date) + ' ' + datetime.getDisplayTime(date);
             html += '</td>';
-            return html += '</tr>';
+            html += '</tr>';
+            return html;
         }).join('');
         page.querySelector('.resultBody').innerHTML = rows;
         loading.hide();

src/controllers/dashboard/dlna/profile.js

@@ -149,7 +149,8 @@ import { getParameterByName } from '../../../utils/url.ts';
             li += '<h3 class="listItemBodyText">' + escapeHtml(h.Name + ' = ' + (h.Value || '')) + '</h3>';
             li += '</div>';
             li += '<button type="button" is="paper-icon-button-light" class="btnDeleteXmlAttribute listItemButton" data-index="0"><span class="material-icons delete" aria-hidden="true"></span></button>';
-            return li += '</div>';
+            li += '</div>';
+            return li;
         }).join('') + '</div>';
         const elem = $('.xmlDocumentAttributeList', page).html(html).trigger('create');
         $('.btnDeleteXmlAttribute', elem).on('click', function () {

src/controllers/dashboard/plugins/available/index.js

@@ -129,7 +129,8 @@ function getPluginHtml(plugin, options, installedPlugins) {
     html += '</div>';
     html += '</div>';
     html += '</div>';
-    return html += '</div>';
+    html += '</div>';
+    return html;
 }
 
 function getTabs() {

src/controllers/livetvstatus.js

@@ -49,7 +49,8 @@ function getDeviceHtml(device) {
     html += '</div>';
     html += '</div>';
     html += '</div>';
-    return html += '</div>';
+    html += '</div>';
+    return html;
 }
 
 function renderDevices(page, devices) {

src/scripts/libraryBrowser.js

@@ -115,7 +115,8 @@ export function getQueryPagingHtml (options) {
         html += '</div>';
     }
 
-    return html += '</div>';
+    html += '</div>';
+    return html;
 }
 
 export function showSortMenu (options) {