Skip to content

Commit

Permalink
luci-base: actually use the packaged options for poll.add calls
Browse files Browse the repository at this point in the history
Signed-off-by: Paul Donald <[email protected]>
  • Loading branch information
systemcrash committed Oct 24, 2024
1 parent 345f6b8 commit e34c268
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions modules/luci-base/htdocs/luci-static/resources/luci.js
Original file line number Diff line number Diff line change
Expand Up @@ -925,7 +925,7 @@
* @hideconstructor
* @classdesc
*
* The `Request.poll` class provides some convince wrappers around
* The `Request.poll` class provides some convenience wrappers around
* {@link LuCI.poll} mainly to simplify registering repeating HTTP
* request calls as polling functions.
*/
Expand Down Expand Up @@ -980,7 +980,7 @@
opts = Object.assign({}, options, { timeout: ival * 1000 - 5 });

var fn = function() {
return Request.request(url, options).then(function(res) {
return Request.request(url, opts).then(function(res) {
if (!Poll.active())
return;

Expand Down

2 comments on commit e34c268

@zxlhhyccc
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Failed to check the firmware version, causing the firmware version to be empty.

Image

@systemcrash
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sigh nothing to do with this commit. See #7343

Please sign in to comment.