203160 - Link to mod_throttle has changed

Status: RESOLVED FIXED

(Bugzilla :: Documentation, defect)

Description

[Martin Wulffeld]

Link to mod_throttle in installation chapter should be changed to
http://www.snert.com/Software/mod_throttle/

Comment 1

[Martin Wulffeld]

Attachment: Patch

Comment 2

[Jacob Steenhagen]

Comment on attachment 121509 [details] [diff] [review]
Patch

>-      <ulink url="http://www.snert.com/Software/Throttle/">
>+      <ulink url="http://www.snert.com/Software/mod_throttle/">
>       http://www.snert.com/Software/Throttle/</ulink>. 

This changes the link, but not what is displayed.  If you use an empty <ulink/>
tag, DocBook will use the URL as the hyperlinked text. For example:

<ulink url="http://www.bugzilla.org"/>

Will become:

<a href="http://www.bugzilla.org">http://www.bugzilla.org</a>

when compiled into the HTML form.

I've been trying to replace the form you see in that current line with this
form whenever I run into it to prevent having to change the same URL twice when
once will do :)

Comment 3

[Martin Wulffeld]

Attachment: Patch v2

Sorry for not checking my patch better. This should do it.

Should I go through the rest of the docs and fix the remaining "double" ulinks?

Comment 4

[Jacob Steenhagen]

Comment on attachment 121527 [details] [diff] [review]
Patch v2

I really hate to do this again, but I didn't look at the page initially, only
the patch.

There is one more instance of that incorrect URL in that file about 10 lines
down.

As far as fixing the other places that use a double URL instead of an empty
<ulink/> tag: sure, that'd be great (file a different bug, though).

Comment 5

[Martin Wulffeld]

Attachment: Patch v3

Fixed.

Comment 6

[Jacob Steenhagen]

Looks good :)

I'll look at bug 203314 when I have more time :)

Comment 7

[Martin Wulffeld]

Cool. Does it need sr? Otherwise feel free to commit it :)

Comment 8

[Jacob Steenhagen]

No it doesn't... I just didn't have time this morning to do it :)

Checking in installation.xml;
/cvsroot/mozilla/webtools/bugzilla/docs/xml/installation.xml,v  <-- 
installation.xml
new revision: 1.42; previous revision: 1.41
done


Checking in installation.xml;
/cvsroot/mozilla/webtools/bugzilla/docs/xml/installation.xml,v  <-- 
installation.xml
new revision: 1.18.2.11; previous revision: 1.18.2.10
done