search

olsson82 / rivendellweb

Web system for manage Rivendell Radio Automation
https://olsson82.github.io/rivwebdoc/
MIT License
10 stars 1 forks source link

Forbidden editing events and clocks #34

Closed alastairtech closed 1 month ago

alastairtech commented 1 month ago

Forgive me if this isn't really an issue or a simple mistake on my behalf.

Everything is working nicely except when editing events or clocks. Apache give me a forbidden error and reports an undefined array key id in the logs.

I've doubled checked AllowOverride All is set correctly and messed with the permissions on the manager directory. Doesn't seems to help much.

Any guidance would be welcome.

Thanks.

olsson82 commented 1 month ago

Forgive me if this isn't really an issue or a simple mistake on my behalf.

Everything is working nicely except when editing events or clocks. Apache give me a forbidden error and reports an undefined array key id in the logs.

I've doubled checked AllowOverride All is set correctly and messed with the permissions on the manager directory. Doesn't seems to help much.

Any guidance would be welcome.

Thanks.

Its ok, im here to help. Have you checked so you have .htaccess file in manager folder ? It's needed for rewrite of the url should work.

The link to edit a clock shout look like this : https://url,com/manager/clock/THECLOCKNAME

/Andreas

alastairtech commented 1 month ago

Ah! Seems to be a different issue. It's unable to edit events and clocks with spaces in the name.

Confirmed working on events and clocks with no spaces.

Don't know how easy that would be to fix.

olsson82 commented 1 month ago

Ah! Seems to be a different issue. It's unable to edit events and clocks with spaces in the name.

Confirmed working on events and clocks with no spaces.

Don't know how easy that would be to fix.

Ok, thanks for let me know, this i have never tried before. I will look in to this right away, and see if i can get it to handle space.

/Andreas

olsson82 commented 1 month ago

This has been fixed and should be working now. You can update manually if you need it right now, from commit c9e4979 Will be on the next release.

alastairtech commented 1 month ago

Tested and confirmed working on my end.

Excellent work.