Bugzilla – Attachment 34459 Details for
Bug 11559
Professional cataloger's interface
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 11559 - Supporting changes for Rancor
Bug-11559---Supporting-changes-for-Rancor.patch (text/plain), 8.38 KB, created by
Kyle M Hall (khall)
on 2014-12-16 14:50:09 UTC
(
hide
)
Description:
Bug 11559 - Supporting changes for Rancor
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2014-12-16 14:50:09 UTC
Size:
8.38 KB
patch
obsolete
>From d0abe11fad0c138acb71f35d62b41db5b208ff34 Mon Sep 17 00:00:00 2001 >From: Jesse Weaver <pianohacker@gmail.com> >Date: Tue, 11 Mar 2014 11:13:51 +0100 >Subject: [PATCH] Bug 11559 - Supporting changes for Rancor > > * Extends login screen to pass along #hash > * Adds JSONP support to C4::Service > * Extends humanmsg to allow per-message classes > * Adds proper charset to results of svc/bib >--- > C4/Auth.pm | 2 + > C4/Service.pm | 16 +++++++++-- > .../intranet-tmpl/lib/jquery/plugins/humanmsg.js | 29 ++++++++++--------- > koha-tmpl/intranet-tmpl/prog/en/css/humanmsg.css | 15 ++++++---- > .../intranet-tmpl/prog/en/css/staff-global.css | 13 ++++++--- > koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt | 8 +++++ > svc/bib | 2 +- > 7 files changed, 57 insertions(+), 28 deletions(-) > >diff --git a/C4/Auth.pm b/C4/Auth.pm >index b808c66..5d21981 100644 >--- a/C4/Auth.pm >+++ b/C4/Auth.pm >@@ -341,6 +341,8 @@ sub get_template_and_user { > $template->param(dateformat => C4::Context->preference('dateformat')) > } > >+ $template->param(auth_forwarded_hash => $in->{'query'}->param('auth_forwarded_hash')); >+ > # these template parameters are set the same regardless of $in->{'type'} > > # Set the using_https variable for templates >diff --git a/C4/Service.pm b/C4/Service.pm >index d9990ff..512ed15 100644 >--- a/C4/Service.pm >+++ b/C4/Service.pm >@@ -55,6 +55,16 @@ BEGIN { > > our ( $query, $cookie ); > >+sub _output { >+ my ( $response, $status ) = @_; >+ >+ if ( $query->param( 'callback' ) ) { >+ output_with_http_headers $query, $cookie, $query->param( 'callback' ) . '(' . $response->output . ');', 'js'; >+ } else { >+ output_with_http_headers $query, $cookie, $response->output, 'json', $status; >+ } >+} >+ > =head1 METHODS > > =head2 init >@@ -113,7 +123,7 @@ sub return_error { > $response->param( message => $error ) if ( $error ); > $response->param( type => $type, %flags ); > >- output_with_http_headers $query, $cookie, $response->output, 'json', '400 Bad Request'; >+ _output( $response, '400 Bad Request' ); > exit; > } > >@@ -159,7 +169,7 @@ sub return_multi { > } > > $response->param( 'multi' => JSON::true, responses => \@responses_formatted, @flags ); >- output_with_http_headers $query, $cookie, $response->output, 'json', '207 Multi-Status'; >+ _output( $response, '207 Multi-Status' ); > } > > exit; >@@ -177,7 +187,7 @@ exit with HTTP status 200. > sub return_success { > my ( $class, $response ) = @_; > >- output_with_http_headers $query, $cookie, $response->output, 'json'; >+ _output( $response ); > } > > =head2 require_params >diff --git a/koha-tmpl/intranet-tmpl/lib/jquery/plugins/humanmsg.js b/koha-tmpl/intranet-tmpl/lib/jquery/plugins/humanmsg.js >index 4a2d62c..6b3be81 100644 >--- a/koha-tmpl/intranet-tmpl/lib/jquery/plugins/humanmsg.js >+++ b/koha-tmpl/intranet-tmpl/lib/jquery/plugins/humanmsg.js >@@ -21,7 +21,7 @@ var humanMsg = { > if (msgOpacity != undefined) humanMsg.msgOpacity = parseFloat(msgOpacity); > > // Inject the message structure >- jQuery(appendTo).append('<div id="'+humanMsg.msgID+'" class="humanMsg"><div class="round"></div><div id="'+humanMsg.msgID+'-contents"></div><div class="round"></div></div> <div id="'+humanMsg.logID+'"><p class="launcher">'+logName+'</p><ul></ul></div>') >+ jQuery(appendTo).append('<div id="'+humanMsg.msgID+'" class="humanMsg"><div id="'+humanMsg.msgID+'-contents"></div></div> <div id="'+humanMsg.logID+'"><p class="launcher">'+logName+'</p><ul></ul></div>'); > > jQuery('#'+humanMsg.logID+' p').click( > function() { jQuery(this).siblings('ul').slideToggle() } >@@ -29,28 +29,29 @@ var humanMsg = { > }, > > displayAlert: function(msg, options) { >- humanMsg.displayMsg(msg, options, true); >+ humanMsg.displayMsg('<p>' + msg + '</p>', $.extend({log: false}, options), true); > }, > >- displayMsg: function(msg, options, is_alert) { >+ displayMsg: function(msg, options) { > if (msg == '') > return; > >- if (options != undefined) { >- delay = 'delay' in options ? parseInt(options.delay) * 1000 : 1000 >- life = 'life' in options ? parseInt(options.life) * 1000 : Infinity >- } else { >- delay = 1000 >- life = Infinity >- } >+ options = $.extend({ >+ delay: 1000, >+ life: Infinity, >+ log: true, >+ className: '', >+ }, options); > >+ clearTimeout(humanMsg.t1); > clearTimeout(humanMsg.t2); > > // Inject message >- jQuery('#'+humanMsg.msgID+'-contents').html(is_alert ? ('<p>' + msg + '</p>') : msg) >+ jQuery('#'+humanMsg.msgID+'-contents').html(msg); > > // Show message >- jQuery('#'+humanMsg.msgID).show().animate({ opacity: humanMsg.msgOpacity}, 200, function() { >+ jQuery('#'+humanMsg.msgID).attr('class', 'humanMsg ' + options.className).show().animate({ opacity: humanMsg.msgOpacity}, 200, function() { >+ if ( !options.log ) return true; > jQuery('#'+humanMsg.logID) > .show().children('ul').prepend('<li>'+msg+'</li>') // Prepend message to log > .children('li:first').slideDown(200) // Slide it down >@@ -64,9 +65,9 @@ var humanMsg = { > }) > > // Watch for mouse & keyboard in `delay` >- humanMsg.t1 = setTimeout("humanMsg.bindEvents()", delay) >+ humanMsg.t1 = setTimeout("humanMsg.bindEvents()", options.delay) > // Remove message after `life` >- humanMsg.t2 = setTimeout("humanMsg.removeMsg()", life) >+ humanMsg.t2 = setTimeout("humanMsg.removeMsg()", options.life) > }, > > bindEvents: function() { >diff --git a/koha-tmpl/intranet-tmpl/prog/en/css/humanmsg.css b/koha-tmpl/intranet-tmpl/prog/en/css/humanmsg.css >index 47eb2e6..f9b0071 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/css/humanmsg.css >+++ b/koha-tmpl/intranet-tmpl/prog/en/css/humanmsg.css >@@ -9,6 +9,7 @@ html, body { > } > > .humanMsg { >+ border-radius: 5px; > font: normal 20px/50px Helvetica, Arial, Sans-Serif; > letter-spacing: -1px; > position: fixed; >@@ -23,6 +24,14 @@ html, body { > z-index: 100000; > } > >+.humanError { >+ background-color: #400; >+} >+ >+.humanSuccess { >+ background-color: #040; >+} >+ > .humanMsg h3 { > margin: 0.3em; > margin-top: 0; >@@ -30,12 +39,6 @@ html, body { > font-weight: bold; > } > >-.humanMsg .round { >- border-left: solid 2px white; >- border-right: solid 2px white; >- font-size: 1px; height: 2px; >- } >- > .humanMsg p { > margin: 0.3em; > } >diff --git a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css >index 5eb6aab2..5c56e74 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css >+++ b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css >@@ -405,13 +405,18 @@ dd { > font-weight : normal; > } > >-div#toolbar { >+.btn-toolbar { > background-color : #EDF4F6; > padding: 5px 5px 5px 5px; > border-radius: 5px 5px 0 0; > border: 1px solid #E6F0F2; > } > >+.btn-toolbar .yui-menu-button button, >+.btn-toolbar .yui-button-button button { >+ line-height : 1.7em; >+} >+ > ul.toolbar { > padding-left : 0; > } >@@ -2385,8 +2390,8 @@ video { > background-position:-48px -166px; > } > >-#toolbar .btn, >-#toolbar .dropdown-menu { >+.btn-toolbar .btn, >+.btn-toolbar .dropdown-menu { > font-size: 13px; > } > a.btn:link, >@@ -2740,4 +2745,4 @@ span.onsite_checkout { > background-color : rgba(255, 242, 206, 0.5); > border-radius: 4px; > border : 1px solid #FFF2CE; >-} >\ No newline at end of file >+} >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt >index 5f97758..ee4b6ba 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/auth.tt >@@ -78,4 +78,12 @@ > </div> > </div> > >+<script> >+$(document).ready( function() { >+ if ( document.location.hash ) { >+ $( '#loginform' ).append( '<input name="auth_forwarded_hash" type="hidden" value="' + document.location.hash + '"/>' ); >+ } >+} ); >+</script> >+ > [% INCLUDE 'intranet-bottom.inc' %] >diff --git a/svc/bib b/svc/bib >index d7cd169..2ae7519 100755 >--- a/svc/bib >+++ b/svc/bib >@@ -68,7 +68,7 @@ sub fetch_bib { > my $biblionumber = shift; > my $record = GetMarcBiblio( $biblionumber, $query->url_param('items') ); > if (defined $record) { >- print $query->header(-type => 'text/xml'); >+ print $query->header(-type => 'text/xml', -charset => 'utf-8'); > print $record->as_xml_record(); > } else { > print $query->header(-type => 'text/xml', -status => '404 Not Found'); >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 11559
:
26069
|
26070
|
26071
|
26104
|
26653
|
26654
|
27140
|
27141
|
27142
|
27757
|
27758
|
27759
|
27760
|
27761
|
27900
|
27901
|
27902
|
27993
|
27995
|
34458
|
34459
|
34460
|
34461
|
34462
|
34463
|
36243
|
36244
|
36245
|
36246
|
36247
|
36248
|
36455
|
38533
|
38534
|
38535
|
38536
|
38537
|
38538
|
38539
|
40153
|
40154
|
40155
|
40543
|
40555
|
40556
|
40557
|
40558
|
40811
|
40812
|
40813
|
40814
|
40815
|
41425
|
41453
|
41476
|
42373
|
43261
|
43547
|
43614
|
43615
|
43616
|
43617
|
43618
|
43619
|
43620
|
43621
|
43622
|
43623
|
43703
|
43704
|
43705
|
43706
|
43707
|
43708
|
43709
|
43710
|
43711
|
43712
|
43713
|
43772
|
43790
|
43883
|
43969
|
44016
|
44018
|
44022
|
44023
|
44024
|
44025
|
44026
|
44027
|
44028
|
44029
|
44030
|
44031
|
44032
|
44033
|
44034
|
44035
|
44036