Viele kleine Probleme, brauche "Denkanschubser"

[expanse]

Hallo,

wie ich heute schon einmal in einem anderen Post geschrieben habe, ist der Shop, den ich erstelle, auf die richtige URL umgezogen.

Jetzt fallen mir aber noch unmengen an kleinen(?) Fehler auf und ich weiß es nicht wo ich anfangen soll zu suchen. Ich habe die Version: modified eCommerce Shopsoftware v1.05 dated: 2010-07-18 SP1a

Zum Einen kann ich die Rechnungsadresse nicht mehr ändern. Klicke ich auf "ändern" kommt keine Fehlermeldung, aber auch nichts anderes, als der blaue Hintergrund. Ich habe auch schon in der  checkout_psayment_address.php geschaut, aber dort habe ich keine Änderungen vorgenommen?  Ich habe heute nur noch so alles eingestellt, dass die Daten vom Shop  in der Lieferadresse  und bestellschein sind etc. Wo könnte denn hier der Fehler liegen bzw. in welcher php?

Zum Anderen bekommen Kunden, die sich neu anmelden auch nur eine leere Seite (allerdings wenn man dann auf "zurück" geht oben im Browser sind sie angemeldet. Aber eine Anmelde E-Mail bekommen sie auch nicht.

Oh man, ich habe mich soo gefreut, war so stolz die "Slideshow" und "ohne Versandkosten" etc. hinzubekommen und jetzt hänge ich in den letzten Zügen und bleibe stecken

Kann mir jemand helfen?  Hier mal die Seite. Ihr könnt euch auch gerne mal anmelden, ich lösch es dann später (das funktioniert noch ^^)  http://www.medialmed.de/

Liebe Grüße

Steffi

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

[expanse]

Hier auch mal die checkout_payment_address.php, aber dort habe ich keinen Fehler gefunden (aber das heißt ja nichts ^^)

Nachtrag: Und als Krönchen obendrauf: Nach einer Bestellung kommt auch ein weißes Fenster und keine E-Mail an den Kunden

Code: PHP  [Auswählen]

<?php
/* -----------------------------------------------------------------------------------------
   $Id: checkout_payment_address.php 1607 2011-01-04 16:18:09Z Tomcraft1980 $

   modified eCommerce Shopsoftware - community made shopping
   http://www.modified-shop.org

   Copyright (c) 2010 modified eCommerce Shopsoftware
   -----------------------------------------------------------------------------------------
   based on:
   (c) 2000-2001 The Exchange Project  (earlier name of osCommerce)
   (c) 2002-2003 osCommerce(checkout_payment_address.php,v 1.13 2003/05/27); www.oscommerce.com
   (c) 2003 nextcommerce (checkout_payment_address.php,v 1.14 2003/08/17); www.nextcommerce.org
   (c) 2006 XT-Commerce (checkout_payment_address.php 993 2005-07-06)

   Released under the GNU General Public License
   ---------------------------------------------------------------------------------------*/

include ('includes/application_top.php');
// create smarty elements
$smarty = new Smarty;
// include boxes
require (DIR_FS_CATALOG.'templates/'.CURRENT_TEMPLATE.'/source/boxes.php');
// include needed functions
require_once (DIR_FS_INC.'xtc_count_customer_address_book_entries.inc.php');
require_once (DIR_FS_INC.'xtc_address_label.inc.php');

// if the customer is not logged on, redirect them to the login page
if (!isset ($_SESSION['customer_id'])) {
  xtc_redirect(xtc_href_link(FILENAME_LOGIN, '', 'SSL'));
}
// if there is nothing in the customers cart, redirect them to the shopping cart page
if ($_SESSION['cart']->count_contents() < 1) {
  xtc_redirect(xtc_href_link(FILENAME_SHOPPING_CART));
}
$error = false;
$process = false;
if (isset ($_POST['action']) && ($_POST['action'] == 'submit')) {
  // process a new billing address
  if (xtc_not_null($_POST['firstname']) && xtc_not_null($_POST['lastname']) && xtc_not_null($_POST['street_address'])) {
    $process = true;

    if (ACCOUNT_GENDER == 'true') {
      $gender = xtc_db_prepare_input($_POST['gender']);
    }
    if (ACCOUNT_COMPANY == 'true') {
      $company = xtc_db_prepare_input($_POST['company']);
    }
    $firstname = xtc_db_prepare_input($_POST['firstname']);
    $lastname = xtc_db_prepare_input($_POST['lastname']);
    $street_address = xtc_db_prepare_input($_POST['street_address']);
    if (ACCOUNT_SUBURB == 'true') {
      $suburb = xtc_db_prepare_input($_POST['suburb']);
    }
    $postcode = xtc_db_prepare_input($_POST['postcode']);
    $city = xtc_db_prepare_input($_POST['city']);
    $country = xtc_db_prepare_input($_POST['country']);
    if (ACCOUNT_STATE == 'true') {
      $zone_id = xtc_db_prepare_input($_POST['zone_id']);
      $state = xtc_db_prepare_input($_POST['state']);
    }

    if (ACCOUNT_GENDER == 'true') {
      if (($gender != 'm') && ($gender != 'f')) {
        $error = true;

        $messageStack->add('checkout_address', ENTRY_GENDER_ERROR);
      }
    }

    if (strlen($firstname) < ENTRY_FIRST_NAME_MIN_LENGTH) {
      $error = true;

      $messageStack->add('checkout_address', ENTRY_FIRST_NAME_ERROR);
    }

    if (strlen($lastname) < ENTRY_LAST_NAME_MIN_LENGTH) {
      $error = true;

      $messageStack->add('checkout_address', ENTRY_LAST_NAME_ERROR);
    }

    if (strlen($street_address) < ENTRY_STREET_ADDRESS_MIN_LENGTH) {
      $error = true;

      $messageStack->add('checkout_address', ENTRY_STREET_ADDRESS_ERROR);
    }

    if (strlen($postcode) < ENTRY_POSTCODE_MIN_LENGTH) {
      $error = true;

      $messageStack->add('checkout_address', ENTRY_POST_CODE_ERROR);
    }

    if (strlen($city) < ENTRY_CITY_MIN_LENGTH) {
      $error = true;

      $messageStack->add('checkout_address', ENTRY_CITY_ERROR);
    }

    if (ACCOUNT_STATE == 'true') {
      $zone_id = 0;
      $check_query = xtc_db_query("select count(*) as total from ".TABLE_ZONES." where zone_country_id = '".(int) $country."'");
      $check = xtc_db_fetch_array($check_query);
      $entry_state_has_zones = ($check['total'] > 0);
      if ($entry_state_has_zones == true) {
        $zone_query = xtc_db_query("select distinct zone_id from ".TABLE_ZONES." where zone_country_id = '".(int) $country."' and (zone_name like '".xtc_db_input($state)."%' or zone_code like '%".xtc_db_input($state)."%')");
        if (xtc_db_num_rows($zone_query) > 1) {
          $zone_query = xtc_db_query("select distinct zone_id from ".TABLE_ZONES." where zone_country_id = '".(int) $country."' and zone_name = '".xtc_db_input($state)."'");
        }
        if (xtc_db_num_rows($zone_query) >= 1) {
          $zone = xtc_db_fetch_array($zone_query);
          $zone_id = $zone['zone_id'];
        } else {
          $error = true;

          $messageStack->add('create_account', ENTRY_STATE_ERROR_SELECT);
        }
      } else {
        if (strlen($state) < ENTRY_STATE_MIN_LENGTH) {
          $error = true;

          $messageStack->add('checkout_address', ENTRY_STATE_ERROR);
        }
      }
    }

    if ((is_numeric($country) == false) || ($country < 1)) {
      $error = true;

      $messageStack->add('checkout_address', ENTRY_COUNTRY_ERROR);
    }

    if ($error == false) {
      $sql_data_array = array (
      'customers_id' => $_SESSION['customer_id'],
      'entry_firstname' => $firstname,
      'entry_lastname' => $lastname,
      'entry_street_address' => $street_address,
      'entry_postcode' => $postcode,
      'entry_city' => $city,
      'entry_country_id' => $country
      );

      if (ACCOUNT_GENDER == 'true') {
        $sql_data_array['entry_gender'] = $gender;
      }
      if (ACCOUNT_COMPANY == 'true') {
        $sql_data_array['entry_company'] = $company;
      }
      if (ACCOUNT_SUBURB == 'true') {
        $sql_data_array['entry_suburb'] = $suburb;
      }
      if (ACCOUNT_STATE == 'true') {
        if ($zone_id > 0) {
          $sql_data_array['entry_zone_id'] = $zone_id;
          $sql_data_array['entry_state'] = '';
        } else {
          $sql_data_array['entry_zone_id'] = '0';
          $sql_data_array['entry_state'] = $state;
        }
      }

      xtc_db_perform(TABLE_ADDRESS_BOOK, $sql_data_array);

      $_SESSION['billto'] = xtc_db_insert_id();

      if (isset ($_SESSION['payment'])) {
        unset ($_SESSION['payment']);
      }

      xtc_redirect(xtc_href_link(FILENAME_CHECKOUT_PAYMENT, '', 'SSL'));
    }
    // process the selected billing destination
  }
  elseif (isset ($_POST['address'])) {
    $reset_payment = false;
    if (isset ($_SESSION['billto'])) {
      if ($billto != $_POST['address']) {
        if (isset ($_SESSION['payment'])) {
          $reset_payment = true;
        }
      }
    }

    $_SESSION['billto'] = (int)$_POST['address']; //DokuMan - 2010-12-17 - added xtc_db_prepare_input / franky_n - 2010-12-27 corrected to(int)

    $check_address_query = xtc_db_query("select count(*) as total from ".TABLE_ADDRESS_BOOK." where customers_id = '".(int)$_SESSION['customer_id']."' and address_book_id = '".(int)$_SESSION['billto']."'"); //DokuMan - 2010-12-17 - added int-typecasting
    $check_address = xtc_db_fetch_array($check_address_query);

    if ($check_address['total'] == '1') {
      if ($reset_payment == true) {
        unset ($_SESSION['payment']);
      }
      xtc_redirect(xtc_href_link(FILENAME_CHECKOUT_PAYMENT, '', 'SSL'));
    } else {
      unset ($_SESSION['billto']);
    }
    // no addresses to select from - customer decided to keep the current assigned address
  } else {
    $_SESSION['billto'] = $_SESSION['customer_default_address_id'];

    xtc_redirect(xtc_href_link(FILENAME_CHECKOUT_PAYMENT, '', 'SSL'));
  }
}

// if no billing destination address was selected, use their own address as default
if (!isset ($_SESSION['billto'])) {
  $_SESSION['billto'] = $_SESSION['customer_default_address_id'];
}

$breadcrumb->add(NAVBAR_TITLE_1_PAYMENT_ADDRESS, xtc_href_link(FILENAME_CHECKOUT_PAYMENT, '', 'SSL'));
$breadcrumb->add(NAVBAR_TITLE_2_PAYMENT_ADDRESS, xtc_href_link(FILENAME_CHECKOUT_PAYMENT_ADDRESS, '', 'SSL'));

$addresses_count = xtc_count_customer_address_book_entries();
require (DIR_WS_INCLUDES.'header.php');

$smarty->assign('FORM_ACTION', xtc_draw_form('checkout_address', xtc_href_link(FILENAME_CHECKOUT_PAYMENT_ADDRESS, '', 'SSL'), 'post', 'onsubmit="return check_form_optional(checkout_address);"'));

if ($messageStack->size('checkout_address') > 0) {
  $smarty->assign('error', $messageStack->output('checkout_address'));
}

if ($process == false) {
  $smarty->assign('ADDRESS_LABEL', xtc_address_label($_SESSION['customer_id'], $_SESSION['billto'], true, ' ', '<br />'));

  if ($addresses_count > 1) {
//BOF - Dokuman - 2009-08-21 - Better layout on multiple shipping/billing addresses
/*
    $address_content = '<table border="0" width="100%" cellspacing="0" cellpadding="0">';
    $radio_buttons = 0;

    $addresses_query = xtc_db_query("select address_book_id,
                                            entry_firstname as firstname,
                                            entry_lastname as lastname,
                                            entry_company as company,
                                            entry_street_address as street_address,
                                            entry_suburb as suburb,
                                            entry_city as city,
                                            entry_postcode as postcode,
                                            entry_state as state,
                                            entry_zone_id as zone_id,
                                            entry_country_id as country_id
                                     from ".TABLE_ADDRESS_BOOK."
                                     where customers_id = '".$_SESSION['customer_id']."'");
    while ($addresses = xtc_db_fetch_array($addresses_query)) {
//BOF - Dokuman - 2009-08-19 - BUGFIX: #0000221 addressformat error in payment/shipping addresses
    //$format_id = xtc_get_address_format_id($address['country_id']);
      $format_id = xtc_get_address_format_id($addresses['country_id']);
//EOF - Dokuman - 2009-08-19 - BUGFIX: #0000221 addressformat error in payment/shipping addresses
      $address_content .= ' <tr>
                      <td>'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>
                      <td colspan="2"><table border="0" width="100%" cellspacing="0" cellpadding="2">
                      ';
      if ($addresses['address_book_id'] == $_SESSION['billto']) {
        $address_content .= '                  <tr id="defaultSelected" class="moduleRowSelected" onmouseover="rowOverEffect(this)" onmouseout="rowOutEffect(this)" onclick="selectRowEffect(this, '.$radio_buttons.')">'."\n";
      } else {
        $address_content .= '                  <tr class="moduleRow" onmouseover="rowOverEffect(this)" onmouseout="rowOutEffect(this)" onclick="selectRowEffect(this, '.$radio_buttons.')">'."\n";
      }
      $address_content .= '
                          <td>'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>
                          <td class="main" colspan="2"><strong>'.$addresses['firstname'].' '.$addresses['lastname'].'</strong></td>
                          <td class="main" align="right">'.xtc_draw_radio_field('address', $addresses['address_book_id'], ($addresses['address_book_id'] == $_SESSION['billto'])).'</td>
                          <td>'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>
                        </tr>
                        <tr>
                          <td>'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>
                          <td colspan="3"><table border="0" cellspacing="0" cellpadding="2">
                            <tr>
                              <td>'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>
                              <td class="main">'.xtc_address_format($format_id, $addresses, true, ' ', ', ').'</td>
                              <td>'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>
                            </tr>
                          </table></td>
                          <td>'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>
                        </tr>
                      </table></td>
                      <td>'.xtc_draw_separator('pixel_trans.gif', '10', '1').'</td>
                    </tr>';

      $radio_buttons ++;
    }
    $address_content .= '</table>';
*/
    $address_content = '<ol id="address_block">';
    $radio_buttons = 0;

    $addresses_query = xtc_db_query("select address_book_id,
                                            entry_firstname as firstname,
                                            entry_lastname as lastname,
                                            entry_company as company,
                                            entry_street_address as street_address,
                                            entry_suburb as suburb,
                                            entry_city as city,
                                            entry_postcode as postcode,
                                            entry_state as state,
                                            entry_zone_id as zone_id,
                                            entry_country_id as country_id
                                       from ".TABLE_ADDRESS_BOOK."
                                       where customers_id = '".(int)$_SESSION['customer_id']."'"); //DokuMan - 2010-12-17 - added int-typecasting
    while ($addresses = xtc_db_fetch_array($addresses_query)) {
      $format_id = xtc_get_address_format_id($addresses['country_id']);

      // BOF - Tomcraft - 2011-01-04 - make checkout process valid
      //$address_content .= sprintf('<li>%s<label for="field_addresses_%s"> %s %s</label><br /><span class="address">%s</span></li>', xtc_draw_radio_field('address',$addresses['address_book_id'], ($addresses['address_book_id'] == $_SESSION['sendto'])), $addresses['address_book_id'], $addresses['firstname'], $addresses['lastname'],xtc_address_format($format_id, $addresses, true, ' ', ', '));
      $address_content .= sprintf('<li>%s<label for="field_addresses_%s"> %s %s</label><br /><span class="address">%s</span></li>', xtc_draw_radio_field('address',$addresses['address_book_id'], ($addresses['address_book_id'] == $_SESSION['sendto']), 'id="field_addresses_'.$addresses['address_book_id'].'"'), $addresses['address_book_id'], $addresses['firstname'], $addresses['lastname'], xtc_address_format($format_id, $addresses, true, ' ', ', '));
      // EOF - Tomcraft - 2011-01-04 - make checkout process valid
      $radio_buttons ++;
    }
    $address_content .= '</ol>';
//EOF - Dokuman - 2009-08-21 - Better layout on multiple shipping/billing addresses

    $smarty->assign('BLOCK_ADDRESS', $address_content);

  }
}

if ($addresses_count < MAX_ADDRESS_BOOK_ENTRIES) {
  require (DIR_WS_MODULES.'checkout_new_address.php');
}
$smarty->assign('BUTTON_CONTINUE', xtc_draw_hidden_field('action', 'submit').xtc_image_submit('button_continue.gif', IMAGE_BUTTON_CONTINUE));

if ($process == true) {
  $smarty->assign('BUTTON_BACK', '<a href="'.xtc_href_link(FILENAME_CHECKOUT_PAYMENT_ADDRESS, '', 'SSL').'">'.xtc_image_button('button_back.gif', IMAGE_BUTTON_BACK).'</a>');
}
$smarty->assign('FORM_END', '</form>');
$smarty->assign('language', $_SESSION['language']);
$main_content = $smarty->fetch(CURRENT_TEMPLATE.'/module/checkout_payment_address.html');
$smarty->assign('main_content', $main_content);
$smarty->caching = 0;
if (!defined('RM')) {
  $smarty->load_filter('output', 'note');
}
$smarty->display(CURRENT_TEMPLATE.'/index.html');
include ('includes/application_bottom.php');
?>

[Gradler]

Warum nimmst Du nicht eine frische 1.06, da denke ich können Dir dann schneller wesentlich mehr Leute helfen als bei einer total veralteten 1.05sp1b.

Gerd

[expanse]

Tut mir leid, ich habe nicht so viel Ahnung davon. Und ich habe die version installiert, was direkt auf meinem Hoster bereit gestellt wurde.
Wie installiere ich denn die neue Version und wäre das ein Risikofaktor für die Seite?
Liebe Grüße

[Gradler]

Kommt immer drauf an, was Du bereits eingestellt hast und wieviele zusätzliche Erweiterungen eingepaßt (funktionierend!) wurde. Der Aufwand ist zwar groß, daber der Stand ist dann halt aktuell und jeder der neu anfängt wird nur noch die 1.06 hier downloaden können. So werden dann voraussichtlich die meisten Probleme und Fragen diese Version betreffen.

Gerd

[expanse]

Also eine neuinstallation ist mir grade (noch) zu unsicher. Bevor wieder alles kaputt ist, das kann ich mir wegen dem Kunde nicht leisten Trotzdem danke, ich werde auch darauf zurückkommen. Ich wusste nicht einmal, dass ich eine alte Version habe 
Ich bin jetzt grade auch ein Schritt weiter: Auf der Subdomain, wo die Seite vorher war, läuft auch alles einwandfrei. Deshalb muss es an dem Umzug liegen, dass irgendein Pfad noch nicht stimmt. Vielleicht geht mir jetzt ein Licht auf...^^

LG

[Gradler]

Oh weija ... Gerade mal, wenn man sehr wenig Ahnung hat sollt man das gerade eben nicht für Kunden mit noch weniger Ahnung machen. Hast Du denn wenigstens den Sicherheitsfix und die Buttonlösung integriert?

Gerd

[expanse]

Hey,
ja, Buttonlösung, Sicherheitsfix etc. alles gemacht.
Ich bin auch eigentlich nur für Drucktechnische Sachen zuständig gewesen, aber musste notgedrungen in html etc. einspringen (war nicht meine Idee^^).
An sich macht es mir Spaß, ich lerne das gerne und bisher klappt das auch. Wahrscheinlich auch nur, weil ich mich nicht mit halben Dingen zufrieden gebe und mich solange informiere, bis auch etwas klappt. Und ich erst am Shop rumprobiere, wnen es analog auf meinem Rechner klappt. Und ich denke mal, dass ich für den jetzigen Fehler auch nichts kann, mache ja immer Backups, habe alles kontrolliert und wie gesagt auf der alten Subdomain klappt alles, deswegen guckt grade der It-Fachmann nach, der den Umzug gemacht hat.

Liebe Grüße