6.0.0-beta1
7/6/25

[#10610] Error when click on link for a page that doesn't exist
Summary Error when click on link for a page that doesn't exist
Queue Wicked
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org, mrubinsk (at) horde (dot) org
Requester michael (at) bigmichi1 (dot) de
Created 10/04/2011 (5024 days ago)
Due
Updated 10/11/2011 (5017 days ago)
Assigned 10/07/2011 (5021 days ago)
Resolved 10/11/2011 (5017 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
10/11/2011 08:10:24 AM Jan Schneider State ⇒ Resolved
 
10/11/2011 12:18:36 AM Git Commit Comment #14 Reply to this comment
Changes have been made in Git for this ticket:

Wicked page names can also contain the forward slash.
Bug: 10610

  1 files changed, 2 insertions(+), 2 deletions(-)
http://git.horde.org/horde-git/-/commit/dead4aa4ad14666d32d522d72572410e187eb170
10/10/2011 04:10:17 PM Git Commit Comment #13 Reply to this comment
Changes have been made in Git for this ticket:

Fix quoting (Bug #10610).

  1 files changed, 3 insertions(+), 3 deletions(-)
http://git.horde.org/horde-git/-/commit/b3f75ebe1882d6e097420e8a9395b54a0fdddb3f
10/10/2011 09:57:00 AM Jan Schneider Deleted Original Message
 
10/10/2011 09:56:50 AM Git Commit Comment #12 Reply to this comment
Changes have been made in Git for this ticket:

Fix Horde_Db's buildClause() usage.
Don't use it where we explicitly want to match only the begin or end of
strings. Don't use double placeholders where we use it correctly.

Bug: 10610

  2 files changed, 5 insertions(+), 8 deletions(-)
http://git.horde.org/horde-git/-/commit/3c57b237a1220a3d88125c0356db0c014a64cb0b
10/10/2011 09:56:47 AM Git Commit Comment #11 Reply to this comment
Changes have been made in Git for this ticket:

[jan] Correctly escape binding characters in buildClause() (Bug #10610).

  8 files changed, 297 insertions(+), 27 deletions(-)
http://git.horde.org/horde-git/-/commit/5f3cbc1cf248033bccd3cb67b54f32b99ec27a47
10/10/2011 09:56:36 AM Jan Schneider Comment #10
Assigned to Jan Schneider
Assigned to Michael Rubinsky
Reply to this comment
Try these fixes.

Michael needs to take care of the lighttpd rules.
10/10/2011 05:23:52 AM michael (at) bigmichi1 (dot) de Comment #9
New Attachment: horde_debug.txt Download
Reply to this comment
sorry for the wrong file. here is the correct one
10/09/2011 07:55:06 PM Jan Schneider Comment #8 Reply to this comment
This is horde.log, not horde_debug.txt.
10/09/2011 03:47:24 PM michael (at) bigmichi1 (dot) de Comment #7
New Attachment: horde[1].log
Reply to this comment
added the line to Base.php and did the same like i did for the other log
10/09/2011 02:06:41 PM Jan Schneider Comment #6 Reply to this comment
Ah, what I missed earlier is that there is indeed some parameter 
placeholder in the query: the url's question mark. So there are two 
problems: we don't escape placeholders there, and the passed pages 
name is incorrect. The latter is probably a problem with the rewriting 
rules. But for now we can fix the first issue.
Can you add this code at line 810 of Horde/Db/Adapter/Base.php:
Horde::debug($error);
and upload the content of /tmp/horde_debug.txt?
10/09/2011 01:46:25 PM michael (at) bigmichi1 (dot) de Comment #5
New Attachment: horde.log Download
Reply to this comment
this is the entire log file from beginning clicking on the link till 
the error message appears. user test created the page with the linkand 
user xxx tries to add the subpage for the link
10/09/2011 01:02:00 PM Jan Schneider Comment #4 Reply to this comment
The debug log should at least reveal more detailed queries to the SQL 
backend though.
10/09/2011 12:09:03 PM michael (at) bigmichi1 (dot) de Comment #3 Reply to this comment
i have set up logging to DEBUG level but i ndidn't get any backtrace 
or something more. i personally think there is a problem with the 
provided rewrite rules, because if i change rewriting in horde no none 
everything works like expected
10/07/2011 08:07:30 PM Jan Schneider Comment #2
State ⇒ Feedback
Reply to this comment
Cannot reproduce, and I can't see how this error in the log could 
happen. Try to get a complete backtrace for that error.
10/04/2011 05:00:33 PM michael (at) bigmichi1 (dot) de Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Error when click on link for a page that doesn't exist
Queue ⇒ Wicked
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
i set up a new wiki with this content:

== Anleitungen ==
* [[aendern des Passwortes | ändern des Passwortes]]

when i now click on the link i got the message:
Interner Fehler beim Anzeigen dieser Seite

i'm using the lighttpd rules which are provided by the wicked package 
and also enabled mod_rewrite in horde config

in my horde.log file i found this entry:

2011-10-04T18:52:37+02:00 ERR: HORDE [wicked] SQL 
Horde_Db_Adapter_Base::_replaceParameters
         Parameter count mismatch: SELECT * FROM wicked_pages WHERE
           LOWER(page_name) LIKE
           LOWER('%aendern_des_Passwortes?referrer=Wiki/Home%') [pid 
4266 on line 810 of "/var/www/pear/php/Horde/Db/Adapter/Base.php"]

Saved Queries