Add configuration option for image size and title length

bywatersolutions · Dec 2, 2016 · 649ba51 · 649ba51
1 parent ac89230
commit 649ba51
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 9 deletions.
diff --git a/Koha/Plugin/Com/ByWaterSolutions/CoverFlow.pm b/Koha/Plugin/Com/ByWaterSolutions/CoverFlow.pm
@@ -65,7 +65,13 @@ sub report {
     my $json = get_report( { cgi => $cgi } );
     my $data = from_json($json);
 
-    $template->param( 'data' => $data, coverlinks => $self->retrieve_data('coverlinks'), showtitle => $self->retrieve_data('showtitle'), );
+    $template->param(
+            'data'      => $data,
+            coverlinks  => $self->retrieve_data('coverlinks'),
+            showtitle   => $self->retrieve_data('showtitle'),
+            size_limit  => $self->retrieve_data('size_limit'),
+            title_limit => $self->retrieve_data('title_limit'),
+            );
 
     print $cgi->header(
         {
@@ -89,7 +95,14 @@ sub configure {
         my $template = $self->get_template( { file => 'configure.tt' } );
 
         ## Grab the values we already have for our settings, if any exist
-        $template->param( mapping => $self->retrieve_data('mapping'), coverlinks => $self->retrieve_data('coverlinks'), showtitle => $self->retrieve_data('showtitle'), custom_image => $self->retrieve_data('custom_image'), );
+        $template->param(
+                mapping => $self->retrieve_data('mapping'),
+                coverlinks => $self->retrieve_data('coverlinks'),
+                showtitle => $self->retrieve_data('showtitle'),
+                custom_image => $self->retrieve_data('custom_image'),
+                size_limit => $self->retrieve_data('size_limit'),
+                title_limit => $self->retrieve_data('title_limit'),
+                );
 
 
         print $cgi->header(
@@ -111,6 +124,8 @@ sub configure {
                 coverlinks         => $coverlinks,
                 showtitle          => $showtitle,
                 custom_image       => $custom_image,
+                size_limit         => $cgi->param('size_limit'),
+                title_limit        => $cgi->param('title_limit'),
                 last_configured_by => C4::Context->userenv->{'number'},
             }
         );

diff --git a/Koha/Plugin/Com/ByWaterSolutions/CoverFlow/configure.tt b/Koha/Plugin/Com/ByWaterSolutions/CoverFlow/configure.tt
@@ -44,9 +44,17 @@ $(document).ready(function() {
         <label for="custom_image">Custom image to use if none is found (enter a full URL):</label>
         <input type="text" id="custom_image" name="custom_image" value="[% custom_image %]" />
 
+        <br/>
+        <label for="size_limit">Limit image size (height in pixels)</label>
+        <input type="number" id="size_limit" name="size_limit" value="[% size_limit %]" />
+
+        <br/>
+        <label for="title_limit">Limit title length (width in pixels)</label>
+        <input type="number" id="title_limit" name="title_limit" value="[% title_limit %]" />
+
         [% IF error %]<div class="warn alert error">Error: [% error %]</div>[% END %]
 
-        <label for="mapping">Mapping: </label>
+        <br/><label for="mapping">Mapping: </label>
         <textarea rows="15" cols="200" name="mapping">[% mapping %]</textarea>
         <div>
             Add a line for each html element you'd like to replace with a widget and the id of the report to use to generate the widget.

diff --git a/Koha/Plugin/Com/ByWaterSolutions/CoverFlow/report.tt b/Koha/Plugin/Com/ByWaterSolutions/CoverFlow/report.tt
@@ -6,23 +6,23 @@
                     [% IF (coverlinks) %]
                         <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% d.biblionumber %]">
                             [% IF d.localcover %]
-                                <img src="/cgi-bin/koha/opac-image.pl?biblionumber=[% d.biblionumber %]" />
+                                <img [% IF size_limit %]style="max-height:[% size_limit %]px;"[% END %] src="/cgi-bin/koha/opac-image.pl?biblionumber=[% d.biblionumber %]" />
                             [% ELSE %]
-                                <img src="https://images-na.ssl-images-amazon.com/images/P/[% d.isbn %].01.LZZZZZZZ.jpg" />
+                                <img [% IF size_limit %]style="max-height:[% size_limit %]px;"[% END %] src="https://images-na.ssl-images-amazon.com/images/P/[% d.isbn %].01.LZZZZZZZ.jpg" />
                             [% END %]
                         </a>
                         [% IF showtitle %]
                             <br/>
-                            <p>[% d.title %]</p>
+                            <p [% IF title_limit %]style="max-width:[% title_limit %]px;display:inline-block;"[% END %]>[% d.title %]</p>
                         [% END %]
                     [% ELSE %]
                         [% IF d.localcover %]
-                            <img src="/cgi-bin/koha/opac-image.pl?biblionumber=[% d.biblionumber %]" />
+                            <img [% IF size_limit %]style="max-height:[% size_limit %]px;"[% END %] src="/cgi-bin/koha/opac-image.pl?biblionumber=[% d.biblionumber %]" />
                         [% ELSE %]
-                            <img src="https://images-na.ssl-images-amazon.com/images/P/[% d.isbn %].01.LZZZZZZZ.jpg" />
+                            <img [% IF size_limit %]style="max-height:[% size_limit %]px;"[% END %] src="https://images-na.ssl-images-amazon.com/images/P/[% d.isbn %].01.LZZZZZZZ.jpg" />
                         [% END %]
                         <br/>
-                        <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% d.biblionumber %]">[% d.title %]</a>
+                        <a [% IF title_limit %]style="max-width:[% title_limit %]px;display:inline-block;"[% END %] href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% d.biblionumber %]">[% d.title %]</a>
                     [% END %]
                 </li>
             [% END %]