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.
I'm using editablefields to make a 'priority' field editable in a 'tasklist' view.
However, currently this is as good as useless in real life, because after selecting a priority, it takes about 15 seconds for the edit to process. What could be the reasons that this takes so long?
Comment | File | Size | Author |
---|---|---|---|
#7 | editablefields-2429467-7.patch | 1.74 KB | StoraH |
#3 | editablefields-2429467.patch | 1.8 KB | Renee S |
Comments
Comment #1
Renee S CreditAttribution: Renee S commentedThe reason is because the editablefield loads the whole dang view each time you edit a field. I've got a line on an almost-working patch for this, and it will be posted on Monday!
Comment #2
catmurray CreditAttribution: catmurray commentedHas there been any progress on this? Much appreciated!
Comment #3
Renee S CreditAttribution: Renee S commentedAh, yes! The attached patch was contributed by @rickyoh.
Comment #4
Renee S CreditAttribution: Renee S commentedIt replaces the editablefields markup with inline forms. The initial view rendering is no faster*, but the AJAX part is substantially improved.
* This can be improved if you allow form caching to be routed to memcache or disk. There's some concern that a cache-full flush could lose form information if somebody is mid-edit, but using long-lived caches and disk with lots of space I haven't found this to ever be an issue. It would definitely be a problem if you were caching to APC or something more ephemeral.
Comment #5
Renee S CreditAttribution: Renee S commentedComment #6
Renee S CreditAttribution: Renee S commentedComment #7
StoraH CreditAttribution: StoraH commentedThanks! This will hopefully fix a major issue we have with a lot of editors changing content at the same time. I just update paths in the patch.
Comment #8
joelpittetQuick review
Probably should remove the line if this is not needed anymore?
I'm really unsure why this needed to be changed?