How to reproduce:

1. Create new asset type with some field or use any existing.
2. Create form_alter hook in your custom module for update form settings.
3. Paste asset in CKeditor and click "Edit" link for overriding setting.
4. Alter hooks will not implemented, because "override form" is constructed as new form ("assets_wysiwyg_form" instead of "asset_edit_ASSET_BUNDLE_form")

CommentFileSizeAuthor
#1 asset--asset_override_form_altering-2173179-1.patch704 bytesSer_Mir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Ser_Mir’s picture

Ser_Mir CreditAttribution: Ser_Mir commented
FileSize
asset--asset_override_form_altering-2173179-1.patch704 bytes
Ser_Mir’s picture

Ser_Mir CreditAttribution: Ser_Mir commented

Check my patch, please. Works fine for me

IRuslan’s picture

IRuslan CreditAttribution: IRuslan as a volunteer and at DrupalJedi commented
Status: Active » Fixed

Good catch, pushed to dev.
It's pretty ugly implemented forms and in general they should be redone in future.

  • IRuslan committed be1a136 on 7.x-1.x authored by Ser_Mir 
    Issue #2173179 by Ser_Mir: Asset WYSIWYG override form not uses alter...

Status: Fixed » Closed (fixed)

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