Bug 742083 - US47: User and team administration rights for my language team defined for the translation team coordinator
US47: User and team administration rights for my language team defined for th...
Status: CLOSED CURRENTRELEASE
Product: Zanata
Classification: Community
Component: Component-Logic, Component-UI (Show other bugs)
1.4
Unspecified Unspecified
high Severity medium
: Sprint-28
: 1.4.2
Assigned To: Carlos Munoz
Ding-Yi Chen
:
Depends On: 742854
Blocks: 693610 736898 Zanata-1.4.2
  Show dependency treegraph
 
Reported: 2011-09-28 20:29 EDT by Carlos Munoz
Modified: 2011-10-28 03:02 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: 8
Clone Of:
Environment:
Last Closed: 2011-10-28 03:02:22 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Carlos Munoz 2011-09-28 20:29:50 EDT
Tasks to complete:

- Contact Admin as an action on the Help page
- admin/coordinator page to add/remove coordinators from language team
   - security rule so that coordinators are translators
- Contact Coordinator(s) - send arbitrary email to language co-ordinator(s)
   - available to any logged-in user
   - emails all coordinators for the language team
   - sends to system-configured email address if no coordinator
   - emails to have link directly to language team page
- coordinator page to list/add/remove users from language team
- the above two pages' functionality may be combined with existing team page
- Ask to Join Language Team - generate template email when joining team
- Join Language Team - now only enabled for admin
Comment 1 David Mason 2011-10-06 00:12:12 EDT
Remaining tasks:

- Contact Coordinator(s) - send arbitrary email to language co-ordinator(s)
   - available to any logged-in user
   - emails all coordinators for the language team
   - sends to system-configured email address if no coordinator
   - emails to have link directly to language team page

- Ask to Join Language Team - generate template email when joining team
Comment 2 David Mason 2011-10-12 00:51:32 EDT
Merged into 1.4 branch
Comment 3 Ding-Yi Chen 2011-10-16 20:54:45 EDT
- admin/coordinator page to add/remove coordinators from language team
   - security rule so that coordinators are translators

Which one does it means?
1. Once assigned as coordinators, the user automatically gains "translator" role.
2. A user need to have translator role to be able to be added as coordinator
  (and be able to be shown in "Find Users To Add" dialog).

The other are VERIFIED with Zanata version 1.4.2-SNAPSHOT (20111014-1722)
Comment 4 Sean Flanigan 2011-10-17 01:40:17 EDT
Number 1, more or less.  We should write security rules (in security.drl) so that language coordinators will be allowed to add/modify/upload translations for their languages.  While we're at it, we should refactor some of the existing security checks to go through security.drl.
Comment 5 Carlos Munoz 2011-10-19 23:18:39 EDT
Added security rules to allow language team coordinators to add/modify translations. Also refactored all the tags that render conditionally based on permissions to use the security rules instead of business logic (Only for the language team page). We should probably do a sweep of the whole application looking for places where we can use security rules instead of manually validating access via business methods.
Comment 7 Carlos Munoz 2011-10-20 00:09:14 EDT
Merged into 1.4 branch.
Comment 9 Ding-Yi Chen 2011-10-20 21:09:14 EDT
On Zanata version 1.4.2-SNAPSHOT (20111021-1030)
The non-admin team member still cannot translate.

RE-ASSIGNED
Comment 10 Carlos Munoz 2011-10-20 22:10:11 EDT
Corrected security check in the Translation Unit Handler to do a check based on the language memberships.

Merged into 1.4 branch.
Comment 11 Carlos Munoz 2011-10-20 23:34:00 EDT
Corrected security check in the Translation Unit Handler to do a check based on the language memberships.

Merged into 1.4 branch.
Comment 12 Ding-Yi Chen 2011-10-21 00:11:40 EDT
VERIFIED with Zanata version 1.4.2-SNAPSHOT (20111021-1207)

Note You need to log in before you can comment on or make changes to this bug.