107
Quote:
Mikhail Miguel wrote:
hum... there some problems with pmlite.php and readmsg.php (without ".XOOPS_URL."/ or something)
Hey,
I found a quickfix for this error, but
I don't like quickfixes. When I started coding ShortUrls, I didn't know Urls like
http://www.domain.com/readpmsg.php?start=0&total_messages=1&PHPSESSID=4500f8ec8e0e17a704b4533aab2cb5e3
could exists. What I mean is, urls with scripts running from the Xoop root dir and with more than 2 parameters.
I think the best way to go is to once more slightly modify the way ShortUrls works in the following way:
1)
URLs like:/modules/newbb/viewtopic.php?topic_id=1&forum=1&post_id=1#forumpost1
Will become:m-newbb-viewtopic+topic_id-1-forum-1-post_id-1.html#forumpost1
(note the '+' (plus) sign separating files from parameters, and that modules folder is now referred as 'm')
2)
URLs in the root, as /readpmsg.php?start=0&total_messages=1&PHPSESSID=4500f8ec8e0e17a704b4533aab2cb5e3
will become:/readpmsg+start-0-total_messages-1-PHPSESSID-4500f8ec8e0e17a704b4533aab2cb5e3.html
I would like to know the opinion of each one of you who have been testing ShortURLs and helping me debug it. I think this change is necessary to allow ShortURLs have greater compatibility with current version of XOOPS and with a newer version whenever it is released.
