jQuery : Passing Variable Value From On Click Event Failed

Question

I have this javascript :

<?
var dropzone_maxfiles = '.$product_maxfiles.';

$( ".product_imgfiles" ).click(function(event) {
    event.preventDefault();
    var img_file = $( this ).children( "img" ).attr( "src" );

    $.ajax({
        url: "controller/ctrl.dropzonejs-cleaner.php?token='.$product_token.'&key='.$secret_token.'&imgfile="+img_file+""
    });

    $( this ).parent( "div" ).remove();                 
    dropzone_maxfiles = dropzone_maxfiles + 1;
});

var myDropzone = new Dropzone(".dropzone", { 
    url: "controller/ctrl.dropzonejs.php?token='.$product_token.'&key='.$secret_token.'",
    maxFilesize: 2,
    maxFiles: dropzone_maxfiles,
    acceptedFiles: ".jpeg, .jpg, .png, .gif"
});

myDropzone.on("complete", function() {
    var rejected = myDropzone.getRejectedFiles();

    if (rejected == "" ) {
        $(".btn-vendor-addnew").removeClass("disabled");
        $(".dropzone-error").css("display","none");
    }else {
        myDropzone.removeAllFiles(true);
        $(".btn-vendor-addnew").addClass("disabled");
        $(".dropzone-error").css("display","block");

        $.ajax({
            url: "controller/ctrl.dropzonejs-cleaner.php?token='.$product_token.'&key='.$secret_token.'"
        });
    }
});
?>

the problem is : dropzone_maxfiles = dropzone_maxfiles + 1; value doesn't exist in maxFiles: dropzone_maxfiles,

how to pass that dropzone_maxfiles variable from on click event into Dropzone maxFiles? thank you.

There also shouldn't be a trailing comma after the line maxFiles: dropzone_maxfiles — Jonathan Lam
– Jonathan Lam, Commented Apr 12, 2016 at 2:00
what you see is short version of the real case. I need PHP to get $product_maxfiles from PHP — Saint Robson
– Saint Robson, Commented Apr 12, 2016 at 2:00
@JonathanLam : actually there are more lines inside myDropzone. I just make it short and focus on the problem — Saint Robson
– Saint Robson, Commented Apr 12, 2016 at 2:01
Primitive values are not copied by "reference", so updating the variable at a later time doesn't update the object passed to the function, which is probably already consumed somehow by new Dropzone — adeneo
– adeneo, Commented Apr 12, 2016 at 2:03

Jonathan Lam · Accepted Answer · 2016-04-12 04:23:29Z

2

Like @adeneo said in the comments, the property maxFiles of the Dropzone object will not change automatically if the variable is changed.

Instead, try reinitializing it with the current variable on click, or perhaps just change that property value (I'm not sure exactly how Dropzone is implemented, so there might be some setter method to do this for you).

EDIT (upon request): a basic example:

$( ".product_imgfiles" ).click(function(event) {
    // ... event handling code          
    dropzone_maxfiles = dropzone_maxfiles + 1;

    // add this to "reset" the Dropzone
    myDropzone = new Dropzone(".dropzone", { 
        url: "controller/ctrl.dropzonejs.php?token='.$product_token.'&key='.$secret_token.'",
        maxFilesize: 2,
        maxFiles: dropzone_maxfiles,
        acceptedFiles: ".jpeg, .jpg, .png, .gif"
    });

});

edited Apr 12, 2016 at 4:23

answered Apr 12, 2016 at 2:05

Jonathan Lam

17.5k17 gold badges72 silver badges99 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

Saint Robson Over a year ago

Hi bro, I just updated my full code because I'm not sure which one is the constructor and setter. I'm still new in this jQuery world. usually in PHP, when the variable updated, then the rest of the code will get updated value as well. but not in jQuery. any idea how to solve this situation? thank you.

Jonathan Lam Over a year ago

@RobertHanson Ok, I'll provide the simplest example: to simply "reset" the variable with the updated value. Tell me if it works.

Saint Robson Over a year ago

bro, logically what you do is pretty make sense... but after I implement your code, the dropzone still read var myDropzone which generated by $(document).ready(function(){. this is really strange... why this reset trick is not working?

Jonathan Lam Over a year ago

@RobertHanson Silly me! Sorry, you don't need the var in the declaration -- this creates a local variable (so it update the global one).

Collectives™ on Stack Overflow

jQuery : Passing Variable Value From On Click Event Failed

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related