Entity reference module seems to be the new black when it comes to referencing to content. We should look into the possibility of replacing some of our node and user reference fields, since they feel a bit old and dusty. We should try and investigate this before releasing the first beta release.

Roadmap:

* Replace all specific reference fields with entity reference.
* Remove all the old views and replace with alternate solutions.
* Profit!

CommentFileSizeAuthor
#7 nodestream-use-entity-reference-1292508-7.patch1.63 KBfabsor
#7 ns-core-use-entity-reference-1292508-7.patch225.7 KBfabsor

Comments

fabsor’s picture

fabsor commented

In order to make this happen, we need to make sure this works: #1295276: Compatability with EntityReference module

pontus_nilsson’s picture

pontus_nilsson
Location Stockholm
commented

Small update on this and a reminder to myself. I am now using entity reference with the patch supplied here to see if I stumble upon any issues.

fabsor’s picture

fabsor commented
Priority: Normal » Major

We will need to get this in soon, so I'm raising the priority =)

fabsor’s picture

fabsor commented

Work on changing this is going on at: http://drupalcode.org/project/ns_core.git/shortlog/refs/heads/1292508-en...

pontus_nilsson’s picture

pontus_nilsson
Location Stockholm
commented

What's the roadmap on this issue? Any more leads than the roadmap above that states replace with alternative solution? How can I help out?

pontus_nilsson’s picture

pontus_nilsson
Location Stockholm
commented

All node reference fields are now changed to Entity reference in that branch.

fabsor’s picture

fabsor commented
Status: Active » Needs review
StatusFileSize
new ns-core-use-entity-reference-1292508-7.patch225.7 KB
new nodestream-use-entity-reference-1292508-7.patch1.63 KB

Here is a patch for all of the efforts that has been done on this issue. This has changed:

* All nodes and user references have been changed to entity references.
* All views that just fetches references on a particular node have been removed in favor of Panels Reference formatter.

The two patches attached are made for NodeStream and NS Core. The same content exists in a branch.

fabsor’s picture

fabsor commented
Status: Needs review » Needs work

This needs a bit more work.

* The entity reference CTools relationship landed recently. We need to refactor to work with that instead.
* We recently discovered that Entity reference does not have an access check when viewing entities. We are working on that problem here: #1412572: No access check when referred items are being viewed
* We also need to merge all other fixes.

fabsor’s picture

fabsor commented

The access check patch is now in entity reference.

pontus_nilsson’s picture

pontus_nilsson
Location Stockholm
commented
Priority: Major » Critical

Bumping priority just for the hell of it :) This blocks many other issues.

fabsor’s picture

fabsor commented

Almost there... =)

fabsor’s picture

fabsor commented
Status: Needs work » Fixed

This is now in 7.x-2.x! This is a huge change, and we really needed to get this in in order to move forward, so I am very happy about this!

gumdrop’s picture

gumdrop commented

Thanks for your hard work. Love how this is going.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Adding a roadmap