There is a need for us to target specific form's instead of all forms on the site. I created a patch that allows for this to happen based on the form id. You just have to go into the hide submit button admin page and add a list of ids to target specific forms, and if left empty it will target all forms.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

marioangulo created an issue. See original summary.

marioangulo’s picture

marioangulo’s picture

Updated method name that creates array of ids to be more specific to what it does.

snehalgaikwad’s picture

Assigned: marioangulo » snehalgaikwad
snehalgaikwad’s picture

Assigned: snehalgaikwad » Unassigned
Status: Active » Needs review
FileSize
9.69 KB

Here is updated patch with option to enable or disable hide submit button functionality for specific forms.

pratik_kamble’s picture

@snehalgaikwad I was unable to apply the patch. Please use PHP_EOL to create the array of form ids. I think the current implementation of the logic to pass the current form id needs refactoring.

pratik_kamble’s picture

Status: Needs review » Needs work
siddhant.bhosale’s picture

Assigned: Unassigned » siddhant.bhosale
siddhant.bhosale’s picture

Assigned: siddhant.bhosale » Unassigned
Status: Needs work » Needs review
FileSize
5.16 KB

Fixed the mentioned issue.

tanmaykadam’s picture

Assigned: Unassigned » tanmaykadam
tanmaykadam’s picture

Please add the configuration settings
/admin/config/content/hide-submit

Please add the form specific settings

Please save the node

Button will disappear.

Patch is working properly.

tanmaykadam’s picture

Assigned: tanmaykadam » Unassigned
tanmaykadam’s picture

Status: Needs review » Reviewed & tested by the community
nehajyoti’s picture

For #8 - Does this makes sense to change variables to hide_submit_form_ids & hide_submit_negate to keep consistency with all the existing variables?

siddhant.bhosale’s picture

Assigned: Unassigned » siddhant.bhosale
siddhant.bhosale’s picture

I have changed the variable names and uploading the patch for the same.

nehajyoti’s picture

Status: Reviewed & tested by the community » Fixed
marioangulo’s picture

Status: Fixed » Closed (fixed)

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