Verified Commit 297a2c2d authored by Pierre-Elliott Bécue's avatar Pierre-Elliott Bécue 🚼
Browse files

Approval contains the intent for emeritus/removed and therefore should also be private in that case

parent c4dc33d6
Pipeline #242037 failed with stage
in 5 minutes and 27 seconds
...@@ -97,7 +97,7 @@ class RequirementVisitorPermissions(ProcessVisitorPermissions): ...@@ -97,7 +97,7 @@ class RequirementVisitorPermissions(ProcessVisitorPermissions):
# Declarations of intent for emeritus/removed processes are only posted # Declarations of intent for emeritus/removed processes are only posted
# on -private, and must therefore be only visible to DDs # on -private, and must therefore be only visible to DDs
if self.requirement.type == "intent": if self.requirement.type in ("intent", "approval"):
if (self.process.applying_for not in (const.STATUS_EMERITUS_DD, const.STATUS_REMOVED_DD) if (self.process.applying_for not in (const.STATUS_EMERITUS_DD, const.STATUS_REMOVED_DD)
or (self.visitor is not None and self.visitor.is_dd) or (self.visitor is not None and self.visitor.is_dd)
or (self.visitor == self.person)): or (self.visitor == self.person)):
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment