[open-ils-commits] [GIT] Evergreen ILS branch rel_2_2 updated. 0b7ce9a6f0b416403f56e549c1410ff4fa01f8bf

Evergreen Git git at git.evergreen-ils.org
Wed Dec 19 09:57:36 EST 2012


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_2_2 has been updated
       via  0b7ce9a6f0b416403f56e549c1410ff4fa01f8bf (commit)
       via  0a14b5951e6e80a73daa7d03485cf69513bd82dc (commit)
       via  6a154db860444956b413bb3b3a5eaf3cc865ec6b (commit)
      from  d09f81737f241b8327cf027463121e3314711463 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 0b7ce9a6f0b416403f56e549c1410ff4fa01f8bf
Author: Dan Scott <dscott at laurentian.ca>
Date:   Tue Dec 18 08:52:55 2012 -0500

    Dojo IDL: Set the "Accept-Language" header
    
    The internationalization support for /reports/fm_IDL.xml keys off of the
    Accept-Language header - which, if not set to ^[a-z]{2}-[A-Z]{2}$, falls
    back to en-US. Setting the header in the dojo.xhrGet() request makes
    Apache able to serve up the right flavour of /reports/fm_IDL.xml
    
    Signed-off-by: Dan Scott <dscott at laurentian.ca>
    Signed-off-by: Pasi Kallinen <pasi.kallinen at pttk.fi>

diff --git a/Open-ILS/web/js/dojo/fieldmapper/IDL.js b/Open-ILS/web/js/dojo/fieldmapper/IDL.js
index 7efb967..8ccdca5 100644
--- a/Open-ILS/web/js/dojo/fieldmapper/IDL.js
+++ b/Open-ILS/web/js/dojo/fieldmapper/IDL.js
@@ -4,8 +4,7 @@ if(!dojo._hasResource["fieldmapper.IDL"]) {
     dojo.provide("fieldmapper.IDL");
     dojo.declare('fieldmapper.IDL', null, {
     
-        _URL_PATH : '/reports/fm_IDL.xml', // XXX locale?
-        // -- just need to set up xmlent and use '/reports/'+OpenSRF.locale+'/fm_IDL.xml'
+        _URL_PATH : '/reports/fm_IDL.xml',
         NS_REPORTS : 'http://open-ils.org/spec/opensrf/IDL/reporter/v1',
         NS_PERSIST : 'http://open-ils.org/spec/opensrf/IDL/persistence/v1',
         NS_OBJ : 'http://open-ils.org/spec/opensrf/IDL/objects/v1',
@@ -56,6 +55,7 @@ if(!dojo._hasResource["fieldmapper.IDL"]) {
                         handleAs : 'xml',
                         sync : true,
                         timeout : 10000,
+                        headers : {"Accept-Language": OpenSRF.locale},
                         load : function (response) {
                             self._parse(response);
                             fieldmapper.IDL.loaded = true;

commit 0a14b5951e6e80a73daa7d03485cf69513bd82dc
Author: Dan Scott <dscott at laurentian.ca>
Date:   Tue Dec 18 07:39:18 2012 -0500

    Set locale properly in Apache configuration
    
    A RewriteCond only applies to the next RewriteRule, while our config was
    attempting to apply multiple rules for a single condition. Fix that up
    by using skip (S) rules to apply blocks of rules for the condition in
    question.
    
    Also, use the HTTP:Accept-Language value directly, rather than relying
    on a backreference (which did not seem to be working as desired).
    
    These fixes enable screens like the Register Patron interface to present
    field labels from the IDL in the correct locale, whereas previously they
    would always default to en-US.
    
    Signed-off-by: Dan Scott <dscott at laurentian.ca>
    Signed-off-by: Pasi Kallinen <pasi.kallinen at pttk.fi>

diff --git a/Open-ILS/examples/apache/eg_vhost.conf b/Open-ILS/examples/apache/eg_vhost.conf
index ebcc1f0..0052443 100644
--- a/Open-ILS/examples/apache/eg_vhost.conf
+++ b/Open-ILS/examples/apache/eg_vhost.conf
@@ -216,19 +216,17 @@ RewriteRule ^/opac/[a-z]{2}-[A-Z]{2}/extras/slimpac/(.*)$ /opac/%1/extras/slimpa
 # ----------------------------------------------------------------------------------
 # Run server-side XUL and XHTML through xmlent to load the correct XML entities
 # ----------------------------------------------------------------------------------
-RewriteCond %{HTTP:Accept-Language} ([a-z]{2}-[A-Z]{2})
-RewriteRule ^/xul/	-	[E=locale:%1]
-RewriteRule ^/reports/	- 	[E=locale:%1]
-
+RewriteCond %{HTTP:Accept-Language} ^([a-z]{2}-[A-Z]{2})$
 # Default to en-US if we haven't matched a locale of the form xx-YY 
-RewriteCond %{HTTP:Accept-Language} !([a-z]{2}-[A-Z]{2})
-RewriteRule ^/xul/	-	[E=locale:en-US]
-RewriteRule ^/reports/	- 	[E=locale:en-US]
-
-# Default to en-US if we are just given en
-RewriteCond %{ENV:locale} ^$ [OR]
-RewriteCond %{ENV:locale} ^en$
-RewriteRule . - [E=locale:en-US]
+RewriteRule .? - [S=4]
+RewriteRule ^/xul/      -       [E=locale:en-US]
+RewriteRule ^/reports/  -       [E=locale:en-US]
+RewriteRule .? - [E=locale:en-US]
+RewriteRule .? - [S=3]
+# Otherwise, set our real locale
+RewriteRule ^/xul/      -       [E=locale:%{HTTP:Accept-Language}]
+RewriteRule ^/reports/  -       [E=locale:%{HTTP:Accept-Language}]
+RewriteRule .? - [E=locale:%{HTTP:Accept-Language}]
 
 <LocationMatch /xul/.*\.x?html$>
     Options +Includes

commit 6a154db860444956b413bb3b3a5eaf3cc865ec6b
Author: Pasi Kallinen <pasi.kallinen at pttk.fi>
Date:   Sun Dec 2 22:49:42 2012 +0200

    LP#1078596: Cannot translate strings handled by fieldmapper
    
    Strings that should be translatable only show up in English,
    for example in the Patron Registration screen.
    
    There's a bug in the fieldmapper where it sets a variable to
    an empty array, but later in the code only checks whether that
    particular variable exists, not if it's an empty array.
    
    Signed-off-by: Pasi Kallinen <pasi.kallinen at pttk.fi>
    Signed-off-by: Dan Scott <dscott at laurentian.ca>

diff --git a/Open-ILS/web/js/dojo/fieldmapper/IDL.js b/Open-ILS/web/js/dojo/fieldmapper/IDL.js
index 9cc7a3c..7efb967 100644
--- a/Open-ILS/web/js/dojo/fieldmapper/IDL.js
+++ b/Open-ILS/web/js/dojo/fieldmapper/IDL.js
@@ -32,7 +32,7 @@ if(!dojo._hasResource["fieldmapper.IDL"]) {
                 window._preload_fieldmapper_IDL = null;
             }
 
-            if(!fieldmapper.IDL.fmclasses || (classlist && classlist.length)) {
+            if(!fieldmapper.IDL.fmclasses || !fieldmapper.IDL.fmclasses.length || (classlist && classlist.length)) {
                 var idl_url = this._URL_PATH;
 
                 if (classlist.length && (classlist.length > 1 || classlist[0] != '*')) {

-----------------------------------------------------------------------

Summary of changes:
 Open-ILS/examples/apache/eg_vhost.conf  |   22 ++++++++++------------
 Open-ILS/web/js/dojo/fieldmapper/IDL.js |    6 +++---
 2 files changed, 13 insertions(+), 15 deletions(-)


hooks/post-receive
-- 
Evergreen ILS


More information about the open-ils-commits mailing list