Piwigo Bugtracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0002152Piwigouser commentspublic2011.01.29 12:562011.04.16 20:06
Reportereric 
Assigned Tomistic100 
PrioritynormalSeverityfeatureReproducibilityalways
StatusclosedResolutionfixed 
PlatformWampServerOSWindows VistaOS Version2.1
Product Version2.1.6 
Target Version2.2.1Fixed in Version2.2.1 
Summary0002152: Comments revalidation when modified
DescriptionIf gallery is set for the admin validates comments filed before publication and allows users to edit their own comments, users can modify the comments as they want (that's normal) but the admin no longer has to validate them again and they are immediately visible.

To enforce protection against Spam, admins should revalidate every change on comments in this case.
Additional Informationhttp://fr.piwigo.org/forum/viewtopic.php?id=19385 [^]
TagsNo tags attached.
browserany
Database engine and version
PHP version
Web serverApache 1.3.x
Attached Files

- Relationships

-  Notes
(0004951)
svn (reporter)
2011.04.06 10:55

[Subversion] r10097 by mistic100 on trunk

-----[Subversion commit log]----------------------------------------------------
bug:2152 Comments revalidation when modified
(0004952)
rvelices (developer)
2011.04.06 11:20

Quick question: is there a reason to have changed picture_comment.inc.php in your commit ?

The test if (!isset($comment_action)) is completeley useless in my opinion ..
(0004953)
mistic100 (developer)
2011.04.06 11:31

it is, because for display a message I can't redirect the user, then $_POST['content'] is defined, and the page 'picture_comment.inc.php' try to add a new comment with the content of the updated comment.

But... it could be easier to unset $_POST['content'] after saving the comment.
will change this
(0004960)
svn (reporter)
2011.04.06 21:24

[Subversion] r10122 by mistic100 on trunk

-----[Subversion commit log]----------------------------------------------------
bug:2152 no special parameter for updated comment validation

- Issue History
Date Modified Username Field Change
2011.01.29 12:56 eric New Issue
2011.01.29 12:56 eric browser => any
2011.01.29 12:56 eric Web server => Apache 1.3.x
2011.04.04 21:00 mistic100 Status new => assigned
2011.04.04 21:00 mistic100 Assigned To => mistic100
2011.04.06 10:55 svn Checkin
2011.04.06 10:55 svn Note Added: 0004951
2011.04.06 10:58 mistic100 Status assigned => resolved
2011.04.06 10:58 mistic100 Fixed in Version => 2.3
2011.04.06 10:58 mistic100 Resolution open => fixed
2011.04.06 11:20 rvelices Note Added: 0004952
2011.04.06 11:20 rvelices Status resolved => feedback
2011.04.06 11:20 rvelices Resolution fixed => reopened
2011.04.06 11:31 mistic100 Note Added: 0004953
2011.04.06 21:24 svn Checkin
2011.04.06 21:24 svn Note Added: 0004960
2011.04.06 21:31 mistic100 Status feedback => resolved
2011.04.06 21:31 mistic100 Fixed in Version 2.3 => 2.2.0
2011.04.06 21:31 mistic100 Resolution reopened => fixed
2011.04.16 20:05 plg Status resolved => closed
2011.04.16 20:06 plg Fixed in Version 2.2.0 => 2.2.1
2011.04.16 20:06 plg Target Version => 2.2.1


Copyright © 2000 - 2015 MantisBT Team
Contact
Powered by Mantis Bugtracker