[wp-meta] [Making WordPress.org] #447: Link to actual source from reference pages

Making WordPress.org noreply at wordpress.org
Sat Apr 26 23:58:47 UTC 2014


#447: Link to actual source from reference pages
-------------------------+-------------------------------------
Reporter:  DrewAPicture  |      Owner:
    Type:  enhancement   |     Status:  new
Priority:  normal        |  Component:  developer.wordpress.org
Keywords:  has-patch     |
-------------------------+-------------------------------------
 Currently, "source" links on reference pages link to the corresponding
 "source" term archives, and as someone has [https://github.com/rmccue/WP-
 Parser/issues/105 already suggested] for WP-Parser, we should at the very
 least, link the file and line number in trac browser.

 The attached patch renames the `get_source_file_link()` function to the
 more apt `get_source_file_archive_link()` name, which opens the door to
 using a new version of `get_source_file_link()` to actually return the
 trac browser URL with or without the line number appended.

 Two other things to note:
 * This reiterates the need to just completely skip duplicate hooks when
 parsing, as those duplicates can have a tendency to overwrite the
 documented versions of themselves.
 * In the patch I used a generic "More in this file" string as the anchor
 label for the source term archive. This is in hesitation to leveraging
 term labels due to translation issues there cited in core many times.
 Ideally, of course, I'd prefer something like "More ''hooks'' in this
 file", or "More ''functions'' in this file", etc. Might be another ticket
 for another day.

--
Ticket URL: <https://meta.trac.wordpress.org/ticket/447>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org


More information about the wp-meta mailing list