Skip to content
This repository was archived by the owner on Aug 27, 2022. It is now read-only.

Fix Typo in admin.php#322

Merged
andi34 merged 1 commit intoandi34:devfrom
F4bsi:patch-2
Sep 26, 2021
Merged

Fix Typo in admin.php#322
andi34 merged 1 commit intoandi34:devfrom
F4bsi:patch-2

Conversation

@F4bsi
Copy link
Copy Markdown

@F4bsi F4bsi commented Sep 25, 2021

Prerequisites checklist

What is the purpose of this pull request? (put an "x" next to an item)

  • Documentation update
  • Bug fix
  • New feature
  • Other, please explain:

What changes did you make? (Give an overview)

When using a custom style and not creating '../resources/css/custom_chromakeying.css' the code tries to copy the default file, which results in a warning because the path has duplicate .css and the file cannot be found:

<br />
<b>Warning</b>:  copy(../resources/css/modern_chromakeying.css.css): failed to open stream: No such file or directory in <b>/var/www/html/api/admin.php</b> on line <b>117</b><br />
"success"

This results in the admin ui staying in the "saving" state indefinitely. Also it still returns "success" which might need to be changed.

Is there anything you'd like reviewers to focus on?

When using a custom style and not creating '../resources/css/custom_chromakeying.css' the code tries to copy the default file, which results in an error because the path has duplicate .css and the file cannot be found: 

<br />
<b>Warning</b>:  copy(../resources/css/modern_chromakeying.css.css): failed to open stream: No such file or directory in <b>/var/www/html/api/admin.php</b> on line <b>117</b><br />
"success"

This results in the admin ui staying in the "saving" state indefinitely. Also it still returns "success" which might need to be changed.
@andi34 andi34 merged commit c993241 into andi34:dev Sep 26, 2021
@F4bsi F4bsi deleted the patch-2 branch September 26, 2021 08:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants