pull-requests: change the naming from #NUM into !NUM.

- We'll later on enable global pattern matching and from now on refercing a pr should be
used with an ! not to confuse #NUM which is tickets/issues

rhodecode/lib/vcs/conf/settings.py

@@ -56,7 +56,7 b" HOOKS_HOST = '127.0.0.1'"
 
 
 MERGE_MESSAGE_TMPL = (
-    u'Merge pull request #{pr_id} from {source_repo} {source_ref_name}\n\n '
+    u'Merge pull request !{pr_id} from {source_repo} {source_ref_name}\n\n '
     u'{pr_title}')
 MERGE_DRY_RUN_MESSAGE = 'dry_run_merge_message_from_rhodecode'
 MERGE_DRY_RUN_USER = 'Dry-Run User'

rhodecode/templates/admin/my_account/my_account_pullrequests.mako

@@ -52,11 +52,11 b''
              { data: {"_": "target_repo",
                       "sort": "target_repo"}, title: "${_('Target Repo')}", className: "td-targetrepo", orderable: false},
              { data: {"_": "name",
-                      "sort": "name_raw"}, title: "${_('Name')}", className: "td-componentname", "type": "num" },
+                      "sort": "name_raw"}, title: "${_('Id')}", className: "td-componentname", "type": "num" },
+             { data: {"_": "title",
+                      "sort": "title"}, title: "${_('Title')}", className: "td-description" },
              { data: {"_": "author",
                       "sort": "author_raw"}, title: "${_('Author')}", className: "td-user", orderable: false },
-             { data: {"_": "title",
-                      "sort": "title"}, title: "${_('Title')}", className: "td-description" },
              { data: {"_": "comments",
                       "sort": "comments_raw"}, title: "", className: "td-comments", orderable: false},
              { data: {"_": "updated_on",

rhodecode/templates/changeset/changeset_file_comment.mako

@@ -68,9 +68,9 b''
                   % if comment.pull_request:
                       <a href="${h.route_path('pullrequest_show',repo_name=comment.pull_request.target_repo.repo_name,pull_request_id=comment.pull_request.pull_request_id)}">
                           % if comment.status_change:
-                              ${_('pull request #%s') % comment.pull_request.pull_request_id}:
+                              ${_('pull request !{}').format(comment.pull_request.pull_request_id)}:
                           % else:
-                              ${_('pull request #%s') % comment.pull_request.pull_request_id}
+                              ${_('pull request !{}').format(comment.pull_request.pull_request_id)}
                           % endif
                       </a>
                   % else:

rhodecode/templates/data_table/_dt_elements.mako

@@ -371,9 +371,9 b''
 <%def name="pullrequest_name(pull_request_id, target_repo_name, short=False)">
     <a href="${h.route_path('pullrequest_show',repo_name=target_repo_name,pull_request_id=pull_request_id)}">
       % if short:
-        #${pull_request_id}
+        !${pull_request_id}
       % else:
-        ${_('Pull request #%(pr_number)s') % {'pr_number': pull_request_id,}}
+        ${_('Pull request !{}').format(pull_request_id)}
       % endif
     </a>
 </%def>

rhodecode/templates/pullrequests/pullrequest_show.mako

@@ -3,7 +3,7 b''
 <%namespace name="dt" file="/data_table/_dt_elements.mako"/>
 
 <%def name="title()">
-    ${_('%s Pull Request #%s') % (c.repo_name, c.pull_request.pull_request_id)}
+    ${_('{} Pull Request !{}').format(c.repo_name, c.pull_request.pull_request_id)}
     %if c.rhodecode_name:
         &middot; ${h.branding(c.rhodecode_name)}
     %endif
@@ -45,7 +45,7 b''
       <div class="summary-details block-left">
         <% summary = lambda n:{False:'summary-short'}.get(n) %>
         <div class="pr-details-title">
-            <a href="${h.route_path('pull_requests_global', pull_request_id=c.pull_request.pull_request_id)}">${_('Pull request #%s') % c.pull_request.pull_request_id}</a> ${_('From')} ${h.format_date(c.pull_request.created_on)}
+            <a href="${h.route_path('pull_requests_global', pull_request_id=c.pull_request.pull_request_id)}">${_('Pull request !{}').format(c.pull_request.pull_request_id)}</a> ${_('From')} ${h.format_date(c.pull_request.created_on)}
             %if c.allowed_to_update:
               <div id="delete_pullrequest" class="pull-right action_button ${'' if c.allowed_to_delete else 'disabled' }" style="clear:inherit;padding: 0">
                   % if c.allowed_to_delete:

rhodecode/templates/pullrequests/pullrequests.mako

@@ -78,11 +78,11 b''
          { data: {"_": "status",
                   "sort": "status"}, title: "", className: "td-status", orderable: false},
          { data: {"_": "name",
-                  "sort": "name_raw"}, title: "${_('Name')}", className: "td-componentname", "type": "num" },
+                  "sort": "name_raw"}, title: "${_('Id')}", className: "td-componentname", "type": "num" },
+         { data: {"_": "title",
+                  "sort": "title"}, title: "${_('Title')}", className: "td-description" },
          { data: {"_": "author",
                   "sort": "author_raw"}, title: "${_('Author')}", className: "td-user", orderable: false },
-         { data: {"_": "title",
-                  "sort": "title"}, title: "${_('Title')}", className: "td-description" },
          { data: {"_": "comments",
                   "sort": "comments_raw"}, title: "", className: "td-comments", orderable: false},
          { data: {"_": "updated_on",

rhodecode/tests/models/test_pullrequest.py

@@ -296,7 +296,7 b' class TestPullRequestModel(object):'
         Session().commit()
 
         message = (
-            u'Merge pull request #{pr_id} from {source_repo} {source_ref_name}'
+            u'Merge pull request !{pr_id} from {source_repo} {source_ref_name}'
             u'\n\n {pr_title}'.format(
                 pr_id=pull_request.pull_request_id,
                 source_repo=safe_unicode(
@@ -340,7 +340,7 b' class TestPullRequestModel(object):'
         assert pull_request.pull_request_state == PullRequest.STATE_CREATED
 
         message = (
-            u'Merge pull request #{pr_id} from {source_repo} {source_ref_name}'
+            u'Merge pull request !{pr_id} from {source_repo} {source_ref_name}'
             u'\n\n {pr_title}'.format(
                 pr_id=pull_request.pull_request_id,
                 source_repo=safe_unicode(
@@ -379,7 +379,7 b' class TestPullRequestModel(object):'
         Session().commit()
 
         message = (
-            u'Merge pull request #{pr_id} from {source_repo} {source_ref_name}'
+            u'Merge pull request !{pr_id} from {source_repo} {source_ref_name}'
             u'\n\n {pr_title}'.format(
                 pr_id=pull_request.pull_request_id,
                 source_repo=safe_unicode(