merb

Comments and changes to this ticket

• 

  Michael Klishin (antares) June 29th, 2008 @ 08:54 AM

  • → Tag changed from “controller merb-core views” to “controller major merb-core views”
  • → Milestone changed from “” to “0.9.4”
  • → State changed from “new” to “open”
  • → Assigned user changed from “Daniel Neighman (hassox)” to “Yehuda Katz (wycats)”
• 

  Will Prater July 6th, 2008 @ 02:27 AM

  Attached a failing spec for you guys.

  • 0001-Added-view-for-throw_content-spec.patch 2.3 KB
• 

  Will Prater July 17th, 2008 @ 01:48 AM

  Any chance on getting this fixed soon? I've added a failing spec. Thanks

• 

  Michael Klishin (antares) July 17th, 2008 @ 02:39 AM

  • → State changed from “open” to “resolved”
  • → Assigned user changed from “Yehuda Katz (wycats)” to “Michael Klishin (antares)”

  I am not very familiar with rendering stuff but your fix makes spec pass and looks like it should concat, not replace. So I applied it, should be fixed in HEAD.

  If not, blame me for applying broken patches :)

• 

  Will Prater July 17th, 2008 @ 02:45 AM

  Thanks.. this is probably a better fix

  • output.diff 610 Bytes
• 

  Michael Klishin (antares) July 17th, 2008 @ 02:54 AM

  Will,

  Line above assigns [] to content being thrown. Then you ||= it with empty string. Am I missing something? Anyway, this patch blows up about 10 spec examples in -core so I am better leave this as is :)