Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Error when clicking on/off button in Auto-Timer #1396

Closed
TheEarthrise opened this issue Sep 1, 2021 · 5 comments
Closed

Error when clicking on/off button in Auto-Timer #1396

TheEarthrise opened this issue Sep 1, 2021 · 5 comments
Assignees
@wedebe
Labels
BUG 🐞 Not working as expected HIGH ❗️ Considered to be of higher priority Modern Interface *Only* affects the responsive design

Comments

@TheEarthrise
Copy link

When i click the on/off button on an auto-timer entry, i get this:
grafik

Tested with multiple browsers.

Version: 1.4.8-git20210830-r2
System: VTi 15.0
Hardware: VU+ UNO 4K SE
@rdamas
Copy link
Contributor

rdamas commented Sep 2, 2021

Reproducible here:

root@vusolo4k:~# curl -v 'http://vusolo4k/autotimer/change?id=7&enabled=1'
*   Trying 2a02:8109:283f:f5f9:21d:ecff:fe0a:beae...
* Connected to vusolo4k (2a02:8109:283f:f5f9:21d:ecff:fe0a:beae) port 80 (#0)
> GET /autotimer/change?id=7&enabled=1 HTTP/1.1
> Host: vusolo4k
> User-Agent: curl/7.42.1
> Accept: */*
> 
< HTTP/1.1 404 Not Found
< Date: Thu, 02 Sep 2021 14:18:35 GMT
< Content-Length: 153
< Content-Type: text/html; charset=utf-8
< Server: TwistedWeb/16.2.0
< Set-Cookie: TWISTED_SESSION=430ddb273dda158137c167f95fd0aacc; Path=/
< 

<html>
  <head><title>404 - No Such Resource</title></head>
  <body>
    <h1>No Such Resource</h1>
    <p>No such child resource.</p>
  </body>
</html>

@jbleyel
Copy link
Contributor

jbleyel commented Sep 2, 2021

Looks like autotimer plugin on VTi do not have "change" function?

@jbleyel
Copy link
Contributor

jbleyel commented Sep 2, 2021

Hi @rdamas do you have AutoTimerChangeResource?

@rdamas
Copy link
Contributor

rdamas commented Sep 2, 2021 •
edited

No, no AutoTimerChangeResource:

root@vusolo4k:/usr/lib/enigma2/python/Plugins/Extensions/AutoTimer# grep class AutoTimerResource.py
class AutoTimerBaseResource(resource.Resource):
class AutoTimerBackgroundThread(threading.Thread):
class AutoTimerBackgroundingResource(AutoTimerBaseResource, threading.Thread):
class AutoTimerDoParseResource(AutoTimerBackgroundingResource):
class AutoTimerSimulateResource(AutoTimerBackgroundingResource):
class AutoTimerTestResource(AutoTimerBackgroundingResource):
class AutoTimerListAutoTimerResource(AutoTimerBaseResource):
class AutoTimerRemoveAutoTimerResource(AutoTimerBaseResource):
class AutoTimerAddOrEditAutoTimerResource(AutoTimerBaseResource):
class AutoTimerChangeSettingsResource(AutoTimerBaseResource):
class AutoTimerSettingsResource(resource.Resource):

IIUC the classic view uses /autotimer/edit for this purpose?

@jbleyel jbleyel added BUG 🐞 Not working as expected HIGH ❗️ Considered to be of higher priority Modern Interface *Only* affects the responsive design labels Sep 2, 2021
@wedebe
Copy link
Collaborator

wedebe commented Sep 2, 2021 •
edited

Hmmm, I have the same issue on OpenATV7.0 as of 2021-08-31.
We did some work a good while back to allow rename/dis/enabling an AutoTimer by id (edit requires the entire AutoTimer object). I can't find that code in the repo tho :-/

I've found the changes at
https://github.com/oe-alliance/enigma2-plugins/blob/master/autotimer/src/AutoTimerResource.py#L460
and
https://github.com/oe-alliance/enigma2-plugins/blob/python3/autotimer/src/AutoTimerResource.py#L472

I'll investigate a feature-detect approach for hiding the buttons when an image doesn't have the required code.

@jbleyel jbleyel mentioned this issue Sep 30, 2021
Closed
@TheEarthrise TheEarthrise mentioned this issue Oct 8, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wedebe wedebe

Labels
BUG 🐞 Not working as expected HIGH ❗️ Considered to be of higher priority Modern Interface *Only* affects the responsive design
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jbleyel @wedebe @rdamas @TheEarthrise