- Status changed to Needs review
6 months ago 2:55pm 28 May 2024 Thanks for reopening this. Looks like there's been a reversion in DraggableViews 2.1.4. I'm pulling the following error again:
Warning: Undefined property: Drupal\views\ResultRow::$nid in Drupal\draggableviews\DraggableViews->getIndex() (line 44 of /code/web/modules/contrib/draggableviews/src/DraggableViews.php)
- 🇨🇦Canada iStryker
Is this a duplicate of 🐛 Allow any entity in DraggableViews::getIndex() Needs review . To me it looks the same, but this one tries to make hierarchery work
- Merge request !46Issue #2985291 by marcus_w, istryker: Support whatever base view entity id is → (Merged) created by iStryker
- 🇨🇦Canada iStryker
istryker → changed the visibility of the branch 2.1.x to hidden.
-
istryker →
committed ea882a72 on 2.1.x
Issue #2985291 by istryker, marcus_w: Support identifiers other than nid
-
istryker →
committed ea882a72 on 2.1.x
- 🇨🇦Canada iStryker
I fixed things up a little. I renamed everything 'base view entity id' cause that's what it is. This should support any entity type that can be a view. The only thing that might be a problem is crossing different type of relationships in the view. I am not too worried about this though, as that is probably already broken.
- Status changed to Fixed
10 days ago 12:49am 6 November 2024 Automatically closed - issue fixed for 2 weeks with no activity.