SNN 2.4.1 no longer can change subjects/audiences in KUPU

Bug #110570 reported by Tuomas Tonteri
10
Affects Status Importance Assigned to Milestone
SilvaNewsNetwork
Fix Released
Undecided
Andy Altepeter

Bug Description

Upgrading SNN from 2.1.2 to 2.4.1 and Silva from 1.5.9 -> 1.6.0 final, there is no longer a field in KUPU to change the subjects and audiences of News Article.

Kupu which came with 1.5.9 was 1.3.6 and Kupu, which came with 1.6 final is 1.4b1.

Tags: snn
Kit Blake (kitblake)
Changed in silva:
assignee: nobody → jasper-infrae
Revision history for this message
Tuomas Tonteri (tuomas-tonteri) wrote : Broke between 2.4 -> 2.4.1 and why

Getting this to work was so critical to me, so I started testing different Silva versions and finally found, that between 2.4 and 2.4.1 the feature broke. Then I did a diff between the directories of 2.4 and 2.4.1 and found that kupustyles.css.dtml had a good amount of changes. I copied the file over from 2.4 and now all newly created news articles again have the properties tab to change subjects and audiences.

SilvaNews/views/edit/VersionedContent/NewsItem/kupustyles.css.dtml

It's also broken for Silva 2.0 / SNN 2.5

Revision history for this message
Eric Casteleijn (thisfred) wrote :

Thank you, Tuomas! I will have a look at the differences and see whether someone can fix this on the trunk and 2.4 branch of SNN.

Revision history for this message
Eric Casteleijn (thisfred) wrote :

Wim, with Tuomas' detailed bug report you should have enough information to fix this, I think. Let me know if I can do anything to help. Just investigate who made those changes and why, so that we don't end up throwing away changes that were actually fixes for other problems.

Changed in silva:
assignee: jasper-infrae → wim-boucquaert
Revision history for this message
Andy Altepeter (aaltepet) wrote :

I'll take a look at this, as I'm basically driving the next SNN release. I can make the change in the 2.4 tag as well.

I think it would be prudent to import the SilvaDocument kupustyles in the SNN kupustyles, instead of copying. That way changes like this won't bork SilvaNews in the future...

Changed in silva:
assignee: wim-boucquaert → aaltepet
Revision history for this message
Andy Altepeter (aaltepet) wrote :

Fix committed for SNN 2.5.1

Changed in silva-news-network:
status: New → Fix Committed
Changed in silva-news-network:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.