Skip to content
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.

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

Akismet Integration: the comment_author parameter should be restricted to the name field #1781

Open
kbrown9 opened this issue Jun 4, 2024 · 0 comments · May be fixed by #1945
Open

Akismet Integration: the comment_author parameter should be restricted to the name field #1781

kbrown9 opened this issue Jun 4, 2024 · 0 comments · May be fixed by #1945
Assignees
Milestone

Comments

@kbrown9
Copy link

kbrown9 commented Jun 4, 2024

The FrmEntryValidate::is_akismet_guest_info_value function attempts to find a value to use as the comment_author parameter for Akismet. First, it checks for name field ids. If none exist, it uses the value of the first field with a length less than 200.

Preferably, the end user would be able to select which field to use for the comment_author parameter. If that's not possible, the form should send a comment_author parameter only when the form contains a Name field. If the form does not contain a Name field, the comment_author value should be empty.

@Crabcyborg Crabcyborg assigned truongwp and unassigned AbdiTolesa Aug 20, 2024
@AbdiTolesa AbdiTolesa linked a pull request Aug 23, 2024 that will close this issue
@Crabcyborg Crabcyborg assigned AbdiTolesa and unassigned truongwp Aug 26, 2024
@Crabcyborg Crabcyborg added this to the 6.15.1 milestone Oct 1, 2024
@Crabcyborg Crabcyborg modified the milestones: 6.16, 6.16.1 Oct 22, 2024
@Crabcyborg Crabcyborg modified the milestones: 6.16.1, 6.16.2 Nov 11, 2024
@Crabcyborg Crabcyborg modified the milestones: 6.16.2, 6.16.3 Nov 20, 2024
@Crabcyborg Crabcyborg modified the milestones: 6.16.3, 6.17 Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants