[Bb-trac] Re: [bbPress] #785: bbpress does not deal well with a title of just a comma

bbPress bb-trac at lists.bbpress.org
Sat Feb 16 00:51:28 GMT 2008


#785: bbpress does not deal well with a title of just a comma
----------------------+-----------------------------------------------------
 Reporter:  _ck_      |        Owner:     
     Type:  defect    |       Status:  new
 Priority:  normal    |    Milestone:     
Component:  Back-end  |      Version:     
 Severity:  major     |   Resolution:     
 Keywords:            |  
----------------------+-----------------------------------------------------
Comment (by mdawaffe):

 The problem is that the slug was being sanitize to {{{""}}}.

 This is half fixed by [1107].

 Now when new topics are submitted that generate an empty slug, the topic
 is not created, and an unhelpful error message is displayed (This topic is
 closed).

 We can either

  a. Catch the error earlier and show a more pertinent message (Topic title
 invalid. blah blah).
  b. If the generated slug is empty, fill it with something generic like
 {{{"0-x"}}}, and create the topic.  ({{{x}}} is an incrementor to preserve
 slug uniqueness, {{{0}}} just because that's how we roll.)

 Both are easy to implement.

-- 
Ticket URL: <http://trac.bbpress.org/ticket/785#comment:2>
bbPress <http://bbpress.org/>
Innovative forum development


More information about the Bb-trac mailing list