Not all sites can use contributors in a meaningful way. We should find a way to make this feature optional.

CommentFileSizeAuthor
#2 nodestream-optional-contributor-1357962-2.patch914 bytesfabsor
#2 ns-core-optional-contributor-1357962-2.patch8.59 KBfabsor

Comments

fabsor’s picture

fabsor commented

I'm opening a separate branch for this in ns_core, 1357962-optional-contributor

fabsor’s picture

fabsor commented
Title: Make contributors optional for blogs, pages and articles » Make contributors optional for blogs, pages, articles and media
Status: Active » Needs review
StatusFileSize
new ns-core-optional-contributor-1357962-2.patch8.59 KB
new nodestream-optional-contributor-1357962-2.patch914 bytes

I think we have all the functionality in place to do this now.

Here is what I have done:
* Implemented support for having optional components in default config and a UI for enabling those optional components.
* Added contributor references as optional fields for ns_article, ns_page, ns_blog and ns_media.
* Made sure that the fields are installed if you use the web product, since it relies a lot on this functionality.

In order to test that everything is working you should do the following:
1. Apply the nodestream patch to the nodestream repository and run drush make.
2. Apply the ns_core patch to ns_core.
3. Install nodestream.
4. Enable ns_article, ns_page and ns_media. None of them should have contributor fields by default.
5. go to admin/structure/extensions and enable the fields.
6. Check all content types and make sure they have fields.
7. Reinstall NodeStream
8. Enable the web newspaper product directly.
9. Contributor fields should be in place already.

pontus_nilsson’s picture

pontus_nilsson
Location Stockholm
commented

This patch nodestream-optional-contributor-1357962-2.patch is now commited.

blambi’s picture

blambi commented
Status: Needs review » Reviewed & tested by the community

I tested ns-core-optional-contributor-1357962-2.patch according to fabsor's specifications, and I also tried to break things.

It went really well, nothing broke or was there when it shouldn't or the other way around.

fabsor’s picture

fabsor commented
Status: Reviewed & tested by the community » Fixed

Alright, I merged this patch. Awesome!

Status: Fixed » Closed (fixed)

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