svn commit: r1863562 - in /ofbiz/ofbiz-framework/branches/release18.12: ./ themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/apple/ themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/classic/ themes/common-theme/webapp/co...

Author: jleroux
Date: Mon Jul 22 15:22:03 2019
New Revision: 1863562

URL: http://svn.apache.org/viewvc?rev=1863562&view=rev
Log:
"Applied fix from trunk for revision: 1863560"
------------------------------------------------------------------------
r1863560 | jleroux | 2019-07-22 17:21:14 +0200 (lun. 22 juil. 2019) | 37 lignes

Fixed: Unknown request [images]; this request does not exist or cannot be
called directly.
(OFBIZ-10895)

Today Olivier reported directly to me that there was an issue in HR:
<<If you go to humanres, at the level of the tree that appears the icons to expand
the tree no longer appear (but you can click...)
This is due to the path of the d.png image which is wrong in the css file
(there is too much image in the path)>>

I should have seen it because the d.png file is, with other related image files,
in the same directory than style.css. And there was an inconsistency in the
style.css files regarding the d.png file path.

We should note that we have still this error in log:
2019-07-22 16:25:26,275 |jsse-nio-8443-exec-7 |ControlServlet                
|E| Error in request handler:
org.apache.ofbiz.webapp.control.RequestHandlerException: Unknown request [d.png];
this request does not exist or cannot be called directly.
at org.apache.ofbiz.webapp.control.RequestHandler.doRequest
(RequestHandler.java:277) ~[ofbiz.jar:?]

As I wrote in my last commit for OFBIZ-10895:
The only safe way to go is to not only check the error log for
"Unknown request [...."
but also to look for 404 in access log, not all request show erroneously in
error log.

Some error appears before the user is logged in, as I wrote in the Jira:
<<I noticed that this error message often appears before login. A 404 can be
seen in the access log and then once logged the issue no longer exists
(ie we get a 200). I have no yet investigated why...>>

Nevertheless, I want to remove this misleading error message. I hope I'll do
that before forgetting.

Thanks: Olivier Heintz
------------------------------------------------------------------------

Modified:
    ofbiz/ofbiz-framework/branches/release18.12/   (props changed)
    ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/apple/style.css
    ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/classic/style.css
    ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/default-rtl/style.css
    ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/default/style.css

Propchange: ofbiz/ofbiz-framework/branches/release18.12/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jul 22 15:22:03 2019
@@ -10,4 +10,4 @@
 /ofbiz/branches/json-integration-refactoring:1634077-1635900
 /ofbiz/branches/multitenant20100310:921280-927264
 /ofbiz/branches/release13.07:1547657
-/ofbiz/ofbiz-framework/trunk:1849931,1850015,1850023,1850530,1850647,1850685,1850694,1850711,1850914,1850918,1850921,1850948,1850953,1851006,1851013,1851068,1851074,1851130,1851158,1851200,1851224,1851247,1851254,1851315,1851319,1851350,1851353,1851433,1851500,1851805,1851885,1851998,1852503,1852587,1852818,1852882,1853070,1853109,1853691,1853745,1853750,1854306,1854457,1854683,1855078,1855083,1855287,1855371,1855403,1855488,1855492,1855497,1855501,1855898,1856212,1856405,1856455,1856459-1856460,1856484,1856598,1856610,1856613,1856617,1856667,1857088,1857099,1857152,1857154,1857173,1857180,1857213,1857392,1857617,1857692,1857813,1858035,1858092,1858180,1858250,1858256,1858275,1858319,1858347,1858432,1858444,1858483,1858523,1858539,1858965,1858980,1859033,1859055,1859087,1859255,1859263,1859268,1859543,1859571,1859576,1859691,1859694,1859698,1859704,1859708,1859735,1859796,1859800,1859807,1859871,1859877,1859882,1859909,1859911,1859915,1859931,1859968,1859972,1859981,1860082,1860141,
 1860274,1860357,1860526,1860592,1860597,1860613,1860797,1861615,1861811,1861815,1861828,1861834,1861837,1861849,1861859,1861869,1862045-1862046,1862207,1862271,1862278,1862466,1862648
+/ofbiz/ofbiz-framework/trunk:1849931,1850015,1850023,1850530,1850647,1850685,1850694,1850711,1850914,1850918,1850921,1850948,1850953,1851006,1851013,1851068,1851074,1851130,1851158,1851200,1851224,1851247,1851254,1851315,1851319,1851350,1851353,1851433,1851500,1851805,1851885,1851998,1852503,1852587,1852818,1852882,1853070,1853109,1853691,1853745,1853750,1854306,1854457,1854683,1855078,1855083,1855287,1855371,1855403,1855488,1855492,1855497,1855501,1855898,1856212,1856405,1856455,1856459-1856460,1856484,1856598,1856610,1856613,1856617,1856667,1857088,1857099,1857152,1857154,1857173,1857180,1857213,1857392,1857617,1857692,1857813,1858035,1858092,1858180,1858250,1858256,1858275,1858319,1858347,1858432,1858444,1858483,1858523,1858539,1858965,1858980,1859033,1859055,1859087,1859255,1859263,1859268,1859543,1859571,1859576,1859691,1859694,1859698,1859704,1859708,1859735,1859796,1859800,1859807,1859871,1859877,1859882,1859909,1859911,1859915,1859931,1859968,1859972,1859981,1860082,1860141,
 1860274,1860357,1860526,1860592,1860597,1860613,1860797,1861615,1861811,1861815,1861828,1861834,1861837,1861849,1861859,1861869,1862045-1862046,1862207,1862271,1862278,1862466,1862648,1863560

Modified: ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/apple/style.css
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/apple/style.css?rev=1863562&r1=1863561&r2=1863562&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/apple/style.css (original)
+++ ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/apple/style.css Mon Jul 22 15:22:03 2019
@@ -6,7 +6,7 @@
 
 .jstree-apple > ul { background:url("bg.jpg") left top repeat; }
 .jstree-apple li,
-.jstree-apple ins { background-image:url("images/d.png"); background-repeat:no-repeat; background-color:transparent; }
+.jstree-apple ins { background-image:url("d.png"); background-repeat:no-repeat; background-color:transparent; }
 .jstree-apple li { background-position:-90px 0; background-repeat:repeat-y;  }
 .jstree-apple li.jstree-last { background:transparent; }
 .jstree-apple .jstree-open > ins { background-position:-72px 0; }

Modified: ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/classic/style.css
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/classic/style.css?rev=1863562&r1=1863561&r2=1863562&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/classic/style.css (original)
+++ ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/classic/style.css Mon Jul 22 15:22:03 2019
@@ -5,7 +5,7 @@
  */
 
 .jstree-classic li,
-.jstree-classic ins { background-image:url("images/d.png"); background-repeat:no-repeat; background-color:transparent; }
+.jstree-classic ins { background-image:url("d.png"); background-repeat:no-repeat; background-color:transparent; }
 .jstree-classic li { background-position:-90px 0; background-repeat:repeat-y;  }
 .jstree-classic li.jstree-last { background:transparent; }
 .jstree-classic .jstree-open > ins { background-position:-72px 0; }

Modified: ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/default-rtl/style.css
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/default-rtl/style.css?rev=1863562&r1=1863561&r2=1863562&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/default-rtl/style.css (original)
+++ ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/default-rtl/style.css Mon Jul 22 15:22:03 2019
@@ -5,7 +5,7 @@
  */
 
 .jstree-default-rtl li,
-.jstree-default-rtl ins { background-image:url("images/d.png"); background-repeat:no-repeat; background-color:transparent; }
+.jstree-default-rtl ins { background-image:url("d.png"); background-repeat:no-repeat; background-color:transparent; }
 .jstree-default-rtl li { background-position:-90px 0; background-repeat:repeat-y; }
 .jstree-default-rtl li.jstree-last { background:transparent; }
 .jstree-default-rtl .jstree-open > ins { background-position:-72px 0; }

Modified: ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/default/style.css
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/default/style.css?rev=1863562&r1=1863561&r2=1863562&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/default/style.css (original)
+++ ofbiz/ofbiz-framework/branches/release18.12/themes/common-theme/webapp/common/js/jquery/plugins/jsTree/themes/default/style.css Mon Jul 22 15:22:03 2019
@@ -5,7 +5,7 @@
  */
 
 .jstree-default li,
-.jstree-default ins { background-image:url("images/d.png"); background-repeat:no-repeat; background-color:transparent; }
+.jstree-default ins { background-image:url("d.png"); background-repeat:no-repeat; background-color:transparent; }
 .jstree-default li { background-position:-90px 0; background-repeat:repeat-y; }
 .jstree-default li.jstree-last { background:transparent; }
 .jstree-default .jstree-open > ins { background-position:-72px 0; }