Development

#3086 (sfComment component of the plugins doesn't save title)

You must first sign up to be able to contribute.

Ticket #3086 (closed defect: fixed)

Opened 9 months ago

Last modified 4 months ago

sfComment component of the plugins doesn't save title

Reported by: jtraub Assigned to: xavier
Priority: minor Milestone:
Component: sfPropelActAsCommentablePlugin Version: 1.0.11
Keywords: Cc:
Qualification: Unreviewed

Description

sfComment component which is shipped with the plugin doesn't save titles. It ignores this field at all.

Change History

04/11/08 12:39:11 changed by sgrove

I don't know how to make a patch, or anything about unit testing, but the change is a simple two-line change to

../list/sfPropelActAsCommentableBehaviorPlugin/modules/sfComment/Lig/BasesfCommentActions.class.php

first in: public function executeAuthenticatedComment()

change

$comment = array('text' => $this->getRequestParameter('sf_comment'));

to

$comment = array('text' => $this->getRequestParameter('sf_comment'), 'title' => $this->getRequestParameter('sf_comment_title'));

and also in: public function executeAnonymousComment() change

      $comment = array('text'         => $this->getRequestParameter('sf_comment'),
                       'author_name'  => $this->getRequestParameter('sf_comment_name'),
                       'author_email' => $this->getRequestParameter('sf_comment_email'), 
                       'namespace'    => $namespace);

to

      $comment = array('title'         => $this->getRequestParameter('sf_comment_title'),
                       'text'         => $this->getRequestParameter('sf_comment'),
                       'author_name'  => $this->getRequestParameter('sf_comment_name'),
                       'author_email' => $this->getRequestParameter('sf_comment_email'), 
                       'namespace'    => $namespace);

Simple enough change! Hope that helps someone, and can be included in an update sometime soon.

08/11/08 13:26:47 changed by xavier

  • status changed from new to closed.
  • resolution set to fixed.

fixed in [10773].