mirror of
https://github.com/silverstripe/silverstripe-reports
synced 2024-10-22 11:05:53 +02:00
BUGFIX: fixed Member name not being saved in database with ReadonlyField()
git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/cms/trunk@60897 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
parent
6178a9838d
commit
cdb19344bf
@ -6,7 +6,7 @@
|
|||||||
*/
|
*/
|
||||||
class PageComment extends DataObject {
|
class PageComment extends DataObject {
|
||||||
static $db = array(
|
static $db = array(
|
||||||
"Name" => "Varchar",
|
"Name" => "Varchar(200)",
|
||||||
"Comment" => "Text",
|
"Comment" => "Text",
|
||||||
"IsSpam" => "Boolean",
|
"IsSpam" => "Boolean",
|
||||||
"NeedsModeration" => "Boolean"
|
"NeedsModeration" => "Boolean"
|
||||||
|
@ -113,7 +113,11 @@ class PageCommentInterface extends RequestHandlingData {
|
|||||||
$member = Member::currentUser();
|
$member = Member::currentUser();
|
||||||
|
|
||||||
if((self::$comments_require_login || self::$comments_require_permission) && $member && $member->FirstName) {
|
if((self::$comments_require_login || self::$comments_require_permission) && $member && $member->FirstName) {
|
||||||
$fields->push(new ReadonlyField("Name", _t('PageCommentInterface.YOURNAME', 'Your name'), $member->getName()));
|
// note this was a ReadonlyField - which displayed the name in a span as well as the hidden field but
|
||||||
|
// it was not saving correctly. Have changed it to a hidden field. It passes the data correctly but I
|
||||||
|
// believe the id of the form field is wrong.
|
||||||
|
$fields->push(new ReadonlyField("NameView", _t('PageCommentInterface.YOURNAME', 'Your name'), $member->getName()));
|
||||||
|
$fields->push(new HiddenField("Name", "", $member->getName()));
|
||||||
} else {
|
} else {
|
||||||
$fields->push(new TextField("Name", _t('PageCommentInterface.YOURNAME', 'Your name')));
|
$fields->push(new TextField("Name", _t('PageCommentInterface.YOURNAME', 'Your name')));
|
||||||
}
|
}
|
||||||
@ -206,7 +210,6 @@ class PageCommentInterface_Form extends Form {
|
|||||||
return "spamprotectionfailed"; //used by javascript for checking if the spam question was wrong
|
return "spamprotectionfailed"; //used by javascript for checking if the spam question was wrong
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
Cookie::set("PageCommentInterface_Name", $data['Name']);
|
Cookie::set("PageCommentInterface_Name", $data['Name']);
|
||||||
|
|
||||||
// If commenting can only be done by logged in users, make sure the user is logged in
|
// If commenting can only be done by logged in users, make sure the user is logged in
|
||||||
|
Loading…
Reference in New Issue
Block a user