Hello,

Great module. I tried a version of the 2.x branch earlier today, that seemed to work fine. When I downloaded it again for another site, things broke with the following errors as the only hits. They show up when I add the "Content: Draggable views weight" sort handler.

Notice: Undefined index: order in views_handler_sort->admin_summary() (line 47 of /home/eh/www/lacollette.com/html/sites/all/modules/views/handlers/views_handler_sort.inc).

Notice: Undefined index: order in views_handler_sort->admin_summary() (line 47 of /home/eh/www/lacollette.com/html/sites/all/modules/views/handlers/views_handler_sort.inc).
CommentFileSizeAuthor
#1 draggableviews-1410460-warning-admin-summary.patch962 bytesygerasimov
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ygerasimov’s picture

ygerasimov CreditAttribution: ygerasimov commented
Status: Active » Needs review
FileSize
draggableviews-1410460-warning-admin-summary.patch962 bytes

@egil Thank you for this bug report. I see where the problem comes from. Could you please test attached patch?

BrightBold’s picture

BrightBold
she/her
Primary language English
Location Boston, MA
CreditAttribution: BrightBold commented

This patch worked for me, and also eliminated the error in #1389440: Undefined index: draggableviews_setting_view in draggableviews_contextual_links_view_alter() (line 180 of draggableviews.module) which was occurring even after I had correctly linked the sort view with the display view. Thanks!

ygerasimov’s picture

ygerasimov CreditAttribution: ygerasimov commented
Status: Needs review » Fixed

@BrightBold. Thank you for testing! Committed.

egil’s picture

egil CreditAttribution: egil commented

@ygerasimov: yes, I can confirm it works as well.

Status: Fixed » Closed (fixed)

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