|
In-Portal Issue Tracker - In-Portal CMS
|
|||||||||||
| Viewing Issue Advanced Details | |||||||||||
|
|
|||||||||||
| ID: | Category: | Type: | Reproducibility: | Date Submitted: | Last Update: | ||||||
| 125 | [In-Portal CMS] Optimization | bug report | always | 2009-07-19 07:39 | 2009-10-03 07:56 | ||||||
|
|
|||||||||||
| Reporter: | alex | Platform: | |||||||||
| Assigned To: | alex | OS: | |||||||||
| Priority: | normal | OS Version: | |||||||||
| Status: | closed | Product Version: | 4.3.9 | ||||||||
| Product Build: | Resolution: | fixed | |||||||||
| ETA: | none | Fixed in Version: | 5.0.0 | ||||||||
| Reference: | |||||||||||
| Change Log Message: | |||||||||||
| Estimate Points: | 0 | ||||||||||
|
|
|||||||||||
| Summary: | 0000125: Javascript cycle "for (index in array)" is used for arrays - replace with normal "for (index = 0; i < array.length; i++)" | ||||||||||
| Description: |
Currently in some places arrays are accessed as objects in cycles: var arr = ['one', 'two']; for (var elem in arr) { alert( arr[elem] ); } This approach works until someone adds new methods to Array class declaration. I propose to change such approach to normal "for" cycle: var arr = ['one', 'two']; for (var elem = 0; i < arr.length; elem++) { alert( arr[elem] ); } |
||||||||||
| Steps To Reproduce: | |||||||||||
| Additional Information: | |||||||||||
| Relationships |
| ||||||||||
| Attached Files: | |||||||||||
|
|
|||||||||||
| Issue History | |||||||||||
| Date Modified | Username | Field | Change | ||||||||
| 2009-10-03 07:56 | administrator | Note Added: 0000649 | |||||||||
| 2009-10-03 07:56 | administrator | Status | resolved => closed | ||||||||
| 2009-08-04 09:21 | alex | Relationship added | child of 0000173 | ||||||||
| 2009-08-04 09:20 | alex | Changeset attached | 5.0.x r12163 | ||||||||
| 2009-07-19 08:17 | alex | Fixed in Version | => 5.0.0 | ||||||||
| 2009-07-19 08:17 | alex | Target Version | => 5.0.0 | ||||||||
| 2009-07-19 08:16 | alex | Note Added: 0000154 | |||||||||
| 2009-07-19 08:16 | alex | Status | active => resolved | ||||||||
| 2009-07-19 08:16 | alex | Resolution | open => fixed | ||||||||
| 2009-07-19 08:16 | alex | Assigned To | => alex | ||||||||
| 2009-07-19 08:16 | alex | Changeset attached | RC r11932 | ||||||||
| 2009-07-19 08:16 | alex | Summary | Javascript cycle "for (index in array)" is used for arrays - replace with normal "for (index = 0; i < array.length; i++) => Javascript cycle "for (index in array)" is used for arrays - replace with normal "for (index = 0; i < array.length; i++)" | ||||||||
| 2009-07-19 07:39 | alex | New Issue | |||||||||
| Notes | |||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||