New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #16672 (closed defect: fixed)

Opened 5 years ago

Last modified 4 years ago

portfile links are bad

Reported by: mark@… Owned by: ryandesign@…
Priority: Normal Milestone:
Component: website Version:
Keywords: Cc: yanjingfeng@…, jmpp@…, blb@…
Port:

Description

from the "available ports" link which gives you http://www.macports.org/ports.php

search for a port ... mysql5 (for instance, but anything will do)

On the query results, there are one or more hits, for instance mysql5 5.0.67 with a the "mysql5" being a link to http://trac.macports.org/browser/trunk/dports/databases%2Fmysql5/Portfile

However, this link is bad, giving an error page The requested URL /browser/trunk/dports/databases/mysql5/Portfile was not found on this server.

This happens for any/all Portfile links from Query results.

The link should be (in this example) http://trac.macports.org/browser/trunk/dports/databases/mysql5/Portfile

(using FireFox 3.03 on XPsp3)

Change History

comment:1 Changed 5 years ago by blb@…

  • Owner changed from jmpp@… to ryandesign@…

ryandesign@ appears to have added a urlencode() call to the result, which is causing this %2F escape of the slash (in r 37825, line 167). Assigning to him to find out if that's right or not...

comment:2 Changed 5 years ago by blb@…

hmm, it spaced my changeset link...that r37825.

comment:3 Changed 5 years ago by blb@…

  • Cc yanjingfeng@… added

Cc reporter of dup #16703.

comment:4 Changed 5 years ago by ryandesign@…

  • Cc jmpp@…, blb@… added
  • Status changed from new to closed
  • Resolution set to fixed

I appear not to have realized at the time that "$row['path']" contained a slash, in the form of "<category>/<port>". I added the escaping out of the habit of escaping everything. In this case, established port and category naming rules hopefully will prevent the need for having this escaping. Juan removed it in r40385.

comment:5 Changed 4 years ago by anonymous

  • Milestone Website & Documentation deleted

Milestone Website & Documentation deleted

Note: See TracTickets for help on using tickets.