Lookup should have item identifier in parameter

[GoogleCodeExporter]

Lookup an item.  It should redirect to a page where the identifier is specified 
(item/lookup/12345678 where 12345678 is an identifier).

Original issue reported on code.google.com by donovanh...@gmail.com on 27 May 2011 at 12:02

[GoogleCodeExporter]

This issue was closed by revision ec1ac3b559a3.

Original comment by donovanh...@gmail.com on 27 May 2011 at 3:02

• Changed state: Fixed

[GoogleCodeExporter]

I couldn't get /items/lookup/1234ADBH to work, so I settled on using 
/items/lookup?q=1234ADBH (I figured time was better spent elsewhere).

Could you verify?

Original comment by donovanh...@gmail.com on 27 May 2011 at 3:03

[GoogleCodeExporter]

I'm getting a plus at the end?

Example:
http://localhost:3000/items/lookup?q=SDmcESO9t+

Original comment by liu.al...@gmail.com on 27 May 2011 at 4:15

• Changed state: Started

[GoogleCodeExporter]

It looks like the "+" is caused by adding a trailing space in the text box 
(putting "12345678" has no "+" but putting "12345678 " adds a "+").

Do you agree with this conclusion?  Do you think we should fix it?

Original comment by donovanh...@gmail.com on 27 May 2011 at 8:39

• Changed state: Fixed

[GoogleCodeExporter]

Yup, looks like thats what's causing it. I think leaving it is fine. Or you 
could use .chomp! or .trim! if you want, one of those might remove trailing 
whitespace, I forget.

Original comment by liu.al...@gmail.com on 27 May 2011 at 4:09

[GoogleCodeExporter]

Original comment by liu.al...@gmail.com on 27 May 2011 at 5:47

• Changed state: Verified