Ticket #409 (closed task: ready_for_review)

Opened 2 years ago

Last modified 2 years ago

[extending] basic version of authorization rules

Reported by: admin Owned by: robert
Priority: major Milestone: WLE modules into D6
Component: WLE Version:
Keywords: Cc:
MoSCoW: Must Have Estimated time after impact analysis:
Related to project: none Tested: no
Accepted: no Estimated Hours: 7

Description

[extending media mediamosa module] At the moment no authorization rules are available in this module. With MediaMosa authorization rules can be very specific. For this version we only need the following:
- a textarea per node where author can add domain names
- let MediaMosa check the domain when a end-user requests the video

Because very often the domain will be identical per node, there should be a default settings form.

Change History

Changed 2 years ago by robert

  • owner set to dpovshed
  • status changed from new to assigned

Changed 2 years ago by dpovshed

  • owner changed from dpovshed to robert

Changed 2 years ago by dpovshed

  • status changed from assigned to closed
  • resolution set to ready_for_review

Robert,

1) please do the general review from our repo:
 http://hg.saga/mediamosa/media_mediamosa/

2) I need your advice with editing node. Seems that even if we already have mediafile attached to a field, in _media_mediamosa_change_cck we are creating a new mediafile ID by calling
_media_mediamosa_upload_prepare() .
It prevent my code to display correctly already assigned rules in the editing form. See my comment in _media_mediamosa_access_create_form, line 395 of media_mediamosa_helpers.inc
What shall we do about this? Shall I modify core code to track this change or we assume that user never editing files?

Thanks!

Changed 2 years ago by forgacs

@Robert, Dennis: I have fixed all the ACL issues. The latest version is in Freya repo.

Note: See TracTickets for help on using tickets.