In-Portal Issue Tracker

Welcome to the In-Portal Open Source CMS Issue Tracker! This is a central management / tracking tool for all types of tasks / issues / bugs for the In-Portal Project. Before reporting any issues, please make sure to read the Guide into Issue Tracker and How to Properly Test and Report Bugs!
Main | My View | View Issues | Change Log | Roadmap | Docs | Wiki | Repositories
  

Viewing Issue Simple Details [ Jump to Notes ] [ Wiki ]  [ Related Changesets ] [ View Advanced ] [ Issue History ] [ Print ]
ID Category Type Reproducibility Date Submitted Last Update
0000125 [In-Portal CMS] Optimization bug report always 2009-07-19 07:39 2009-10-03 07:56
Reporter alex View Status public Project Name In-Portal CMS
Assigned To alex Developer
Priority normal Resolution fixed Fixed in Version 5.0.0
Status closed Product Version 4.3.9 Target Version 5.0.0
Time EstimateNo estimate
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] );
}
Additional Information
Tags No tags attached.
Reference
Change Log Message
Estimate Points 0
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
child of 0000173closed (5.0.1)alex In-Commerce Save button won't work on Add Product Options 

-  Notes
User avatar (0000154)
alex (manager)
2009-07-19 08:16

 Fix committed to RC branch. Commit Message:

Fixes 0000125: Javascript cycle "for (index in array)" is used for arrays - replace with normal "for (index = 0; i < array.length; i++)"
User avatar (0000649)
administrator (administrator)
2009-10-03 07:56

 Closing issues from 5.0.0 version, because version was already released.

- Related Changesets
Modules :: In-Commerce: 5.0.x r12163
Timestamp: 2009-08-04 09:20:44
Author: alex
[ Details ] [ Diff ] 		1. Fixes 0000173: Save button won't work on Add Product Options
2. Related to bug 0000125: Javascript cycle "for (index in array)" is used for arrays - replace with normal "for (index = 0; i < array.length; i++)"
mod - /in-commerce/branches/5.0.x/in-commerce/admin_templates/products/option_edit.tpl [ Diff ] [ File ]
In-Portal CMS: RC r11932
Timestamp: 2009-07-19 08:16:46
Author: alex
[ Details ] [ Diff ] 		Fixes 0000125: Javascript cycle "for (index in array)" is used for arrays - replace with normal "for (index = 0; i < array.length; i++)"
mod - /in-portal/branches/RC/core/admin_templates/js/ajax.js [ Diff ] [ File ]
mod - /in-portal/branches/RC/core/admin_templates/js/calendar/calendar-setup.js [ Diff ] [ File ]
mod - /in-portal/branches/RC/core/admin_templates/js/calendar/calendar.js [ Diff ] [ File ]
mod - /in-portal/branches/RC/core/admin_templates/js/dropdown_mapper.js [ Diff ] [ File ]
mod - /in-portal/branches/RC/core/admin_templates/js/form_controls.js [ Diff ] [ File ]
mod - /in-portal/branches/RC/core/admin_templates/js/forms.js [ Diff ] [ File ]
mod - /in-portal/branches/RC/core/admin_templates/js/grid.js [ Diff ] [ File ]
mod - /in-portal/branches/RC/core/admin_templates/js/grid_scroller.js [ Diff ] [ File ]
mod - /in-portal/branches/RC/core/admin_templates/js/script.js [ Diff ] [ File ]
mod - /in-portal/branches/RC/core/admin_templates/js/tree.js [ Diff ] [ File ]

- 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

Main | My View | View Issues | Change Log | Roadmap | Docs | Wiki | Repositories
  

Web Development by Intechnic
In-Portal Open Source CMS
In-Portal Open Source CMS
Copyright © 2000 - 2009 MantisBT Group

Powered by Mantis Bugtracker