jro_apps/redmine Commit - r1252:93d1b2e0a439

Add Redmine links for repository files using source: and export: keyworkds (#867)....

Jean-Philippe Lang -

r1252:93d1b2e0a439

parent child

Context file:

r1252:93d1b2e0a439

Collapse all files

app/controllers/repositories_controller.rb +2 -1

                def entry
                  @content = @repository.scm.cat(@path, @rev)
                  show_error_not_found and return unless @content
-                 if 'raw' == params[:format]
+                 if 'raw' == params[:format] || @content.is_binary_data?
+                   # Force the download if it's a binary file
                    send_data @content, :filename => @path.split('/').last
                  else
                    # Prevent empty lines when displaying a file with Windows style eol

app/helpers/application_helper.rb +19 -3

                  #     version:"1.0 beta 2" -> Link to version named "1.0 beta 2"
                  #   Attachments:
                  #     attachment:file.zip -> Link to the attachment of the current object named file.zip
-                 text = text.gsub(%r{([\s\(,-^])(!)?(attachment|document|version|commit)?((#|r)(\d+)|(:)([^"][^\s<>]+|"[^"]+"))(?=[[:punct:]]|\s|<|$)}) do |m|
+                 #   Source files:
+                 #     source:some/file -> Link to the file located at /some/file in the project's repository
+                 #     source:some/file@52 -> Link to the file's revision 52
+                 #     source:some/file#L120 -> Link to line 120 of the file
+                 #     source:some/file@52#L120 -> Link to line 120 of the file's revision 52
+                 #     export:some/file -> Force the download of the file
+                 text = text.gsub(%r{([\s\(,-^])(!)?(attachment|document|version|commit|source|export)?((#|r)(\d+)|(:)([^"][^\s<>]+|"[^"]+"))(?=[[:punct:]]|\s|<|$)}) do |m|
                    leading, esc, prefix, sep, oid = $1, $2, $3, $5 || $7, $6 || $8
                    link = nil
                    if esc.nil?
                      if prefix.nil? && sep == 'r'
                        if project && (changeset = project.changesets.find_by_revision(oid))
-                         link = link_to("r#{oid}", {:only_path => only_path, :controller => 'repositories', :action => 'revision', :id => project.id, :rev => oid},
+                         link = link_to("r#{oid}", {:only_path => only_path, :controller => 'repositories', :action => 'revision', :id => project, :rev => oid},
                                                    :class => 'changeset',
                                                    :title => truncate(changeset.comments, 100))
                        end
                          end
                        when 'commit'
                          if project && (changeset = project.changesets.find(:first, :conditions => ["scmid LIKE ?", "#{name}%"]))
-                           link = link_to h("#{name}"), {:only_path => only_path, :controller => 'repositories', :action => 'revision', :id => project.id, :rev => changeset.revision}, :class => 'changeset', :title => truncate(changeset.comments, 100)
+                           link = link_to h("#{name}"), {:only_path => only_path, :controller => 'repositories', :action => 'revision', :id => project, :rev => changeset.revision}, :class => 'changeset', :title => truncate(changeset.comments, 100)
+                         end
+                       when 'source', 'export'
+                         if project && project.repository
+                           name =~ %r{^[/\\]*(.*?)(@([0-9a-f]+))?(#(L\d+))?$}
+                           path, rev, anchor = $1, $3, $5
+                           link = link_to h("#{prefix}:#{name}"), {:controller => 'repositories', :action => 'entry', :id => project, :path => path,
+                                                                   :rev => rev,
+                                                                   :anchor => anchor,
+                                                                   :format => (prefix == 'export' ? 'raw' : nil)},
+                                                                  :class => (prefix == 'export' ? 'source download' : 'source')
                          end
                        when 'attachment'
                          if attachments && attachment = attachments.detect {|a| a.filename == name }

app/views/repositories/entry.rhtml +7 -10

              <div class="autoscroll">
              <table class="filecontent CodeRay">
-               <tbody>
-                 <% line_num = 1 %>
-                 <% syntax_highlight(@path, to_utf8(@content)).each_line do |line| %>
-                 <tr>
-                   <th class="line-num"><%= line_num %></th>
-                   <td class="line-code"><pre><%= line %></pre></td>
-                 </tr>
-                 <% line_num += 1 %>
-                 <% end %>
-               </tbody>
+             <tbody>
+             <% line_num = 1 %>
+             <% syntax_highlight(@path, to_utf8(@content)).each_line do |line| %>
+             <tr><th class="line-num" id="L<%= line_num %>"><%= line_num %></th><td class="line-code"><pre><%= line %></pre></td></tr>
+             <% line_num += 1 %>
+             <% end %>
+             </tbody>
              </table>
              </div>

test/unit/helpers/application_helper_test.rb +28 -15

                  issue_link = link_to('#3', {:controller => 'issues', :action => 'show', :id => 3},
                                             :class => 'issue', :title => 'Error 281 when updating a recipe (New)')
-                 changeset_link = link_to('r1', {:controller => 'repositories', :action => 'revision', :id => 1, :rev => 1},
+                 changeset_link = link_to('r1', {:controller => 'repositories', :action => 'revision', :id => 'ecookbook', :rev => 1},
                                                 :class => 'changeset', :title => 'My very first commit')
                  document_link = link_to('Test document', {:controller => 'documents', :action => 'show', :id => 1},
                  version_link = link_to('1.0', {:controller => 'versions', :action => 'show', :id => 2},
                                                :class => 'version')
+                 source_url = {:controller => 'repositories', :action => 'entry', :id => 'ecookbook', :path => 'some/file'}
                  to_test = {
-                   '#3, #3 and #3.' => "#{issue_link}, #{issue_link} and #{issue_link}.",
-                   'r1' => changeset_link,
-                   'document#1' => document_link,
-                   'document:"Test document"' => document_link,
-                   'version#2' => version_link,
-                   'version:1.0' => version_link,
-                   'version:"1.0"' => version_link,
+                   # tickets
+                   '#3, #3 and #3.'              => "#{issue_link}, #{issue_link} and #{issue_link}.",
+                   # changesets
+                   'r1'                          => changeset_link,
+                   # documents
+                   'document#1'                  => document_link,
+                   'document:"Test document"'    => document_link,
+                   # versions
+                   'version#2'                   => version_link,
+                   'version:1.0'                 => version_link,
+                   'version:"1.0"'               => version_link,
+                   # source
+                   'source:/some/file'           => link_to('source:/some/file', source_url, :class => 'source'),
+                   'source:/some/file@52'        => link_to('source:/some/file@52', source_url.merge(:rev => 52), :class => 'source'),
+                   'source:/some/file#L110'      => link_to('source:/some/file#L110', source_url.merge(:anchor => 'L110'), :class => 'source'),
+                   'source:/some/file@52#L110'   => link_to('source:/some/file@52#L110', source_url.merge(:rev => 52, :anchor => 'L110'), :class => 'source'),
+                   'export:/some/file'           => link_to('export:/some/file', source_url.merge(:format => 'raw'), :class => 'source download'),
                    # escaping
-                   '!#3.' => '#3.',
-                   '!r1' => 'r1',
-                   '!document#1' => 'document#1',
-                   '!document:"Test document"' => 'document:"Test document"',
-                   '!version#2' => 'version#2',
-                   '!version:1.0' => 'version:1.0',
-                   '!version:"1.0"' => 'version:"1.0"',
+                   '!#3.'                        => '#3.',
+                   '!r1'                         => 'r1',
+                   '!document#1'                 => 'document#1',
+                   '!document:"Test document"'   => 'document:"Test document"',
+                   '!version#2'                  => 'version#2',
+                   '!version:1.0'                => 'version:1.0',
+                   '!version:"1.0"'              => 'version:"1.0"',
+                   '!source:/some/file'          => 'source:/some/file',
                  }
                  @project = Project.find(1)
                  to_test.each { |text, result| assert_equal "<p>#{result}</p>", textilizable(text) }

test/unit/mailer_test.rb +1 -1

                  # link to a referenced ticket
                  assert mail.body.include?('<a href="https://mydomain.foo/issues/show/2" class="issue" title="Add ingredients categories (Assigned)">#2</a>')
                  # link to a changeset
-                 assert mail.body.include?('<a href="https://mydomain.foo/repositories/revision/1?rev=2" class="changeset" title="This commit fixes #1, #2 and references #1 &amp; #3">r2</a>')
+                 assert mail.body.include?('<a href="https://mydomain.foo/repositories/revision/ecookbook?rev=2" class="changeset" title="This commit fixes #1, #2 and references #1 &amp; #3">r2</a>')
                end
                # test mailer methods for each language

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages