Ticket #8446 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Macro should check for access privileges

Reported by: mitar Assigned to: rjollos
Priority: high Component: AllAttachmentsMacro
Severity: normal Keywords:
Cc: mitar, otaku42 Trac Release: 0.11

Description

Currently macro displays all attachments, even those for which user does not have access privileges. My fork does this. Please pull it in.

Attachments

Change History

01/29/11 03:59:52 changed by rjollos

  • priority changed from normal to high.
  • status changed from new to assigned.

Thank you, will pull this in soon.

02/13/11 06:40:40 changed by rjollos

Let me know if you want write access to this project, happy to have you contributing!

02/13/11 06:46:57 changed by rjollos

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [9853]) Added permission check. Patch by mitar. Fixes #8446.

02/13/11 06:52:22 changed by rjollos

(In [9854]) Reverting accidental change to revision info in [9853]. Refs #8446.

02/13/11 06:55:09 changed by rjollos

Removed link on main page that pointed to the fork since this has now been pulled in.

(follow-up: ↓ 7 ) 02/13/11 12:28:34 changed by mitar

I would be gladly to contribute directly and not over the fork. ;-) Especially because trac-hacks still uses SVN.

(in reply to: ↑ 6 ; follow-up: ↓ 8 ) 02/13/11 12:30:22 changed by rjollos

  • cc changed from mmitar@gmail.com to mitar, otaku42.

Replying to mitar:

I would be gladly to contribute directly and not over the fork. ;-) Especially because trac-hacks still uses SVN.

Michael, please add mitar as a comitter for the AllAttachmentsMacro.

(in reply to: ↑ 7 ) 02/13/11 20:06:21 changed by otaku42

Replying to rjollos:

Michael, please add mitar as a comitter for the AllAttachmentsMacro.

Done.

Add/Change #8446 (Macro should check for access privileges)




Change Properties
Action