Title says it all!

Here is a Behat test to recreate the issue: #2149175: Write Behat tests for Panopoly "content list" widget

I just wanted to add a note that this doesn't appear to be related to Panopoly Magic - the bug still happens with that module disabled.

Comments

dsnopek’s picture

Issue summary: View changes
dsnopek’s picture

dsnopek’s picture

Status: Active » Fixed

Committed in a3aa393.

caschbre’s picture

We had that issue on recent project and I had one of my devs fix it. I'll check what he did and whether or not he already submitted a patch... if for nothing else that avoiding duplicate patches to ctools. :-)

caschbre’s picture

Talked to my dev... and he had found a Views patch that solved this issue. I haven't looked in depth at either approach to see what is the better option.

Here's the patch we used.

https://drupal.org/comment/7993367#comment-7993367

dsnopek’s picture

Status: Fixed » Active

Aha! Thanks for this. :-) I think the Views patch is probably better, because it should really be the one setting sort_by's default_value and may need to do so for other situations as well. When I get a chance, I'll do some testing and make sure the fix works for Panopoly.

dsnopek’s picture

FYI, @barraponto gave an informal RTBC on IRC and yesterday I did my own testing and was about to make a commit that switched to the Views patch, but then Drupal.org went down. :-/ So, anyway, this is practically ready to I just need to finish committing!

dsnopek’s picture

Title: The "Sort by" won't save in the "Content list" widget » The "Sort by" won't save in the "Content list" widget (switch from ctools patch to views patch)
dsnopek’s picture

barraponto’s picture

I was just about to send the patch... thanks @dsnopek!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.