Skip to content

exclude_from_moderation not properly checked when submitting comments #3349

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
intoeetive opened this issue May 1, 2023 · 0 comments · Fixed by #3359
Closed

exclude_from_moderation not properly checked when submitting comments #3349

intoeetive opened this issue May 1, 2023 · 0 comments · Fixed by #3359
Assignees
@intoeetive

Comments

@intoeetive
Copy link
Contributor

intoeetive commented May 1, 2023 •
edited
Loading

When submitting comment as guest, the following is added to error log

ERROR - 2023-05-01 03:31:53 --> Severity: E_WARNING  --> Undefined array key "exclude_from_moderation" 
\ExpressionEngine-Private\system\ee\ExpressionEngine\Addons\comment\mod.comment.php 1515

We need to update the check to return true if it's set to y for any of assigned roles
@intoeetive intoeetive self-assigned this May 1, 2023
intoeetive added a commit that referenced this issue May 3, 2023
@intoeetive
Resolved #3349 where comment moderation preference were not fully che…
eca01c2 
…cked
@intoeetive intoeetive mentioned this issue May 3, 2023
Merged
intoeetive added a commit that referenced this issue May 3, 2023
@intoeetive
Resolved #3349 where comment moderation preference were not fully che…
797997d 
…cked

(cherry picked from commit eca01c2)
@intoeetive intoeetive mentioned this issue May 3, 2023
Merged
intoeetive added a commit that referenced this issue Jun 16, 2023
@intoeetive
Resolved #3349 where comment moderation preference were not fully che…
5c5304c 
…cked
intoeetive added a commit that referenced this issue Jun 26, 2023
@intoeetive
Resolved #3349 where comment moderation preference were not fully che…
10cf357 
…cked

(cherry picked from commit eca01c2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@intoeetive intoeetive

Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@intoeetive