Cache results of parsing URI parameters #982

Open
szeiger opened this Issue Apr 21, 2011 · 6 comments

Comments

Projects
None yet
5 participants
@szeiger

szeiger commented Apr 21, 2011

Menu.param allows you to define a parser and an encoder for a URI parameter. The parser is a good place to retrieve an item from a database and automatically get a 404 if it is not found, but the parsing is performed multiple times per request and the result is not cached. As a workaround, you can manually thread the result through a RequestVar. It would be nice to have a caching Menu.param available in Lift out of the box.

@indrajitr

This comment has been minimized.

Show comment
Hide comment
@indrajitr

indrajitr Jul 21, 2011

Member

Updating tickets (#919, #938, #950, #956, #976, #980, #982, #999, #1008, #1024, #1025, #1032, #1034, #1051)

Member

indrajitr commented Jul 21, 2011

Updating tickets (#919, #938, #950, #956, #976, #980, #982, #999, #1008, #1024, #1025, #1032, #1034, #1051)

@indrajitr

This comment has been minimized.

Show comment
Hide comment
@indrajitr

indrajitr Sep 9, 2011

Member

Updating tickets (#950, #956, #976, #980, #982, #999, #1008, #1053, #1078, #1092, #1097)

Pushed pending tickets to 2.4-M5

Member

indrajitr commented Sep 9, 2011

Updating tickets (#950, #956, #976, #980, #982, #999, #1008, #1053, #1078, #1092, #1097)

Pushed pending tickets to 2.4-M5

@indrajitr

This comment has been minimized.

Show comment
Hide comment
@indrajitr

indrajitr Nov 11, 2011

Member

Updating tickets (#956, #976, #980, #982, #1008, #1053, #1078, #1092, #1097, #1128)

Member

indrajitr commented Nov 11, 2011

Updating tickets (#956, #976, #980, #982, #1008, #1053, #1078, #1092, #1097, #1128)

@indrajitr

This comment has been minimized.

Show comment
Hide comment
@indrajitr

indrajitr Dec 22, 2011

Member
Member

indrajitr commented Dec 22, 2011

@github-importer

This comment has been minimized.

Show comment
Hide comment

@ghost ghost assigned dpp Mar 1, 2012

This was referenced Mar 1, 2012

@Shadowfiend

This comment has been minimized.

Show comment
Hide comment
@Shadowfiend

Shadowfiend May 11, 2014

Member

This actually came up again on the list the other day—is the idea here that we would stick the results of the parse in a TransientRequestVar?

Member

Shadowfiend commented May 11, 2014

This actually came up again on the list the other day—is the idea here that we would stick the results of the parse in a TransientRequestVar?

@Shadowfiend Shadowfiend assigned Shadowfiend and unassigned dpp May 11, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment