Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I'm changing the field 'field_project' widget type in one of my sites to 'hidden' because I only have one project and users get confused on that field in the create issue form.
There is a validation hook for that field 'project_issue_field_project_validate' which is causing issues because I've changed the widget type.
I've created this path to fix the issue, so we can change between other widgets than the default 'autocomplete'
Comment | File | Size | Author |
---|---|---|---|
#1 | project_issue-change-field_project-widget-type-2203525-1.patch | 1.49 KB | alexandrezia |
Comments
Comment #1
alexandrezia CreditAttribution: alexandrezia commentedThe patch
Comment #2
alexandrezia CreditAttribution: alexandrezia commentedComment #3
drummThere are a few places where this should follow https://drupal.org/coding-standards better:
break;
one more level.{
on the same line.Since
$widget_type
is only used once, it shouldn't be stored in a variable, just used directly in the switch condition.Since many cases have the same code, they should be grouped together and use fall through.
Comment #4
formatC'vt CreditAttribution: formatC'vt commentedjust like this