[ 
https://issues.apache.org/jira/browse/WICKET-4868?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Thomas Heigl updated WICKET-4868:
---------------------------------

    Description: 
The upload progress bar never finishes its final loop because of the strict 
comparison operator "===".

{code}
var update = responseAsText.split('|');
var progressPercent = update[1];
var status = update[2];
...

if (progressPercent === 100) {
   Wicket.DOM.hide(this.statusid);
   Wicket.DOM.hide(this.barid);
} else {
   this.scheduleUpdate();
}
{code}

The {{progressPercent}} variable is a string, thus the strict comparison always 
fails. The string should either be parsed into a numeric value or compared as a 
string, e.g. progressPercent === "100".

  was:
The upload progress bar never finishes its final loop because of the strict 
comparison operator "===".

{code}
var update = responseAsText.split('|');
var progressPercent = update[1];
var status = update[2];

...

if (progressPercent === 100) {
        Wicket.DOM.hide(this.statusid);
        Wicket.DOM.hide(this.barid);
} else {
        this.scheduleUpdate();
}
{code}

The {{progressPercent}} variable is a string, thus the strict comparison always 
fails. The string should either be parsed into a numeric value or compared as a 
string, e.g. progressPercent === "100".

    
> UploadProgressBar never stops checking for updates
> --------------------------------------------------
>
>                 Key: WICKET-4868
>                 URL: https://issues.apache.org/jira/browse/WICKET-4868
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 6.2.0
>            Reporter: Thomas Heigl
>
> The upload progress bar never finishes its final loop because of the strict 
> comparison operator "===".
> {code}
> var update = responseAsText.split('|');
> var progressPercent = update[1];
> var status = update[2];
> ...
> if (progressPercent === 100) {
>    Wicket.DOM.hide(this.statusid);
>    Wicket.DOM.hide(this.barid);
> } else {
>    this.scheduleUpdate();
> }
> {code}
> The {{progressPercent}} variable is a string, thus the strict comparison 
> always fails. The string should either be parsed into a numeric value or 
> compared as a string, e.g. progressPercent === "100".

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to