#580
  • Created on
    2010-07-09 18:39
  • Reported by
    dchenbecker
  • Status
    Fixed
  • Priority
    Normal (3)
  • Assigned to
    dchenbecker
  • Milestone
    2.1-RC1
  • Plan Level
    -
  • Due Date
    -

Attachments

Related Tickets

Add new subtask or bug

LiftSession.processSnippet should handle StatefulSnippet instances returned via LiftRules.snippetDispatch

Given a StatefulSnippet class Foo, the following code returns a new instance every time because we don't properly locate current registered instances:


LiftRules.snippetDispatch.append {
  case "TestHello" => new TestHello
}


See http://groups.google.com/group/liftweb/browse_thread/thread/45feeef36b91ee9b for details
User picture
  on 2010-07-30 14:40 *
By dchenbecker
Milestone changed from 2.1-M1 to 2.1-M2
User picture
  on 2010-08-19 19:49 *
By dchenbecker
Initial work checked in on dcb-issue-580. Asking DPP for review to make sure this looks OK.
User picture
  on 2010-08-24 20:47 *
By dchenbecker
Status changed from Accepted to Test
User picture
  on 2010-08-24 21:55 *
By Anonymous
Status changed from Test to Fixed
(In revision:d7b1c1205bc255a4e1c83a8098c12b13cad5e169) Fixed snippetDispatch to work with StatefulSnippets

Closes #580

Reorganized the resolution of snippet instances to properly
handle the case where snippetDispatch is returning a new
StatefulSnippet instance.

Branch: master