#6837 imported trac content with r123 refs still link to old host

asf_release_1.1.0
closed
Cory Johns
General
2014-11-05
2013-11-04
Dave Brondsema
No

E.g. :

Also fixed for ExecutePipeline? in [r10739](http://sourceforge.net/apps/trac/open-ms/changeset/10739).

I think we want to rewrite this to simply [r10739] or possibly [r10739](r10739) (so that brackets aren't displayed).

IIRC, there are more specific similar syntaxes too that may need to be considered.

Discussion

  • Cory Johns
    Cory Johns
    2013-11-15

    • status: open --> in-progress
    • assigned_to: Cory Johns
     
  • Dave Brondsema
    Dave Brondsema
    2013-11-15

    • Milestone: forge-nov-15 --> forge-nov-29
     
  • Cory Johns
    Cory Johns
    2013-11-15

    • Size: --> 2
     
  • Cory Johns
    Cory Johns
    2013-11-15

    • status: in-progress --> code-review
     
  • Cory Johns
    Cory Johns
    2013-11-15

    allura:cj/6837

     
  • Dave Brondsema
    Dave Brondsema
    2013-11-18

    • QA: Dave Brondsema
     
  • Dave Brondsema
    Dave Brondsema
    2013-11-18

    The linking changes look great. Would it be easy to make a change for the wiki links that have the ? appended? They look like <a class="missing wiki">RedHat?</a> on https://sourceforge.net/apps/trac/mcclidar/ticket/47 for example. The don't come through as a link (since there's no href) which is fine, but the extra ? isn't good especially since its just plain text and not styled as a possible-link any more.

     
  • Cory Johns
    Cory Johns
    2013-11-18

    That would have to be handled at a higher level than these changes, but it shouldn't be too hard with the changes from [#6836]. I'll take a look.

     

    Related

    Tickets: #6836

  • Cory Johns
    Cory Johns
    2013-11-18

    Rebased and force-pushed

     
  • Dave Brondsema
    Dave Brondsema
    2013-11-18

    • status: code-review --> closed