Hello everyone,
i just encountered this issue. It is not critical but i thought it was worth to report.
I accidentally gave a name to a new embed style using the "/" character in it (Audio/Video Production).
I was able to save and use the style with no issue.
But now i can't edit or delete it because when i click the edit or remove link, it tries to load an incorrect url (in the form of "/admin/config/media/vef_video_styles/list/Audio/Video%20Production/edit") .
So maybe there should be a check before saving to not allow some special characters in the style name.

Comments

plopesc’s picture

plopesc
he/him
Primary language Spanish
CreditAttribution: plopesc commented
Status: Active » Fixed

Hello zikaelismik

Thank you very much for reporting this.

Just committed a fix in fc4dfba. This patch rewrites the vef_video_styles plugins architecture to force the plugin machine_name only allows lowercase letters, numbers and underscores. Also allows to enter a human readable name.

Running the update script, you should be able to edit/delete your video_embed style.

Could you give me feedback if it does not work, please?

Regards.

zikaelismik’s picture

zikaelismik CreditAttribution: zikaelismik commented

Hello plopesc,
I updated the module and it works just fine. I was able to edit/remove the style.
I tried to add a new one with the same pattern and now the machine name works fine.
Thanks for the quick response and fix!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.