Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0013881
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] Z. Othersminorhave not tried2010-07-02 19:202011-09-07 13:44
ReportershuehnerView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionunable to reproduceFixed in Version
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product Version2.50SCM revision 
Review Assigned To
Web browser
ModulesUser Interface Application
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0013881: Back button handling not working in one special case

DescriptionBack button handling is not working correctly in the special case described below
Steps To ReproduceUsing new layout
As openbravo admin role
goto general ledger report
use 01.01.2000 - 13.12.2010
and the select the smallbazaar accounting schema
use search (the report output is shown as html below then)
follow first link (31-12-2006)
Follow link in resulting page (Journal Entry 1)
Note a new classic window was opened in the same tab
Use browser back button
-> Error: You do not have sufficient privileges to access this page

plus info in log:

a3196480 785526 [http-8080-3] ERROR org.openbravo.base.VariablesBase - Input: MANUAL not accepted by filter: org.openbravo.base.filter.ValueListFilter@6be9ad6c
TagsNo tags attached.
Attached Filespatch file icon utils_js.patch [^] (4,726 bytes) 2010-07-03 21:30 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0013882 closedshuehner Error thrown from i.e. ValueListFilter Value 'zz' is not an accepted input should contain error location (stacktrace) in logfile 

-  Notes
(0029042)
mtaal (manager)
2010-07-03 21:29

I studied this case in detail. There are a number of obstacles to overcome:
- the computation of the command/mode variable in the utils.js in the sendWindowInfoToMDI does not take into account that windows can have many different types of commands. The current computation defaults to 'manual' for a manual window, resulting in the error page being shown (You do not have sufficient privileges to access this page)
- One way of making this method more advanced is to let it find the command parameter which was used to post the form.
- However this command parameter is not present in the url but is often a posted form attribute. The utils.js is run when the page returns to the browser so the posted info is lost.
- A solution can be to append the command parameter to each url when the submitCommandForm method is called. This approach has been tried and gives good results (more testing is needed).
- Then when the utils.js is run in the browser the command can be read from the url and placed in the history.

See the attached diff for this change.

Another change which is needed is that in the Menu.java the set of allowed commands should be extended/or not limited.

As this issue requires multiple changes in core code and also is a specific case of a manual window it is not done as part of rc1.

- Issue History
Date Modified Username Field Change
2010-07-02 19:20 shuehner New Issue
2010-07-02 19:20 shuehner Assigned To => mtaal
2010-07-02 19:21 shuehner Relationship added related to 0013882
2010-07-03 21:29 mtaal Note Added: 0029042
2010-07-03 21:30 mtaal File Added: utils_js.patch
2010-12-06 07:48 alostale Status new => scheduled
2011-09-07 13:44 mtaal Status scheduled => closed
2011-09-07 13:44 mtaal Resolution open => unable to reproduce


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker