Vote #76949
完了Working external URL prefixes (protocols and 'www' host part) not documented in wiki syntax
0%
説明
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.
journals
--------------------------------------------------------------------------------
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.
--------------------------------------------------------------------------------
I have updated [[RedmineTextFormattingTextile]] ("diff":/projects/redmine/wiki/RedmineTextFormattingTextile/diff?utf8=%E2%9C%93&version=7&version_from=6&commit=View+differences) and [[RedmineTextFormattingMarkdown]] ("diff":/projects/redmine/wiki/RedmineTextFormattingMarkdown/diff?utf8=%E2%9C%93&version=21&version_from=20&commit=View+differences).
--------------------------------------------------------------------------------
This patch updates public/help/*/wiki_syntax_detailed_*.html.
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
Committed, thanks.
--------------------------------------------------------------------------------
Shouldn't we document the following ones explicitly too?
* www.example.com
* sftp://sftp.example.com
* sftps://sftps.example.com
--------------------------------------------------------------------------------
Mischa The Evil wrote:
> Shouldn't we document the following ones explicitly too?
Sure, could you prepare a patch for that?
--------------------------------------------------------------------------------
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).
--------------------------------------------------------------------------------
Because I really want to see the version: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.
--------------------------------------------------------------------------------
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 however[1]. 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)@ |/3. @(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":https://en.wikipedia.org/wiki/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:
<pre>
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
</pre>
fn1. Redmine currently supports 49 languages, each having two detailed syntax pages (textile and markdown), which makes a required number of changes of 98.
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
Committed, thanks.
--------------------------------------------------------------------------------
related_issues
blocks,Closed,26188,Documentation (detailed syntax help & code) additions/improvements