Move checkbox for optionally omitting group content from subscriptions to vertical tab

Created on 11 April 2023, over 1 year ago
Updated 13 April 2023, over 1 year ago

Problem/Motivation

This is essentially a UI update to make the group subscription option less invasive for editors.

User interface changes

The fieldset containing the checkbox to ignore this content for future subscriptions (this is referred to in documentation as placing a hold on an item) is removed from the main portion of the node form and relocated to the vertical tabs (Menu settings, comment settings, url alias, etc)

Additional Checks

While making this change, test to be sure there are no issues if the form action chosen is preview. If there are, put guardrails in place to only make changes to the holds table if the action is actually submit. See group_subscription_form_node_form_alter().

foreach ( array_keys($form['actions']) as $action) {
      if ($action != 'preview' && isset($form['actions'][$action]['#type']) &&
          $form['actions'][$action]['#type'] === 'submit') {
        $form['actions'][$action]['#submit'][] = 'group_subscription_edit_form_submit';
      }
    }

Feature request
Status

Closed: duplicate

Version

3.0

Component

Code

Created by

🇨🇦Canada JayDarnell Guelph, Ontario

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024