mirrors/tt-rss
1
0
Fork 0
mirror of https://git.tt-rss.org/fox/tt-rss.git synced 2025-10-23 08:51:00 +02:00
Code Issues Projects Releases Wiki Activity

Fix some more issues related to ESLint 'eqeqeq' changes.

Browse Source
This commit is contained in:
supahgreg 2025-10-14 03:29:37 +00:00
parent 602d79c87e
commit 362e2dc28c
No known key found for this signature in database
4 changed files with 35 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
js/Article.js
View File

@ -445,17 +445,13 @@ const Article = {
row.removeClassName("Unread");
row.addClassName("active");
PluginHost.run(PluginHost.HOOK_ARTICLE_SET_ACTIVE, row.getAttribute("data-article-id"));
PluginHost.run(PluginHost.HOOK_ARTICLE_SET_ACTIVE, parseInt(row.getAttribute('data-article-id')));
}
}
},
getActive: function () {
const row = document.querySelector("#headlines-frame > div[id*=RROW][class*=active]");
if (row)
return row.getAttribute("data-article-id");
else
return 0;
return row ? parseInt(row.getAttribute('data-article-id')) : 0;
},
scrollByPages: function (page_offset) {
App.Scrollable.scrollByPages(App.byId("content-insert"), page_offset);

4
js/FeedTree.js
View File

@ -204,7 +204,9 @@ define(["dojo/_base/declare", "dojo/dom-construct", "dojo/_base/array", "dojo/co
}
},
getTooltip: function (item) {
return [item.updated, item.error].filter((x) => x && x !== "").join(" - ");
// TODO: item.error is `[""]` for feeds. Need to look into what's happening on the frontend to cause that-- the backend sends a string.
// For now, just adding a check for `[""]`.
return [item.updated, item.error].filter((x) => x && x !== '' && !(Array.isArray(x) && x.length === 1 && x[0] === '')).join(' - ');
},
getIconClass: function (item, opened) {
return (!item || this.model.mayHaveChildren(item)) ? (opened ? "dijitFolderOpened" : "dijitFolderClosed") : "feed-icon";

4
js/Feeds.js
View File

@ -334,7 +334,9 @@ const Feeds = {
setActive: function(id, is_cat) {
console.log('setActive', id, is_cat);
id = parseInt(id);
// id might be a tag string, so check if we have something int-ish
if (Number.isInteger(Number(id)))
id = parseInt(id);
window.requestIdleCallback(() => {
App.Hash.set({

55
js/Headlines.js
View File

@ -59,7 +59,7 @@ const Headlines = {
if (m.type === 'attributes' && ['class', 'data-score'].indexOf(m.attributeName) !== -1) {
const row = m.target;
const id = row.getAttribute("data-article-id");
const id = parseInt(row.getAttribute('data-article-id'));
if (Headlines.headlines[id]) {
const hl = Headlines.headlines[id];
@ -326,7 +326,7 @@ const Headlines = {
const row = rows[i];
if (this.isChildVisible(row)) {
return row.getAttribute("data-article-id");
return parseInt(row.getAttribute('data-article-id'));
}
}
},
@ -350,7 +350,8 @@ const Headlines = {
const last_row = hsp.previousSibling;
// invoke lazy load if last article in buffer is nearly visible OR is active
if (Article.getActive() === last_row.getAttribute("data-article-id") || last_row.offsetTop - 250 <= container.scrollTop + container.offsetHeight) {
if (Article.getActive() === parseInt(last_row.getAttribute('data-article-id'))
|| last_row.offsetTop - 250 <= container.scrollTop + container.offsetHeight) {
hsp.innerHTML = `<span class='text-muted text-small text-center'><img class="icon-three-dots" src="${App.getInitParam('icon_three_dots')}"> ${__("Loading, please wait...")}</span>`;
Headlines.loadMore();
@ -415,7 +416,7 @@ const Headlines = {
Headlines.setCommonClasses(this.headlines.filter((h) => h.id).length);
App.findAll("#headlines-frame > div[id*=RROW]").forEach((row) => {
const id = row.getAttribute("data-article-id");
const id = parseInt(row.getAttribute('data-article-id'));
const hl = this.headlines[id];
if (hl) {
@ -1189,7 +1190,7 @@ const Headlines = {
App.findAll("#headlines-frame > div[id*=RROW][class*=Selected]").forEach(
function (child) {
rv.push(child.getAttribute("data-article-id"));
rv.push(parseInt(child.getAttribute('data-article-id')));
});
// consider active article a honorary member of selected articles
@ -1205,7 +1206,7 @@ const Headlines = {
children.forEach(function (child) {
if (Element.visible(child)) {
rv.push(child.getAttribute("data-article-id"));
rv.push(parseInt(child.getAttribute('data-article-id')));
}
});
@ -1236,7 +1237,7 @@ const Headlines = {
for (let i = 0; i < rows.length; i++) {
const row = rows[i];
const id = row.getAttribute('data-article-id');
const id = parseInt(row.getAttribute('data-article-id'));
if (id === start || id === stop) {
if (!collecting) {
@ -1415,14 +1416,16 @@ const Headlines = {
menu.addChild(new dijit.MenuItem({
label: __("Open original article"),
onClick: function (/* event */) {
Article.openInNewWindow(this.getParent().currentTarget.getAttribute("data-article-id"));
const id = parseInt(this.getParent().currentTarget.getAttribute('data-article-id'));
Article.openInNewWindow(id);
}
}));
menu.addChild(new dijit.MenuItem({
label: __("Display article URL"),
onClick: function (/* event */) {
Article.displayUrl(this.getParent().currentTarget.getAttribute("data-article-id"));
const id = parseInt(this.getParent().currentTarget.getAttribute('data-article-id'));
Article.displayUrl(id);
}
}));
@ -1431,11 +1434,10 @@ const Headlines = {
menu.addChild(new dijit.MenuItem({
label: __("Toggle unread"),
onClick: function () {
const id = parseInt(this.getParent().currentTarget.getAttribute('data-article-id'));
let ids = Headlines.getSelected();
// cast to string
const id = (this.getParent().currentTarget.getAttribute("data-article-id")) + "";
ids = ids.length !== 0 && ids.indexOf(id) !== -1 ? ids : [id];
ids = ids.includes(id) ? ids : [id];
Headlines.selectionToggleUnread({ids: ids, no_error: 1});
}
@ -1444,10 +1446,10 @@ const Headlines = {
menu.addChild(new dijit.MenuItem({
label: __("Toggle starred"),
onClick: function () {
const id = parseInt(this.getParent().currentTarget.getAttribute('data-article-id'));
let ids = Headlines.getSelected();
// cast to string
const id = (this.getParent().currentTarget.getAttribute("data-article-id")) + "";
ids = ids.length !== 0 && ids.indexOf(id) !== -1 ? ids : [id];
ids = ids.includes(id) ? ids : [id];
Headlines.selectionToggleMarked(ids);
}
@ -1456,10 +1458,10 @@ const Headlines = {
menu.addChild(new dijit.MenuItem({
label: __("Toggle published"),
onClick: function () {
const id = parseInt(this.getParent().currentTarget.getAttribute('data-article-id'));
let ids = Headlines.getSelected();
// cast to string
const id = (this.getParent().currentTarget.getAttribute("data-article-id")) + "";
ids = ids.length !== 0 && ids.indexOf(id) !== -1 ? ids : [id];
ids = ids.includes(id) ? ids : [id];
Headlines.selectionTogglePublished(ids);
}
@ -1470,14 +1472,14 @@ const Headlines = {
menu.addChild(new dijit.MenuItem({
label: __("Mark above as read"),
onClick: function () {
Headlines.catchupRelativeTo(0, this.getParent().currentTarget.getAttribute("data-article-id"));
Headlines.catchupRelativeTo(0, parseInt(this.getParent().currentTarget.getAttribute('data-article-id')));
}
}));
menu.addChild(new dijit.MenuItem({
label: __("Mark below as read"),
onClick: function () {
Headlines.catchupRelativeTo(1, this.getParent().currentTarget.getAttribute("data-article-id"));
Headlines.catchupRelativeTo(1, parseInt(this.getParent().currentTarget.getAttribute('data-article-id')));
}
}));
@ -1499,12 +1501,10 @@ const Headlines = {
label: name,
labelId: bare_id,
onClick: function () {
const id = parseInt(this.ownerMenu.currentTarget.getAttribute('data-article-id'));
let ids = Headlines.getSelected();
// cast to string
const id = (this.getParent().ownerMenu.currentTarget.getAttribute("data-article-id")) + "";
ids = ids.length !== 0 && ids.indexOf(id) !== -1 ? ids : [id];
ids = ids.includes(id) ? ids : [id];
Headlines.selectionAssignLabel(this.labelId, ids);
}
@ -1514,11 +1514,10 @@ const Headlines = {
label: name,
labelId: bare_id,
onClick: function () {
let ids = Headlines.getSelected();
// cast to string
const id = (this.getParent().ownerMenu.currentTarget.getAttribute("data-article-id")) + "";
const id = parseInt(this.ownerMenu.currentTarget.getAttribute('data-article-id'));
ids = ids.length !== 0 && ids.indexOf(id) !== -1 ? ids : [id];
let ids = Headlines.getSelected();
ids = ids.includes(id) ? ids : [id];
Headlines.selectionRemoveLabel(this.labelId, ids);
}