Skip to content

/ independent-publisher

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

Jump to bottom

broken output when get_comment_author_link contains percent sign #248

Closed
mritzmann opened this issue Aug 21, 2016 · 2 comments
Closed

broken output when get_comment_author_link contains percent sign #248

mritzmann opened this issue Aug 21, 2016 · 2 comments
Labels
bug
Milestone
v1.8

Comments

@mritzmann
Copy link

@mritzmann mritzmann commented Aug 21, 2016

Error message:
Warning: printf(): Too few arguments in /home/httpd/vhosts/*****/wp-content/themes/independent-publisher/inc/template-tags.php on line 93

Error appears when a guet submit a comment that contains a % in the Webseite-URL.
@raamdev raamdev added the bug label Aug 23, 2016
@raamdev raamdev added this to the Next Release milestone Aug 23, 2016
@raamdev
Copy link
Owner

@raamdev raamdev commented Aug 23, 2016

@mritzmann Thank you for the bug report! I've confirmed that this is in fact a bug and I'll make sure this gets fixed for the release.
raamdev added a commit that referenced this issue Nov 11, 2016
@raamdev
Fix bug when get_comment_author_link contains percent sign
Verified
This commit was signed with a verified signature.
raamdev Raam Dev
GPG key ID: 0EB5BB6BD84199EA Learn about signing commits
fae84b2 
See #248
@raamdev
Copy link
Owner

@raamdev raamdev commented Nov 11, 2016

@mritzmann This has been fixed in the master branch and this fix will go out with the next release. Thanks again for reporting! :-)
@raamdev raamdev closed this Nov 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
v1.8
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@raamdev @mritzmann