:trac: links are broken

[ppurka]

Two issues:

1. The use of :trac:`` is broken on the command line. The second link below shouldn't repeat the ticket number of the first one

sage: integral?
<removed earlier text>
       TESTS:

       A symbolic integral from http://trac.sagemath.org/11445 that was
       incorrect in earlier versions of Maxima:

          sage: f = abs(x - 1) + abs(x + 1) - 2*abs(x)
          sage: integrate(f, (x, -Infinity, Infinity))
          2

       Another symbolic integral, from http://trac.sagemath.org/11445,
       that used to return zero incorrectly:

          sage: f = exp(-x) * sinh(sqrt(x))
          sage: integrate(f, x, 0, Infinity)
          1/2*sqrt(pi)*e^(1/4)

2. reported upstream. The link in the notebook is actually correct (that is I get the correct ticket number in the second URL), but it should open the documentation in a new tab. Otherwise, there is a huge potential of someone losing all their work just because they forgot to shift-click or something.

#13116 fixed trac tickets in docstrings for command line, but I think it wasn't fixed properly. The problem exists at least as far back as sage-5.1.rc0.

---

Apply attachment: trac_13310-trac.patch

Component: documentation

Keywords: trac keyword

Author: John Palmieri

Reviewer: Punarbasu Purkayastha

Merged: sage-5.3.beta1

Issue created by migration from https://trac.sagemath.org/ticket/13310

[jhpalmieri]

comment:1

I think that these are two completely separate issue, and deserve completely separate tickets. In fact, issue 2 is really an issue with the notebook, so should be dealt with upstream. I'm attaching a patch for issue 1.

[jhpalmieri]

Author: John Palmieri

[jhpalmieri]

comment:2

(By the way, I think that the sagenb issue could be fixed by patching the "sphinxify" function, but it should be done at the sagenb github site, not here.)

[ppurka]

Reviewer: Punarbasu Purkayastha

[ppurka]

comment:3

This works. Thanks. Issue 2 is reported upstream.

[ppurka]

comment:4

Hmm... the patch does not end with the correct extension, hopefully it is not a problem.

patchbot apply trac_13310-trac.pach

[jhpalmieri]

comment:5

It shouldn't be a problem, but just in case...

[jdemeyer]

comment:6

Attachment: trac_13310-trac.patch.gz

[jdemeyer]

Merged: sage-5.3.beta1