[Rt-commit] [rtir] 09/12: Grab and pass along the id of the Ticket we're working from.

Kevin Falcone falcone at bestpractical.com
Fri Jul 12 10:40:03 EDT 2013


This is an automated email from the git hooks/post-receive script.

falcone pushed a commit to branch 2.9/edit-search-regressions
in repository rtir.

commit cc7b83a13b0d6718ce2e430e2155da9efa09b0ea
Author: Kevin Falcone <falcone at bestpractical.com>
Date:   Thu Jul 11 19:18:41 2013 -0400

    Grab and pass along the id of the Ticket we're working from.
    
    None of the Refine Page Menus contain the ticket id after any action on
    the Query Builder page.
    
    From an Incident, visiting Merge -> Edit Search -> Add these terms ->
    Merge fails because of this.
    
    Any attempt to click on Add these terms and Search from a refine page
    takes you to an error page about being unable to load the ticket the
    re-calculation of $ResultPage cannot work without access to the original
    id that was in the query args when we started.
    
    The other solution to this involves swapping ARGS so taht ResultPage is
    taken from the submission, but that doesn't fix the Page Menus.
---
 html/RTIR/Search/Elements/BuildQuery | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/html/RTIR/Search/Elements/BuildQuery b/html/RTIR/Search/Elements/BuildQuery
index 30bdc39..fdfa906 100644
--- a/html/RTIR/Search/Elements/BuildQuery
+++ b/html/RTIR/Search/Elements/BuildQuery
@@ -50,6 +50,9 @@
 <input type="hidden" class="hidden" name="SavedSearchId" value="<% $saved_search{'Id'} %>" />
 <input type="hidden" class="hidden" name="ResultPage" value="<% $ResultPage %>" />
 <input type="hidden" class="hidden" name="BaseQuery" value="<% $BaseQuery %>" />
+% if ( $DECODED_ARGS->{id} ) {
+    <input type="hidden" class="hidden" name="id" value="<% $DECODED_ARGS->{id} %>" />
+% }
 
 <input type="hidden" class="hidden" name="Query" value="<% $query{'Query'} %>" />
 <input type="hidden" class="hidden" name="Format" value="<% $query{'Format'} %>" />

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the Rt-commit mailing list