Widerrufsrecht im Warenkorb

Mark0100 · 29. März 2015

Hallo,

ich habe mir gerade ein neues Kundenfeld angelegt und auch auch den Inhalt der tos.php in meine neue wir.php kopiert.

Was muss ich jetzt alles ändern damit beim Klick auf den Link auch der Inhalt der Seite Widerrufsbelehrung angezeigt wird. Wäre toll wenn das jemand hier ändern könnte.

Da sind so viele tos und terms-of-service drin, dass ich nicht weiss was ich da tue

hier der Code der neuen wir.php

HTML

<?php
/**
* field tos
*
* @package VirtueMart
* @subpackage Cart
* @author Max Milbers
* @link http://www.virtuemart.net
* @copyright Copyright (c) 2014 VirtueMart Team. All rights reserved.
* @license http://www.gnu.org/copyleft/gpl.html GNU/GPL2, see LICENSE.php
* @version $Id: cart.php 7682 2014-02-26 17:07:20Z Milbo $
*/
defined('_JEXEC') or die('Restricted access');
$_prefix = $viewData['prefix'];
$field = $viewData['field'];
$tos = $field['value'];
$app = JFactory::getApplication();
if($app->isSite()){
vmJsApi::popup('#full-tos','#terms-of-service');
if (!class_exists('VirtueMartCart')) require(VMPATH_SITE . DS . 'helpers' . DS . 'cart.php');
$cart = VirtuemartCart::getCart();
$cart->prepareVendor();
if(empty($tos) and !VmConfig::get ('agree_to_tos_onorder', true)){
if(is_array($cart->BT) and !empty($cart->BT['tos'])){
$tos = $cart->BT['tos'];
}
}
}
if(!class_exists('VmHtml')) require(VMPATH_ADMIN.DS.'helpers'.DS.'html.php');
echo VmHtml::checkbox ($_prefix.$field['name'], $tos, 1, 0, 'class="terms-of-service required"', 'tos');
if (VmConfig::get ('oncheckout_show_legal_info', 1) and $app->isSite()) {
?>
<div class="terms-of-service">
<label for="tos">
<a href="<?php echo JRoute::_ ('index.php?option=com_virtuemart&view=vendor&layout=tos&virtuemart_vendor_id=1', FALSE) ?>" class="terms-of-service" id="terms-of-service" rel="facebox"
target="_blank">
<span class="vmicon vm2-termsofservice-icon"></span>
<?php echo vmText::_ ('COM_VIRTUEMART_CART_WIR_READ_AND_ACCEPTED') ?>
</a>
</label>
<div id="full-tos">
<h2><?php echo vmText::_ ('COM_VIRTUEMART_CART_WIR') ?></h2>
<?php echo $cart->vendor->vendor_terms_of_service ?>
</div>
</div>
<?php
}
?>

Alles anzeigen

Milbo · 30. März 2015

Servus,

alle BT['tos'] in BT['wir'] um ändern. Die Variable $tos, ja macht sind die umzubenennen, müsste aber auch so gehen.

Dann sind es eigentlich nur die 2 Sprachschlüssel

Mark0100 · 30. März 2015

Hi Milbo,

schon mal Danke für Deine Hilfe,

aber es funktioniert noch nicht ganz. Das Popup Fenster geht zwar auf, aber es werden nur die AGBs angezeigt nicht das Widerrufsrecht.

Ich habe meine wir.php jetzt soweit geändert:

PHP

<?php
/**
* field tos
*
* @package VirtueMart
* @subpackage Cart
* @author Max Milbers
* @link http://www.virtuemart.net
* @copyright Copyright (c) 2014 VirtueMart Team. All rights reserved.
* @license http://www.gnu.org/copyleft/gpl.html GNU/GPL2, see LICENSE.php
* @version $Id: cart.php 7682 2014-02-26 17:07:20Z Milbo $
*/
defined('_JEXEC') or die('Restricted access');
$_prefix = $viewData['prefix'];
$field = $viewData['field'];
$tos = $field['value'];
$app = JFactory::getApplication();
if($app->isSite()){
vmJsApi::popup('#full-tos','#terms-of-service');
if (!class_exists('VirtueMartCart')) require(VMPATH_SITE . DS . 'helpers' . DS . 'cart.php');
$cart = VirtuemartCart::getCart();
$cart->prepareVendor();
if(empty($tos) and !VmConfig::get ('agree_to_tos_onorder', true)){
if(is_array($cart->BT) and !empty($cart->BT['wir'])){
$tos = $cart->BT['wir'];
}
}
}
if(!class_exists('VmHtml')) require(VMPATH_ADMIN.DS.'helpers'.DS.'html.php');
echo VmHtml::checkbox ($_prefix.$field['name'], $wir, 1, 0, 'class="terms-of-service required"', 'wir');
if (VmConfig::get ('oncheckout_show_legal_info', 1) and $app->isSite()) {
?>
<div class="terms-of-service">
<label for="tos">
<a href="<?php echo JRoute::_ ('index.php?option=com_content&view=article&id=5', FALSE) ?>" class="terms-of-service" id="terms-of-service" rel="facebox"
target="_blank">
<span class="vmicon vm2-termsofservice-icon"></span>
<?php echo vmText::_ ('COM_VIRTUEMART_CART_WIR_READ_AND_ACCEPTED') ?>
</a>
</label>
<div id="full-tos">
<h2><?php echo vmText::_ ('COM_VIRTUEMART_CART_WIR') ?></h2>
<?php echo $cart->vendor->vendor_terms_of_service ?>
</div>
</div>
<?php
}
?>

Alles anzeigen

soweit ich das sehe ist die ID in dieser Zeile für den Inhalt verantwortlich:

PHP

<a href="<?php echo JRoute::_ ('index.php?option=com_content&view=article&id=5', FALSE) ?>" class="terms-of-service" id="terms-of-service" rel="facebox"

aber was muss ich da eintragen, damit der Content aus Artikel id=5 auch angezeigt wird??
Beim Mouseover wird der Link richtig angezeigt, aber beim Anklicken wird das Häkchen bei den AGBs gesetzt *kopfkratz

und wenn ich id="terms-of-service" änder passiert gar nichts, also geht kein Popup auf.

Bin da ein wenig überfragt....sorry

Milbo · 30. März 2015

vmJsApi::popup('#full-tos','#terms-of-service');

also full-tos und terms-of-service auch überall ersetzen, damit das js greift. Könnt man optimaler schreiben,...

Mark0100 · 30. März 2015

Hi Milbo,

danke für Deine Geduld, aber ich hab da schon alles Mögliche durchprobiert, aber es will leider nicht.

Das liegt aber auch daran, dass ich nicht wirklich weiß was ich da tue

Könntest Du nicht die Datei komplett ändern?? bitte bitte bitte bitte.....

ich glaube, dass das auch im Sinne vieler anderer Benutzer von VM3 ist, eine fertige Lösung hier zu finden. Das Kundenfeld ist ja sehr einfach zu machen in VM, aber diese Datei ist schon nicht ohne... das ist nicht mal eben geändert...

Milbo · 1. April 2015

Hmmm, typischer Fall für die Membership. Support : VirtueMart Supporter Membership

Kutzi · 7. Juni 2016

PHP

/**
* Creates popup, fancy or other for TOS
*/
static function popup($container,$activator){
static $done = false;
if ($done) return true;
self::loadPopUpLib();
if(VmConfig::get('usefancy',1)) {
$exeL = "jQuery.fancybox ({ div: '".$container."', content: con });";
} else {
$exeL = "jQuery.facebox( { div: '".$container."' }, 'my-groovy-style');";
}
$box = "
jQuery(document).ready(function($) {
jQuery('div".$container."').hide();
var con = jQuery('div".$container."').html();
jQuery('a".$activator."').click(function(event) {
event.preventDefault();
".$exeL."
});
});
";
self::addJScript('box',$box);
$done = true;
return;
}

Alles anzeigen

Im Kopf der Funktion wird mit ...

PHP

static $done = false;
if ($done) return true;

... geprüft ob "done=true" ist. Darum kann man mit in der Kopie von "tos.php" z.B." "wb.php" zwar die Funktion "popup" der Klasse "vmJsApi" aufrufen es wird jedoch nur einmal ein Popup JS eingefügt. Ruft man die Funktion "popup" nun 2 mal in einer Datei oder eben auch in verschiedenen Dateien aber der gleiche View auf dann wird nur 1 Java Script erzeugt und hinzugefügt. Dies führt dazu das man keine 2 Popup Felder in einer View nutzen kann.

Beispiel:

PHP

vmJsApi::popup('#full-tos','#terms-of-service');
vmJsApi::popup('#full-tos-k','#terms-of-service-k');

Meine Lösungsvorschläge:

1) Bitte ändert den VM Code in den kommenden Versionen so ab das die Funktion "popup" in der PHP Klasse vmJsApi einen zusätzlichen Wert übergeben bekommt ...

PHP

static function popup($container,$activator,$js_id)

Die "$js_id" muss dann in der Funktion an das JQuery JS weitergegeben werden. Dann kann auch die "done=true" Abfrage raus.

2) Eine noch bessere Lösung ist wenn ihr die "vmJsApi" so ändert das zuerst alle "popup" JQuery Snippets der Felder gesammelt werden und dann nur ein variables JQuery Snippet geadded wird. Dann wird nicht für jedes Feld das ein Popup hat ein eigenes Js Snippet geadded und das spart Traffic und ist in Hinblick auf Perfomance schick.

Bei Lösung 1 wird in der Funktion "popup" dann ...

PHP

self::addJScript('box' ,$box);

zu

PHP

self::addJScript('box'.$js_id ,$box);

Im Quelltext der Seite wird dann ...

Code

<script id="box" type="text/javascript">//<=!=[=C=D=A=T=A=[
jQuery(document).ready(function($) {
jQuery('div#full-tos').hide();
var con = jQuery('div#full-tos').html();
jQuery('a#terms-of-service').click(function(event) {
event.preventDefault();
jQuery.fancybox ({ div: '#full-tos', content: con });
});
}); //]=]=>
</script>

zu

Code

<script id="box[B][U][COLOR='#FF0000']JEWEILIGEID[/COLOR][/U][/B]_js" type="text/javascript">//<=!=[=C=D=A=T=A=[
jQuery(document).ready(function($) {
jQuery('div#full-tos').hide();
var con = jQuery('div#full-tos').html();
jQuery('a#terms-of-service').click(function(event) {
event.preventDefault();
jQuery.fancybox ({ div: '#full-tos', content: con });
});
}); //]=]=>
</script>

Wie schon erwähnt ist Lösung 2 besser da man bei Lösung 1 je Feld ein JS Snippet pro Popup hat. Ein einziges Snippet mit anderem JQuery Code das mehrere Popups handeln kann ist wie gesagt schöner.

Ich poste hier mit Absicht keine Step by Step Lösung weil ich kein Freund von Änderungen am Code ohne die Entwickler bin. Diese gehen halt beim nächsten Update verloren.

Für alle die trotzdem eine schnelle Lösung suchen ...

a)
In der "wb.php" direkt die Funktion "addJS" von "vmJsApi" aufzurufen und das Jquery clonen und adden. Dabei ist darauf zu achten das es A) Keine DOM Konflikte bei den CSS Selektoren gibt und B) Die JS ID nicht kollidiert.

b) Ein override in der View der Cart zu machen und eigenes JQuery an der jeweiligen Stelle adden. Die richtige Stelle kann man ja über die DOM CSS Selektoren wählen und Joomla bietet ja neben dem VM auch eigene Funktionen zum adden von JS Code. Hier ist der Vorteil das dass Override auch beim Update erhalten bleibt.

Schöner wäre aber eine Lösung durch Euch wackere fleißige Entwickler