diff options
| author | wn_ <[email protected]> | 2021-03-08 15:38:52 +0000 |
|---|---|---|
| committer | wn_ <[email protected]> | 2021-03-08 15:38:52 +0000 |
| commit | fed5158ec50788026500a3e280b52e01e6cec883 (patch) | |
| tree | 2059e88347656feb54129fa33d8affb5289eab17 /js/PrefHelpers.js | |
| parent | 28dd255c30c7bdb528ec1fde2391705fbc9b4cf6 (diff) | |
Default to null 'rv' for plugin update check.
Previously 'rv' was returned as an empty JS array, causing 'p.rv.git_status != 0' to evaluate to true and a misleading 'Ready to update' appearing for certain plugins.
Diffstat (limited to 'js/PrefHelpers.js')
| -rw-r--r-- | js/PrefHelpers.js | 40 |
1 files changed, 21 insertions, 19 deletions
diff --git a/js/PrefHelpers.js b/js/PrefHelpers.js index 651e3f609..3f738aa95 100644 --- a/js/PrefHelpers.js +++ b/js/PrefHelpers.js @@ -697,28 +697,30 @@ const Helpers = { } else { reply.forEach((p) => { - if (p.rv.need_update) { - dialog.plugins_to_update.push(p.plugin); + if (p.rv) { + if (p.rv.need_update) { + dialog.plugins_to_update.push(p.plugin); - const update_button = dijit.getEnclosingWidget( - App.find(`*[data-update-btn-for-plugin="${p.plugin}"]`)); + const update_button = dijit.getEnclosingWidget( + App.find(`*[data-update-btn-for-plugin="${p.plugin}"]`)); - if (update_button) - update_button.domNode.show(); - } + if (update_button) + update_button.domNode.show(); + } - if (p.rv.need_update || p.rv.git_status != 0) { - container.innerHTML += - ` - <li><h3>${p.plugin}</h3> - ${p.rv.stderr ? `<pre class="small text-error pre-wrap">${p.rv.stderr}</pre>` : ''} - ${p.rv.stdout ? `<pre class="small text-success pre-wrap">${p.rv.stdout}</pre>` : ''} - <div class="small"> - ${p.rv.git_status ? App.FormFields.icon("error_outline") + " " + __("Exited with RC: %d").replace("%d", p.rv.git_status) : - App.FormFields.icon("check") + " " + __("Ready to update")} - </div> - </li> - ` + if (p.rv.need_update || p.rv.git_status != 0) { + container.innerHTML += + ` + <li><h3>${p.plugin}</h3> + ${p.rv.stderr ? `<pre class="small text-error pre-wrap">${p.rv.stderr}</pre>` : ''} + ${p.rv.stdout ? `<pre class="small text-success pre-wrap">${p.rv.stdout}</pre>` : ''} + <div class="small"> + ${p.rv.git_status ? App.FormFields.icon("error_outline") + " " + __("Exited with RC: %d").replace("%d", p.rv.git_status) : + App.FormFields.icon("check") + " " + __("Ready to update")} + </div> + </li> + ` + } } dialog.checkNextPlugin(); }); |