Hey Hynekcer, I like your updates and incorporated them. I also removed
the keywords = filter(None, keywords) as I think it is redundant and
unnecessary, I'm not totally sure on that because I'm not exactly sure
what purpose it was serving there. If someone could enlighten me or
confirm that it is unnecessary I would appreciate it. Search continues
to function without it.
-Josh
On 1/9/11 9:49 AM, hynekcer wrote:
Thank you, Josh
I wrote shorter solution inspired by you, but with only 3 added lines.
(DRY principle - don't repeate yourself).
My solution is on this fork
https://bitbucket.org/hynekcer/satchmo-outgoing/changeset/c4d0d356acd0
I liked to make your fix more readable, because I am working on more
unpublished customizations.
--
Josh, now it would be useful to display searched keywords back in the
search textbox. There is nowhere displayed what was searched
eventually not found to see typo. I have many problems with it,
because I tried to add parameters "keywords" and "category" to
templatetags "satchmo_search_form" and "satchmo_category_search_form".
This would require complicated nested if/else templatetags conditions
around it in base.html temlate, which is not good.
Got any idea?
Hynek
On 8 led, 22:26, Josh<[email protected]> wrote:
I forked satchmo on bitbucket and sent a pull request with the changes
made. I'm not sure if that was the best way to go about it but I
figured that way if I make any changes in the future it will be easy
enough to submit a pull.
I've been using my modified version and so far it works perfect but if
anyone else wants to give it a try that would be great.
-Josh
--
You received this message because you are subscribed to the Google Groups "Satchmo
users" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/satchmo-users?hl=en.
