Openbravo Issue Tracking System - Openbravo ERP  | 
| View Issue Details | 
  | 
| ID | Project | Category | View Status | Date Submitted | Last Update | 
| 0007089 | Openbravo ERP | A. Platform | public | 2009-01-23 11:08 | 2009-04-21 12:39 | 
  | 
| Reporter | alostale |   | 
| Assigned To | alostale |   | 
| Priority | urgent | Severity | major | Reproducibility | always | 
| Status | closed | Resolution | fixed |   | 
| Platform |  | OS | 5 | OS Version |  | 
| Product Version | pi |   | 
| Target Version |  | Fixed in Version |  |   | 
| Merge Request Status |  | 
| Review Assigned To |  | 
| OBNetwork customer | No | 
| Web browser |  | 
| Modules | Core | 
| Support ticket |  | 
| Regression level |  | 
| Regression date |  | 
| Regression introduced in release |  | 
| Regression introduced by commit |  | 
| Triggers an Emergency Pack | No | 
  | 
| Summary | 0007089: Dynamic redirect to tabs do not work for modules in manual code | 
| Description | In some manual windows a WAD window is called. As windows defined within modules have a different URL pattern when these windows are trying to redirect to a WAD window in a module it doesn't work. 
 
The problem appears with this code in a number of manual windows: 
 
            String strWindowPath = "", strTabName = ""; 
            if (tab != null && tab.length != 0) { 
                strTabName = FormatUtilities.replace(tab[0].name); 
                strWindowPath = "../" 
                        + FormatUtilities.replace(tab[0].description) + "/" 
                        + strTabName + "_Relation.html"; 
            } else 
                strWindowPath = strDefaultServlet; 
 
            printPageClosePopUp(response, vars, strWindowPath); 
 
Here the URL to be called is composed as the window name and the tab name which is not valid for modules. | 
| Steps To Reproduce |  | 
| Proposed Solution | The way to obtain the URL for a tab shouldn't be hardcoded but it should be obtainable through a method. | 
| Additional Information |  | 
| Tags | Modularity | 
| Relationships | | related to  | defect  | 0007026 |   | closed  | alostale  | Links to new module windows do not work  |  
  | 
| Attached Files |  | 
  | 
| Issue History | 
| Date Modified | Username | Field | Change | 
| 2009-01-23 11:08 | alostale | New Issue |  | 
| 2009-01-23 11:08 | alostale | Assigned To |  => alostale | 
| 2009-01-23 11:08 | alostale | sf_bug_id | 0 => 2530779 | 
| 2009-01-23 11:08 | alostale | OBNetwork customer |  => No | 
| 2009-01-23 11:08 | alostale | Regression testing |  => No | 
| 2009-01-23 11:09 | rafaroda | Tag Attached: Modularity |  | 
| 2009-01-23 11:09 | alostale | Note Added: 0012555 |  | 
| 2009-01-23 11:09 | alostale | Note Added: 0012557 |  | 
| 2009-01-28 08:46 | rafaroda | version |  => trunk | 
| 2009-01-30 15:44 | alostale | Relationship added | related to 0007026 | 
| 2009-01-30 17:13 | rafaroda | Status | new => scheduled | 
| 2009-02-04 12:30 | svnbot | Checkin |  | 
| 2009-02-04 12:30 | svnbot | Note Added: 0013128 |  | 
| 2009-02-04 12:30 | svnbot | Status | scheduled => resolved | 
| 2009-02-04 12:30 | svnbot | Resolution | open => fixed | 
| 2009-02-04 12:30 | svnbot | svn_revision |  => 12753 | 
| 2009-02-04 13:11 | svnbot | Checkin |  | 
| 2009-02-04 13:11 | svnbot | Note Added: 0013133 |  | 
| 2009-02-04 13:11 | svnbot | svn_revision | 12753 => 12759 | 
| 2009-02-04 14:15 | svnbot | Checkin |  | 
| 2009-02-04 14:15 | svnbot | Note Added: 0013140 |  | 
| 2009-02-04 14:15 | svnbot | svn_revision | 12759 => 12768 | 
| 2009-04-21 12:39 | psarobe | Status | resolved => closed | 
	| 
		Notes	 | 
	
		 
	 | 
	| 
		
	 | 
	
		
		
			
				Review all the closePopup* and redirect methods. 
Also review the hardcoded links			 | 
		 
		 
	 | 
	
		 
	 | 
	| 
		
	 | 
	
		
		
			| 
				after this fix the new method should be added to the developers guide			 | 
		 
		 
	 | 
	
		 
	 | 
	
		
		
			| 
				(0013128)
			 | 
		 
		
			| 
				svnbot   
			 | 
		 
		
			| 
				2009-02-04 12:30   
							 | 
		 
		 
	 | 
	
		
		
			
				Repository: openbravo 
Revision: 12753 
Author: alostale 
Date: 2009-02-04 12:30:46 +0100 (Wed, 04 Feb 2009) 
 
fixed bug 0007089: Dynamic redirect to tabs do not work for modules in manual code 
 
--- 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/CopyFromInvoice.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/CopyFromOrder.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/CopyFromPOOrder.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/CopyFromSettlement.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/CreateFromMultiple.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/CreateRegFactAcct.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/CreateVatRegisters.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/DropRegFactAcct.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/EditCCPMeasureValues.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/ExportReferenceData.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/InsertAcces.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/ProjectClose.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/ProjectCopyFrom.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/ProjectSetType.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/RequestAction.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/Test_In.java 
U   trunk/src/org/openbravo/erpCommon/ad_actionButton/UpdateMaintenanceScheduled.java 
U   trunk/src/org/openbravo/erpCommon/ad_forms/AlertManagement.java 
U   trunk/src/org/openbravo/erpCommon/ad_forms/AlertManagement_data.xsql 
U   trunk/src/org/openbravo/erpCommon/ad_forms/InvoiceVendorMultiline.java 
U   trunk/src/org/openbravo/erpCommon/ad_process/CopyFromGLJournal.java 
U   trunk/src/org/openbravo/erpCommon/ad_process/ImportAccountServlet.java 
U   trunk/src/org/openbravo/erpCommon/ad_process/ImportBPartnerServlet.java 
U   trunk/src/org/openbravo/erpCommon/ad_process/ImportBudgetServlet.java 
U   trunk/src/org/openbravo/erpCommon/ad_process/ImportOrderServlet.java 
U   trunk/src/org/openbravo/erpCommon/ad_process/ImportProductServlet.java 
U   trunk/src/org/openbravo/erpCommon/ad_process/ImportTaxServlet.java 
U   trunk/src/org/openbravo/erpCommon/ad_reports/ReportAcctRedirectUtility.java 
U   trunk/src/org/openbravo/erpCommon/ad_reports/ReportAcctRedirectUtility_data.xsql 
U   trunk/src/org/openbravo/erpCommon/businessUtility/TabFilter.java 
U   trunk/src/org/openbravo/erpCommon/utility/UsedByLink.java 
U   trunk/src/org/openbravo/erpCommon/utility/Utility.java 
U   trunk/src/org/openbravo/erpCommon/utility/WindowTree.java 
--- 
 
https://dev.openbravo.com/websvn/openbravo/?rev=12753&sc=1 [^] 
			 | 
		 
		 
	 | 
	
		 
	 | 
	
		
		
			| 
				(0013133)
			 | 
		 
		
			| 
				svnbot   
			 | 
		 
		
			| 
				2009-02-04 13:11   
							 | 
		 
		 
	 | 
	
		
	 | 
	
		 
	 | 
	
		
		
			| 
				(0013140)
			 | 
		 
		
			| 
				svnbot   
			 | 
		 
		
			| 
				2009-02-04 14:15   
							 | 
		 
		 
	 | 
	
		
	 |