Mailing List Archive

[Bricolage #212] output channels list in story/media profile out of context
// Add your reply above here
==================================================
Zdravko Balorda updated this ticket at October 19th, 2010 @ 06:29 AM

The story profile page involves a section for assigning OC to a story. As it is
all OC associated to a Story element are listed regardless of site context. Proper
behaviour would be to list only OC associated to a story site only. I suggest a fix
for this:

+ % my $sid = $story->get_site_id;
% my $all_ocs = [
+ % grep { $_->get_site_id == $sid && !$curr_ocs->{$_->get_id} && chk_authz($_, READ, 1) }
- % grep { ! $curr_ocs->{$_->get_id} && chk_authz($_, READ, 1) }
% $story->get_element_type->get_output_channels
% ];
in comp/widgets/story_prof/list_output_channels.html. The same goes for media profile.

Regards, Zdravko
--------------------------------------------------------------------------------

Assigned: Bugs (at bricolage)
State: new
View this ticket online: http://bricolage.lighthouseapp.com/projects/29601/tickets/212-output-channels-list-in-storymedia-profile-out-of-context
Stop being notified of this ticket's changes: http://bricolage.lighthouseapp.com/projects/29601/tickets/212-output-channels-list-in-storymedia-profile-out-of-context/watch
Update your Profile: http://bricolage.lighthouseapp.com/profile
Support: support@lighthouseapp.com or http://help.lighthouseapp.com
[Bricolage #212] output channels list in story/media profile out of context [ In reply to ]
// Add your reply above here
==================================================
phillipadsmith updated this ticket at October 30th, 2010 @ 12:05 AM
* State changed from "new" to "hold"
* Milestone changed from "" to "2.0.1"
> The story profile page involves a section for assigning OC to a story. As it is
all OC associated to a Story element are listed regardless of site context. Proper
behaviour would be to list only OC associated to a story site only. I suggest a fix
for this:

Hey there Zdravko,

Would you be able to provide this as a patch, or pull request in Github? Would make it much easier to test and commit.

Phillip.
--------------------------------------------------------------------------------

Assigned: Bugs (at bricolage)
State: hold
Milestone: 2.0.1
View this ticket online: http://bricolage.lighthouseapp.com/projects/29601/tickets/212-output-channels-list-in-storymedia-profile-out-of-context
Stop being notified of this ticket's changes: http://bricolage.lighthouseapp.com/projects/29601/tickets/212-output-channels-list-in-storymedia-profile-out-of-context/watch
Update your Profile: http://bricolage.lighthouseapp.com/profile
Support: support@lighthouseapp.com or http://help.lighthouseapp.com