View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003931 | OXID eShop (all versions) | 1.01. Products (product, categories, manufacturer, promotions etc.) | public | 2012-04-25 22:15 | 2012-12-07 14:27 |
| Reporter | jkrug | Assigned To | |||
| Priority | high | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | won't fix | ||
| Product Version | 4.5.10 revision 44222 | ||||
| Summary | 0003931: remove the eval(this.innerHTML); | ||||
| Description | After you select a variant and this is loaded via AJAX, in oxajax.js this (eval(this.innerHTML);) is executed and fails mostly every time, if you forgot to remove some calls if template was changed! Please use .live() / .on() from jQuery to solve that in a clean and easy way! Write less - do more! | ||||
| Tags | Products | ||||
| Theme | Azure | ||||
| Browser | All | ||||
| PHP Version | any | ||||
| Database Version | any | ||||
|
|
Please forget .live() Will be bound if Element is removed from DOM. Use .on() to solve that. Anyway, .live() is depricated in jQuery since version 1.7. Thanks to Daniel Schlichtholz. |
|
|
Also jQuery in latest version is still 1.5.1, which also prevents to use "on" for own extensions. |
|
|
Hi, well this place checks if given javascript is valid, if not - it removes it and throws error meesage to console, if its available, so you could see what should be fixed. This is the correct behavior. We have updated jquery to 1.8.2 and jquery ui to 1.9.1 and will release it in 5.1.0. |
