austgl / open-source-self-check

Automatically exported from code.google.com/p/open-source-self-check
0 stars 0 forks source link

Modifications #21

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Eric,
I was wondering how you would feel about some modifications.  We love the 
checkout, but I wanted to tinker with it and add a few modifications, if you 
don't mind.  Some things I had in mind:

* Option to go straight to the blank checkout list after authenticated.
* Option to show currently checked out items.
* Improvements to work with Koha renewals.
* Enhancements for on-screen card entry.
* Masked manual input for card entry.

I've already added one modification to add a second line of instruction for 
signing in (people didn't get that they had to press enter if they were 
manually typing in their card number).

If you would like any of these modifications, please let me know how you would 
like them.  I've never collaborated on open source before, and I will only be 
doing this in my spare time, so any suggestions on what I can or cannot do to 
contribute to your project would be appreciated.  If you do not want me 
tinkering with it, I will understand.

Christopher Brannon

Original issue reported on code.google.com by CDASupp...@gmail.com on 14 Jun 2012 at 8:22

GoogleCodeExporter commented 9 years ago
Some good ideas. Just keep me posted on your own changes and I'll decide if I 
want to add them to the "official" release.

Original comment by ericmelton1@gmail.com on 14 Jun 2012 at 9:12

GoogleCodeExporter commented 9 years ago
Eric,
Here are a couple modifications I made.  Actually, one wasn't included in my 
list above.  I needed a line on the receipts for the library phone number.

For the phone number, I modified the config.php wording section by adding a new 
variable:

//wording
$library_name= '';
$library_phone= '';  //New Variable

and in the Receipts section, added another header line:

//====================== Paper & Email Receipts ==============
/* add elements to or remove elements from the header & footer arrays below to 
manipulate that piece of the receipt.
the elements will appear on separate lines of the receipt in the order that you 
place them below */ 
$receipt_header[]='Checkout Receipt';
$receipt_header[]=$library_name;
$receipt_header[]='Phone: '.$library_phone; //New header line

Christopher

Original comment by CDASupp...@gmail.com on 18 Jun 2012 at 3:57

GoogleCodeExporter commented 9 years ago
This modification adds a second line (subtext line) for the intro screen text.  
We needed it for more specific information about manually typing in the card 
number.

In the config.php I added the $intro_screen_subtext variable:

$intro_screen_text="Scan or type your card number to begin"; //shown on 
pages/home.php
$intro_screen_subtext="(if typing, press Enter when done)"; //New variable

and in the home.php page, I added the code to include the subtext variable

    <div class="corners" id="banner">
        <span id="swap">
            <img src="images/<?php echo $card_image;?>_card1.png" align="left" class="active" />
            <?php if ($card_image!='magnetic'){ ?>
                <img src="images/<?php echo $card_image;?>_card2.png" align="left"/>
            <?php }?>
        </span>
        <h2><?php echo $intro_screen_text;?>
        <br />  //Page break added
        <span style="font-size:15px;color:#FCFCFC;font-style:italic"><?php echo $intro_screen_subtext;?> //Added new subtext line
        </span>
        </h2>
    </div>

Christopher

Original comment by CDASupp...@gmail.com on 18 Jun 2012 at 4:04

GoogleCodeExporter commented 9 years ago
Eric,
I made a modification, per the request of one of our libraries, to be able to 
disable the option to print.  For some reason some libraries don't want printed 
receipts.  I mimicked the variable in config.php to create a print receipt 
variable (switch):

$allow_email_receipts=true;
$allow_print_receipts=false;  //new switch

I then modified the checkout.php file in pages to look for that switch:

<!--  ============= finish/cancel buttons ============= -->
    <table id="cko_buttons" cellpadding="5">
        <tr>
            <?php if ($allow_print_receipts){?>  //looks for print receipt switch
            <td>
                <div class="ok_button button" id="print" title="selfcheck_button">
                    <h1>Print Receipt</h1>
                </div>
                <div class="thanks_button button" id="print_thanks">
                    <h1>Thanks</h1>
                </div>
            </td>
            <?php }?>  //end of print receipt option
            <?php if (!empty($_SESSION['email']) && $allow_email_receipts){?>
            <td>
                <div class="ok_button button" id="email" title="selfcheck_button">
                    <h1>Email Receipt</h1>
                </div>
                <div class="thanks_button button" id="email_thanks">
                    <h1>Thanks</h1>
                </div>
            </td>
            <?php }?>

Christopher

Original comment by CDASupp...@gmail.com on 26 Jun 2012 at 5:24

GoogleCodeExporter commented 9 years ago
cool. that would be a good addition

Original comment by ericmelton1@gmail.com on 26 Jun 2012 at 5:28

GoogleCodeExporter commented 9 years ago
In response to comment two, adding headers to the email might not be the best 
way to add lines to the email (or printed receipt). We solved this issue by 
adding the variables to the config file and added the following code to 
pages/checkout.php just below this code

                <tr>
                    <td><?php echo $library_name;?></td>
                </tr>

we added this

<? if ("" != $online_catalog_url) { ?>
                <tr>
                    <td style="font-style:italic">Renew your items online:</td>
                </tr>
                <tr>
                    <td style="font-style:italic;"><?php echo $online_catalog_url;?></td>
                </tr>
<? }
  if ("" != $renewal_phone) { ?>
                <tr>
                    <td style="font-style:italic;">Call: <?php echo $renewal_phone;?></td>
                </tr>
<? } ?>

Original comment by lclnetwo...@gmail.com on 18 Jul 2012 at 1:40

GoogleCodeExporter commented 9 years ago

Original comment by ericmelton1@gmail.com on 18 Jul 2012 at 2:44