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.

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

The status of "Publish Series on VIDEOPORTAL" does not get saved #212

Closed
dagraf opened this issue Nov 8, 2023 · 5 comments
Closed

The status of "Publish Series on VIDEOPORTAL" does not get saved #212

dagraf opened this issue Nov 8, 2023 · 5 comments
Assignees
Labels
bug Something isn't working Priority: High Needs to be fixed/taken care of asap

Comments

@dagraf
Copy link

dagraf commented Nov 8, 2023

Describe the bug
If a course administrator activates the setting "Publish Series on VIDEOPORTAL", the chosen option does not get saved after clicking on 'Save'. The roles that get sent to Opencast are correct. But the setting "Publish Series on VIDEOPORTAL" gets shown as not activated. I assume that instead of saving and showing what was set, after hitting save the default template gets shown.

To Reproduce
Steps to reproduce the behavior:

  1. In the plugin configuration: Have some templates configured and set one of them as "Default Template"
  2. Open a series ane go to the tab 'Settings'
  3. Activate the option 'Publish Series on Tobira', choose one of the configured permissions and click on 'Save'
  4. See error: The option "Publish Series on VIDEOPORTAL' is NOT activated.

Expected behavior
The chosen option should not get lost when clicking on "Save".

Environment (please complete the following information):

  • ILIAS version: v7.24 2023-07-20
  • PHP version: PHP 7.4.3
  • Plugin version: 5.3.0
  • Opencast version: 13
  • Browser: Firefox

Additional context
I assume that the bug exists since merging the PR #186. Therefore, @chfsx could you please have a look at this issue?

@dagraf dagraf added bug Something isn't working Priority: High Needs to be fixed/taken care of asap labels Nov 8, 2023
@chfsx
Copy link
Contributor

chfsx commented Nov 21, 2023

The feature seems to me to be difficult to understand by design, I may need more information here:

this is implemented (and works) in such a way that saving works if the following circumstances apply:

  • at least one additional permission template has been created
  • the default template cannot be selected and is not displayed in the form
  • the additional (and selectable) template must have READ and WRITE activated, otherwise it will not be saved in the form

i can't say why this is the case, but it was already the case before. maybe we need to look at the feature together so that i understand it completely and why it has been implemented this way so far.

@dagraf
Copy link
Author

dagraf commented Nov 21, 2023

Thanks for having looked into it. I was aware of the first two bullet points and have some reasons I'm happy to explain in a short meeting (see message in Element/Matrix). Maybe there, we could also come up with solution which is more reasonable or easier to understand.

The third bullet point may cause the problem. In our template the additional templates only have READ activated, WRITE is not activated. So maybe saving the form independent of the status of READ and WRITE would be the solution. Let's discuss this in a short meeting.

@dagraf
Copy link
Author

dagraf commented Nov 21, 2023

@chfsx: See this PR #153 for more information about the feature.

@chfsx
Copy link
Contributor

chfsx commented Nov 23, 2023

we remove the the additional (and selectable) template must have READ and WRITE activated, otherwise it will not be saved in the form implementation which makes no sense at all.

@dagraf
Copy link
Author

dagraf commented Nov 30, 2023

I did a retest with the merged PR #231 and can confirm that everything works now as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Priority: High Needs to be fixed/taken care of asap
Projects
None yet
Development

No branches or pull requests

2 participants