Project

General

Profile

Actions

Defect #21375

closed

Working external URL prefixes (protocols and 'www' host part) not documented in wiki syntax

Added by Till Riedel over 8 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Category:
Documentation
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

As #2953 states /https?:\/\/.*/ and /ftps?:\/\/.*/ links work however documentation on RedmineTextFormatting mentions only http not https,ftp,ftps. Implementation is inconstent with documentations if more than only http works, one would expect other URL schemes to work too as mentioned in #2953.


Files

21375-external-urls-help.diff (59 KB) 21375-external-urls-help.diff patch to update wiki_syntax_detailed_*.html Go MAEDA, 2017-03-21 04:24
21375_more_external_urls.patch (28.1 KB) 21375_more_external_urls.patch Marius BĂLTEANU, 2017-06-17 13:54
0001-Complement-external-URL-documentation-21375.patch (69.6 KB) 0001-Complement-external-URL-documentation-21375.patch Mischa The Evil, 2017-06-18 13:17

Related issues

Blocks Redmine - Patch #26188: Documentation (detailed syntax help & code) additions/improvementsClosedJean-Philippe Lang

Actions
Actions #1

Updated by Toshi MARUYAMA over 8 years ago

  • Description updated (diff)
Actions #2

Updated by Toshi MARUYAMA over 8 years ago

Till Riedel wrote:

As #2953 states /https?:\/\/.*/ and /ftps?:\/\/.*/ links work however documentation on RedmineTextFormatting mentions only http not https,ftp,ftps.

You can update wikis on redmine.org.
This is definition.
source:tags/3.2.0/lib/redmine/wiki_formatting.rb#L113

Implementation is inconstent with documentations if more than only http works, one would expect other URL schemes to work too as mentioned in #2953.

I think #2953 is not related.

Actions #4

Updated by Go MAEDA about 7 years ago

This patch updates public/help/*/wiki_syntax_detailed_*.html.

Actions #5

Updated by Go MAEDA about 7 years ago

  • Target version changed from Candidate for next major release to 3.4.0
Actions #6

Updated by Jean-Philippe Lang about 7 years ago

  • Subject changed from Working External URLs not documented to Working external URL protocols not documented in wiki syntax
  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang
  • Resolution set to Fixed

Committed, thanks.

Actions #7

Updated by Mischa The Evil about 7 years ago

  • Status changed from Closed to Reopened
Shouldn't we document the following ones explicitly too?
Actions #8

Updated by Jean-Philippe Lang almost 7 years ago

  • Assignee changed from Jean-Philippe Lang to Mischa The Evil

Mischa The Evil wrote:

Shouldn't we document the following ones explicitly too?

Sure, could you prepare a patch for that?

Actions #9

Updated by Mischa The Evil almost 7 years ago

Jean-Philippe Lang wrote:

Mischa The Evil wrote:

Shouldn't we document the following ones explicitly too?

Sure, could you prepare a patch for that?

Thanks for the feedback. I'll see what I can do (including the wiki page).

Actions #10

Updated by Marius BĂLTEANU almost 7 years ago

Because I really want to see the 3.4.0 released, I took the liberty to create a patch that specify the www, sftp and sftps as URLs. Mischa, the patch matches your expectations? If yes, I'll update the wiki pages, too.

Actions #11

Updated by Mischa The Evil almost 7 years ago

Marius BALTEANU wrote:

[...] I took the liberty to create a patch that specify the www, sftp and sftps as URLs. Mischa, the patch matches your expectations? If yes, I'll update the wiki pages, too.

Thanks for chiming in. I've looked at your patch; it seems — with 43 changes — incomplete however1. It seems to be generated with a (too) basic search-and-replace, using an insufficient search string. As such only changing (a part of the) the textile docs only.

I just completed an other patch which changes all the translations, slightly differently:

hits old string new string
54 (http, https, ftp, and ftps) (starting with: www, http, https, ftp, ftps, sftp and sftps)
42 (http, https, ftp and ftps)
2 (http, https, ftp, ftps)
Notes:
  • I changed the order in which the link 'types' are mentioned, starting with 'www' as it's probably the most common nowadays (although even that URL-part is being ditched often these days) and it is a hostname identifier instead of a protocol identifier as all the others (informally) are;
  • I added starting with: in front of the inline list for better clarity;
  • I decided not to add a serial comma to the inline list, as such adhering to British English spelling instead of American English (it's not required for clarity, there is no clear, official Redmine guideline on what spelling to use and it feels just more natural to me [and probably many other EU-based users, though I'm biased as my native Dutch language doesn't dictate use of it]).

I'll attach the patch implementing this change. This patch, against current source:/trunk@16676, is produced using git format-patch which makes it apply-able using "patch -p1 < 0001-...". It is patch 1/5 of a patch serial of which I'll post patches 2 to 5 in a new issue as they are less related to this initial issue.
I'll also change the wiki pages accordingly when the patch has been committed.

Environment:

Environment:
  Redmine version                3.3.3.devel@r16676
  Ruby version                   2.3.3-p222 (2016-11-21) [x86_64-linux]
  Rails version                  4.2.8
  Environment                    production
  Database adapter               Mysql2
SCM:
  Subversion                     1.8.8
  Git                            1.9.1
  Filesystem                     
Redmine plugins:
  no plugin installed

1 Redmine currently supports 49 languages, each having two detailed syntax pages (textile and markdown), which makes a required number of changes of 98.

Actions #12

Updated by Mischa The Evil almost 7 years ago

  • Blocks Patch #26188: Documentation (detailed syntax help & code) additions/improvements added
Actions #13

Updated by Jean-Philippe Lang almost 7 years ago

  • Status changed from Reopened to Closed

Committed, thanks.

Actions

Also available in: Atom PDF