Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Big changes are coming in #1024586: Field machine names don't respect the field title. Let's get ready.
Comment | File | Size | Author |
---|---|---|---|
#2 | fullcalendar-1062656-2.patch | 5.69 KB | tim.plunkett |
#1 | fullcalendar-1062656-1.patch | 1.64 KB | tim.plunkett |
Comments
Comment #1
tim.plunkettTested with http://drupal.org/node/1024586#comment-4095330
Comment #2
tim.plunkettHere's a better one.
Comment #3
aspilicious CreditAttribution: aspilicious commentedOk great, I always wondered why we needed the parent options.
general question
----------------
This code will find ALL the date fields in our system even if we don't need them?
Will this not affect performance?
EDIT:
And what will happen if you have a date field not added to the view. That field would not be a part of $data I guess. So $data['field_data_' . $id][$id]['field']['handler'] doesn't exist and it will throw errors?
Comment #4
tim.plunketthook_views_data_alter runs only when the view cache is cleared, and it loops through EVERY possible field on a site. So, no problem, and not a real performance issue.
Comment #5
tim.plunkettOkay, #1024586: Field machine names don't respect the field title is in, this is a big deal now.
Comment #6
aspilicious CreditAttribution: aspilicious commentedAfter some serious brain dead testing I'm positive about the fact that this patch is working :).
Comment #7
tim.plunketthttp://drupal.org/cvs?commit=501200