6.0.0-beta1
7/5/25

[#10636] wrong url in report abuse link
Summary wrong url in report abuse link
Queue Ansel
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org
Requester thpo+horde (at) dotrc (dot) de
Created 10/12/2011 (5015 days ago)
Due
Updated 10/12/2011 (5015 days ago)
Assigned
Resolved 10/12/2011 (5015 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
10/12/2011 10:27:59 AM Jan Schneider Comment #3
Assigned to Jan Schneider
State ⇒ Resolved
Reply to this comment
Horde_Url is correct.
10/12/2011 10:27:54 AM Git Commit Comment #2 Reply to this comment
Changes have been made in Git for this ticket:

[jan] Fix link to report faces (Bug #10636).

  3 files changed, 4 insertions(+), 3 deletions(-)
http://git.horde.org/horde-git/-/commit/b79848d7d1712a06c9da47b2615d608dc9ad9dba
10/12/2011 08:17:32 AM thpo+horde (at) dotrc (dot) de Comment #1
Priority ⇒ 1. Low
Patch ⇒ No
Milestone ⇒
Queue ⇒ Ansel
Summary ⇒ wrong url in report abuse link
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
The "Report abuse" link in ansel/faces/search/image_search.php
is broken:
https://host/ansel/faces/report.php?0=face&1=1

Can be fixed by removing the following patch (although it looks like a 
bug in Horde_Url::add()):

--- a/ansel/templates/faces/search.inc
+++ b/ansel/templates/faces/search.inc
@@ -10,7 +10,7 @@
       <br />
       <img src="<?php echo $faces->getFaceUrl($face['image_id'], 
$face_id) ?>" style="padding: 5px;" alt="<?php echo $facename?>" /><br 
/>
       <?php echo sprintf(ngettext("%d photo", "%d photos", 
count($results)), count($results)); ?><br />
-     <?php echo Horde::url('faces/report.php')->add(array('face', 
$face_id))->link(
+     <?php echo Horde::url('faces/report.php')->add('face', $face_id)->link(
               array('class' => 'small',
                     'title' => _("Report abuse or submit a removal 
request"))) . _("Report") . '</a>'?>
     <?php else: ?>

Saved Queries