Roundup Tracker - Issues

Issue 790326

classification
When chaning to 0.6.0 need to remove ? from search URL
Type: Severity: normal
Components: Installation Versions:
process
Status: closed fixed
:
: jlgijsbers : jlgijsbers, richard, rouilj
Priority: normal :

Created on 2003-08-18 02:01 by rouilj, last changed 2003-08-26 00:26 by richard.

Messages
msg914 Author: [hidden] (rouilj) Date: 2003-08-18 02:01
The format of the url field in the query hyperdb item
changed in 0.6.0 from earlier versions. 0.5.x at least starts
the url field with a '?'. This is no longer supported with the
new search code.

Upgraders need to remove the ? from the front of the query url's
otherwise they won't be properly edited.

Also if people have changed the page.html page, they will need
to make sure to put the ? into the code that iterates over
the user's queries otherwise they will get invalid URL's.

-- rouilj
msg915 Author: [hidden] (richard) Date: 2003-08-18 02:11
Logged In: YES 
user_id=6405

*sigh*, sorry, I missed that bit in the checkins, or I would've had the 
change reversed. This is what comes of me not actually *using* the 
feature :( 
 
Do you feel it's worthwhile reversing the change, or simply including 
instructions regarding how to update the existing queries? 
 
msg916 Author: [hidden] (jlgijsbers) Date: 2003-08-25 14:50
Logged In: YES 
user_id=469548

Reversing the change would probably not be a good idea. I
made this change (and I apologize for not mentioning it in
upgrade.txt) because with the '?' stuck on the first form
element would also have a '?' stuck in front of it (e.g.
'?:columns' instead of ':columns'). This would make us miss
one of the arguments (usually ':columns').

Anyway, I have fixes ready for upgrading.txt and page.html.
If you don't want to reverse the change, I'll check them in
to both maint-0.6 and HEAD.
msg917 Author: [hidden] (richard) Date: 2003-08-26 00:26
Logged In: YES 
user_id=6405

Go for it. 
 
History
Date User Action Args
2003-08-18 02:01:27rouiljcreate