diff --git a/www/artworks/get.php b/www/artworks/get.php index bed50f80..efbb8eea 100644 --- a/www/artworks/get.php +++ b/www/artworks/get.php @@ -8,13 +8,14 @@ $exception = $_SESSION['exception'] ?? null; try{ $artwork = Artwork::GetByUrl(HttpInput::Str(GET, 'artist-url-name'), HttpInput::Str(GET, 'artwork-url-name')); - $isAdminView = $GLOBALS['User']->Benefits->CanReviewArtwork ?? false; + $isReviewerView = $GLOBALS['User']->Benefits->CanReviewArtwork ?? false; + $isAdminView = $GLOBALS['User']->Benefits->CanReviewOwnArtwork ?? false; // If the artwork is not approved, and we're not an admin or the submitter when they can edit, don't show it. if( ($GLOBALS['User'] === null && $artwork->Status != ArtworkStatus::Approved) || - ($GLOBALS['User'] !== null && $artwork->SubmitterUserId != $GLOBALS['User']->UserId && !$isAdminView) + ($GLOBALS['User'] !== null && $artwork->SubmitterUserId != $GLOBALS['User']->UserId && !$isReviewerView) ){ throw new Exceptions\InvalidPermissionsException(); } @@ -69,12 +70,12 @@ catch(Exceptions\InvalidPermissionsException){
Title | -= Formatter::EscapeHtml($artwork->Name) ?> | += Formatter::EscapeHtml($artwork->Name) ?> if($isAdminView){ ?> (#= $artwork->ArtworkId ?>) } ?> |
Artist | - = Formatter::EscapeHtml($artwork->Artist->Name) ?> if(sizeof($artwork->Artist->AlternateNames) > 0){ ?> (A.K.A. | , ) } ?> if($artwork->Artist->DeathYear !== null){ ?> (d. = $artwork->Artist->DeathYear ?>) } ?> + = Formatter::EscapeHtml($artwork->Artist->Name) ?> if(sizeof($artwork->Artist->AlternateNames) > 0){ ?> (A.K.A. , ) } ?> if($artwork->Artist->DeathYear !== null){ ?> (d. = $artwork->Artist->DeathYear ?>) } ?> if($isAdminView){ ?> (#= $artwork->Artist->ArtistId ?>) } ?>|
Status | = Template::ArtworkStatus(['artwork' => $artwork]) ?> | |
Submitted by | -if($artwork->Submitter === null){ ?>Anonymous }else{ ?> if($artwork->Submitter->Name !== null){ ?> = Formatter::EscapeHtml($artwork->Submitter->Name) ?> }else{ ?>= Formatter::EscapeHtml($artwork->Submitter->Email) ?> } ?> } ?> | +if($artwork->Submitter === null){ ?>Anonymous }else{ ?> if($artwork->Submitter->Name !== null){ ?> = Formatter::EscapeHtml($artwork->Submitter->Name) ?> }else{ ?>= Formatter::EscapeHtml($artwork->Submitter->Email) ?> } ?> } ?> if($isAdminView && $artwork->Submitter !== null){ ?> (#= $artwork->Submitter->UserId ?>) } ?> |
Reviewed by | -if($artwork->Reviewer->Name !== null){ ?> = Formatter::EscapeHtml($artwork->Reviewer->Name) ?> }else{ ?>= Formatter::EscapeHtml($artwork->Reviewer->Email) ?> } ?> | +if($artwork->Reviewer->Name !== null){ ?> = Formatter::EscapeHtml($artwork->Reviewer->Name) ?> }else{ ?>= Formatter::EscapeHtml($artwork->Reviewer->Email) ?> } ?> if($isAdminView){ ?> (#= $artwork->Reviewer->UserId ?>) } ?> |