Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wc-3423739/3423740 - fix re-ordering options in CiviCRM Options control #953

Merged
merged 3 commits into from May 9, 2024

Conversation

bsilvern
Copy link
Contributor

Overview

Corrects problems with re-ordering options in CiviCRM Options control when option labels have been changed or when there are many options.

See:

  1. Issue 3423739
  2. Issue 3423740

Before

Unable to correctly order static options under the above conditions.

After

Able to correctly order static options under the above conditions.

@jitendrapurohit
Copy link
Collaborator

Have tested this and confirmed it works.

@bsilvern Can you please fix these warnings while applying the patch

953.diff:34: trailing whitespace.
      
953.diff:44: trailing whitespace.
      //Add to the $field_options array any options that are disabled in the webform. 
953.diff:70: trailing whitespace.
        //delta to a value greater than Drupal::config('system.site')->get('weight_select_max') (default value 100) 
953.diff:73: trailing whitespace.
        '#delta' => !$element['#civicrm_live_options'] && sizeof($all_options) > 10 ? '101' : "10", 
warning: 4 lines add whitespace errors.

@KarinG
Copy link
Collaborator

KarinG commented Apr 8, 2024

Thank you @bsilvern - I just kicked off the tests one more time and on pass will merge.

@bsilvern
Copy link
Contributor Author

bsilvern commented May 9, 2024

@KarinG Any reason this has not yet been merged?

@KarinG
Copy link
Collaborator

KarinG commented May 9, 2024

No particular reason!

@KarinG KarinG merged commit f02bd9c into colemanw:6.x May 9, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants