Gutscheinmodul macht Fehler auf Seite Bezahlart

[Iris]

Hallo,

ich habe das Gutschein-System aktiviert und dann den Bestellvorgang getestet.

Dabei habe ich festgestellt, dass auf der Seite "Bezahlart wählen" jetzt sehr große Abstände sind und zwar ober- und unterhalb von "Anwählen, wenn Sie Ihr Guthaben verwenden möchten".

Weiß jemand, wie ich die raus bekomme?

LG Iris

Linkback: https://www.modified-shop.org/forum/index.php?topic=25943.0

[Iris]

Hallo,

ich habe mal einen screen gemacht, vielleicht hilft das

Es wäre super, wenn mit jemand sagen kann, wie ich die Zwischenräume raus bekomme?

LG Iris

[jannemann]

Hallo Iris,

hast du mal den Template- und Browsercache geleert?
Wenn das nicht hilft, hast du mal einen Link zum Shop?

Schöne Grüße,
Jan

[phpGuru]

@Iris

wird in der Datei shop/include/classes/order_total.php
ganz am Anfang gebastelt.

schau mal nach: fix gv display on checkout
dann kommen viele, viele Zeilen alt Code und dann geht's los. Vielleicht passt da bei dir was nicht zusammen.

Hätte man nach so vielen Jahren auch mal aufräumen können. Mehr grün als schwarz.

Gruß

[Iris]

Hallo,

ich habe mir die Datei angeschaut, aber leider keinen Fehler gefunden.

Hier mal der Code:

Code: PHP  [Auswählen]

function credit_selection() {
                $selection_string = '';
                $close_string = '';
                $credit_class_string = '';
                if (MODULE_ORDER_TOTAL_INSTALLED) {
                     // BOF - vr - 2010-03-03 fix gv display on checkout
                        /*$header_string = '<tr>'."\n";
                        $header_string .= '   <td><table border="0" width="100%" cellspacing="0" cellpadding="2">'."\n";
                        $output1_string .= '      <tr>'."\n";
                        $header_string .= '        <td class="main"><strong>'.TABLE_HEADING_CREDIT.'</strong></td>'."\n";
                        $header_string .= '      </tr>'."\n";
                        $header_string .= '    </table></td>'."\n";
                        $header_string .= '  </tr>'."\n";
                        $header_string .= '<tr>'."\n";
                        $header_string .= '   <td><table border="0" width="100%" cellspacing="1" cellpadding="2" class="infoBox">'."\n";
                        $header_string .= '     <tr class="infoBoxContents"><td><table border="0" width="100%" cellspacing="0" cellpadding="2">'."\n";
                        $header_string .= '       <tr><td width="10">'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>'."\n";
                        $header_string .= '           <td colspan="2"><table border="0" width="100%" cellspacing="0" cellpadding="2">'."\n";
                        $close_string = '                           </table></td>';
                        $close_string .= '<td width="10">'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>';
                        $close_string .= '</tr></table></td></tr></tr></td>';
                        $close_string .= '<tr><td width="100%">'.xtc_draw_separator('pixel_trans.gif', '100%', '10').'</td></tr>';*/
                       
                        $header_string  = '<tr><td>';
                        $header_string .= '<table class="paymentblock" border="0" width="100%" cellspacing="0" cellpadding="6">';
                        $header_string .= '<td width="90%" class="header">'.TABLE_HEADING_CREDIT.'</td>';
                        $header_string .= '<td width="10%" class="header"></td>';
                        $header_string .= '</table>';
                        $header_string .= '</td></tr>';
                        $close_string .= '<tr><td width="100%">'.xtc_draw_separator('pixel_trans.gif', '100%', '10').'</td></tr>';
                        // EOF - vr - 2010-03-03 fix gv display on checkout
                       
                        reset($this->modules);
                        $output_string = '';
                        while (list (, $value) = each($this->modules)) {
                                $class = substr($value, 0, strrpos($value, '.'));
        if ($GLOBALS[$class]->enabled && isset($GLOBALS[$class]->credit_class) && $GLOBALS[$class]->credit_class) {
          $use_credit_string = $GLOBALS[$class]->use_credit_amount();
          if ($selection_string == '') {
            $selection_string = $GLOBALS[$class]->credit_selection();
          }
                                        if (($use_credit_string != '') || ($selection_string != '')) {
                                                $output_string .= '<tr colspan="4"><td colspan="4" width="100%">'.xtc_draw_separator('pixel_trans.gif', '100%', '10').'</td></tr>';
                                                $output_string .= '<tr class="moduleRow" onmouseover="rowOverEffect(this)" onmouseout="rowOutEffect(this)" >';
                                                $output_string .= '<td width="10">'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>';
                                                // BOF - vr - 2010-03-03 fix gv display on checkout
                                                // $output_string .='<td class="main"><strong>'.$GLOBALS[$class]->header.'</strong></td>';
                                                // EOF - vr - 2010-03-03 fix gv display on checkout
                                                $output_string .=  $use_credit_string;
                                                $output_string .= '<td width="10">'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>';
                                                $output_string .= '</tr>'."\n";
                                                $output_string .= $selection_string;
                                        }
                                }
                        }
                        if ($output_string != '') {
                                $output_string = $header_string.$output_string;
                                $output_string .= $close_string;
                        }
                }
                return $output_string;
        }

        //            if ($selection_string !='') {
        //              $output_string .= '</td>' . "\n";
        //              $output_string .= $selection_string;
        //            }

        // update_credit_account is called in checkout process on a per product basis. It's purpose
        // is to decide whether each product in the cart should add something to a credit account.
        // e.g. for the Gift Voucher it checks whether the product is a Gift voucher and then adds the amount
        // to the Gift Voucher account.
        // Another use would be to check if the product would give reward points and add these to the points/reward account.
        //
        function update_credit_account($i) {
                if (MODULE_ORDER_TOTAL_INSTALLED) {
                        reset($this->modules);
                        while (list (, $value) = each($this->modules)) {
                                $class = substr($value, 0, strrpos($value, '.'));
                                if (($GLOBALS[$class]->enabled && isset($GLOBALS[$class]->credit_class) && $GLOBALS[$class]->credit_class)) {
          $GLOBALS[$class]->update_credit_account($i);
        }
                        }
                }
        }
        // This function is called in checkout confirmation.
        // It's main use is for credit classes that use the credit_selection() method. This is usually for
        // entering redeem codes(Gift Vouchers/Discount Coupons). This function is used to validate these codes.
        // If they are valid then the necessary actions are taken, if not valid we are returned to checkout payment
        // with an error
        //
        function collect_posts() {
    if (MODULE_ORDER_TOTAL_INSTALLED) {
      reset($this->modules);
      while (list (, $value) = each($this->modules)) {
        $class = substr($value, 0, strrpos($value, '.'));
        if (($GLOBALS[$class]->enabled && isset($GLOBALS[$class]->credit_class) && $GLOBALS[$class]->credit_class)) {
          $post_var = 'c'.$GLOBALS[$class]->code;
          if (isset($_POST[$post_var]) && $_POST[$post_var]) {
            $_SESSION[$post_var] = $_POST[$post_var];
          }
          $GLOBALS[$class]->collect_posts();
        }
      }
    }
  }
        // pre_confirmation_check is called on checkout confirmation. It's function is to decide whether the
        // credits available are greater than the order total. If they are then a variable (credit_covers) is set to
        // true. This is used to bypass the payment method. In other words if the Gift Voucher is more than the order
        // total, we don't want to go to paypal etc.
        //
        function pre_confirmation_check() {
                global $order;
                if (MODULE_ORDER_TOTAL_INSTALLED) {
                        $total_deductions = 0;
                        reset($this->modules);
                        $order_total = $order->info['total'];
                        while (list (, $value) = each($this->modules)) {
                                $class = substr($value, 0, strrpos($value, '.'));
                                $order_total = $this->get_order_total_main($class, $order_total);
                                if (($GLOBALS[$class]->enabled && isset($GLOBALS[$class]->credit_class) && $GLOBALS[$class]->credit_class)) {
                                        $total_deductions = $total_deductions + $GLOBALS[$class]->pre_confirmation_check($order_total);
                                        $order_total = $order_total - $GLOBALS[$class]->pre_confirmation_check($order_total);
                                }
                        }
                        if ($order->info['total'] - $total_deductions <= 0) {
                                $_SESSION['credit_covers'] = true;
                        } else { // belts and suspenders to get rid of credit_covers variable if it gets set once and they put something else in the cart
                                unset ($_SESSION['credit_covers']);
                        }
                }
        }
        // this function is called in checkout process. it tests whether a decision was made at checkout payment to use
        // the credit amount be applied aginst the order. If so some action is taken. E.g. for a Gift voucher the account
        // is reduced the order total amount.
        //
        function apply_credit() {
                if (MODULE_ORDER_TOTAL_INSTALLED) {
                        reset($this->modules);
                        while (list (, $value) = each($this->modules)) {
                                $class = substr($value, 0, strrpos($value, '.'));
                                if (($GLOBALS[$class]->enabled && isset($GLOBALS[$class]->credit_class) && $GLOBALS[$class]->credit_class)) {
                                        $GLOBALS[$class]->apply_credit();
                                }
                        }
                }
        }
        // Called in checkout process to clear session variables created by each credit class module.
        //
        function clear_posts() {
                if (MODULE_ORDER_TOTAL_INSTALLED) {
                        reset($this->modules);
                        while (list (, $value) = each($this->modules)) {
                                $class = substr($value, 0, strrpos($value, '.'));
                                if (($GLOBALS[$class]->enabled && isset($GLOBALS[$class]->credit_class) && $GLOBALS[$class]->credit_class)) {
                                        $post_var = 'c'.$GLOBALS[$class]->code;
                                        unset ($_SESSION[$post_var]);
                                }
                        }
                }
        }
        // Called at various times. This function calulates the total value of the order that the
        // credit will be appled aginst. This varies depending on whether the credit class applies
        // to shipping & tax
        //
        function get_order_total_main($class, $order_total) {
                global $credit, $order;
                //      if ($GLOBALS[$class]->include_tax == 'false') $order_total=$order_total-$order->info['tax'];
                //      if ($GLOBALS[$class]->include_shipping == 'false') $order_total=$order_total-$order->info['shipping_cost'];
                return $order_total;
        }
        // ICW ORDER TOTAL CREDIT CLASS/GV SYSTEM - END ADDITION
        // GV Code End

        // class constructor
        function order_total() {
                if (defined('MODULE_ORDER_TOTAL_INSTALLED') && xtc_not_null(MODULE_ORDER_TOTAL_INSTALLED)) {
                        $this->modules = explode(';', MODULE_ORDER_TOTAL_INSTALLED);
                        $modules = $this->modules;
                        sort($modules); // cgoenner: we need to include the ot_coupon & ot_gv BEFORE ot_tax
                        reset($modules);
                        while (list (, $value) = each($modules)) {
                                include (DIR_WS_LANGUAGES.$_SESSION['language'].'/modules/order_total/'.$value);
                                include (DIR_WS_MODULES.'order_total/'.$value);

                                $class = substr($value, 0, strrpos($value, '.'));
                                $GLOBALS[$class] = new $class ();
                        }
                        unset($modules);
                }
        }

        function process() {
                $order_total_array = array ();
                if (is_array($this->modules)) {
                        reset($this->modules);
                        while (list (, $value) = each($this->modules)) {
                                $class = substr($value, 0, strrpos($value, '.'));
                                if ($GLOBALS[$class]->enabled) {
          $GLOBALS[$class]->output = array(); //DokuMan - 2011-09-29 - calling $order_total_modules->process() multiple times would duplicate the output data due to the order total modules already being instantiated
                                        $GLOBALS[$class]->process();

                                        for ($i = 0, $n = sizeof($GLOBALS[$class]->output); $i < $n; $i ++) {
                                                if (xtc_not_null($GLOBALS[$class]->output[$i]['title']) && xtc_not_null($GLOBALS[$class]->output[$i]['text'])) {
                                                        $order_total_array[] = array ('code' => $GLOBALS[$class]->code,
                                            'title' => $GLOBALS[$class]->output[$i]['title'],
                                            'text' => $GLOBALS[$class]->output[$i]['text'],
                                            'value' => $GLOBALS[$class]->output[$i]['value'],
                                            'sort_order' => $GLOBALS[$class]->sort_order
                                            );
                                                }
                                        }
                                }
                        }
                }

                return $order_total_array;
        }

        function output() {
                $output_string = '';
                if (is_array($this->modules)) {
                        reset($this->modules);
                        while (list (, $value) = each($this->modules)) {
                                $class = substr($value, 0, strrpos($value, '.'));
                                if ($GLOBALS[$class]->enabled) {
                                        $size = sizeof($GLOBALS[$class]->output);
                                        for ($i = 0; $i < $size; $i ++) {
                                                $output_string .= '              <tr>'."\n".'                <td align="right" class="main">'.$GLOBALS[$class]->output[$i]['title'].'</td>'."\n".'                <td align="right" class="main">'.$GLOBALS[$class]->output[$i]['text'].'</td>'."\n".'              </tr>';
                                        }
                                }
                        }
                }

                return $output_string;
        }
}
?>

Ich hoffe, das hilft, um den Fehler zu finden?

LG Iris

[Tomcraft]

Das musst du dir mit Firebug direkt im Quelltext der Seite anschauen, siehe dazu: FAQ: Liste wichtiger Freeware- & OpenSource-Programme für die Shop-Erstellung

Grüße

Torsten

[jannemann]

Hallo Iris,

was passiert denn, wenn du das Standard Template aktivierst?
Passt dann der Abstand?

Schöne Grüße,
Jan

[Iris]

Hallo Jan,

danke für den Hinweis.

Ich habe es gleich getestet und ja, beim Standard-Template ist alles richtig.

Kannst du mir sagen, in welcher Datei ich jetzt nach dem Fehler suchen muss?

[Tomcraft]

Der Fehler begrenzt sich dann auf die "checkout_payment.html" bzw. "checkout_payment_block.html" bzw. "stylesheet.css".

Grüße

Torsten

[Iris]

vielen Dank 

[lurchi007]

Hallo,

kann mir jemand dabei helfen den Gutschein-Link, den der Kunde per Mail erhält zu ändern.
Sprich wo kann ich das bewerkstelligen, das der Kunde nicht im Warenkorb landet.
Das der Kunde nicht vorab angemeldet ist, sollte realistisch sein.
Gibt es da eine bessere Lösung?