View Issue Details

IDProjectCategoryView StatusLast Update
0004534OXID eShop (all versions)1. ----- eShop frontend -----public2013-03-12 09:06
ReporterWBL_BjoernLange 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version 
Target VersionFixed in Version4.8.0_5.1.0_beta1 
Summary0004534: Wrong Input-Handling of oxajax.js
DescriptionTaken frmo 4.7b2:

ajax : function(activator, params) {
            var self = this;
            var inputs = {};
            var action = "";
            var type = "";
            if (activator[0].tagName == 'FORM') {
                $("input", activator).each(function() {
                    inputs[this.name] = this.value;
                });

If the value is for example a checkbox, not only the value is important but also, if the checkbox is checked or not.
TagsNo tags attached.
ThemeAzure
BrowserAll
PHP Versionany
Database Versionany

Activities

aurimas.gladutis

2012-11-28 15:14

reporter   ~0008015

Hi, added check for checkboxes like this:
$("input", activator).each(function() {
    if (this.type == 'checkbox' && !this.checked) return true;
    inputs[this.name] = this.value;
});

WBL_BjoernLange

2013-03-08 17:27

reporter   ~0008497

The Bug still does exist.

Taken from <http://demoshop.oxid-esales.com/community-edition/out/azure/src/js/widgets/oxajax.js?1359125455>:

if (activator[0].tagName == 'FORM') {
                $("input", activator).each(function() {
                    inputs[this.name] = this.value;
                });
                action = activator.attr("action");
                type = activator.attr("method");
            } else if (activator[0].tagName == 'A') {
                action = activator.attr("href");
            }


Support for selects would be nice aswell.

Linas Kukulskis

2013-03-12 09:06

reporter   ~0008499

it will be released with Next Major version 5.1