Closed Bug 623639 Opened 14 years ago Closed 14 years ago

split woo.bugs.js into woo.bugs.js & woo.dataparser.js for easier database switching

Categories

(Tree Management Graveyard :: OrangeFactor, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jmaher, Unassigned)

Details

Attachments

(1 file)

remove all the code and functions that depend on the raw data from a database query.
Attachment #501708 - Flags: review?(mcote)
Comment on attachment 501708 [details] [diff] [review]
move data parsing functions into woo.dataparser.js

+  /*
+   *  haystack is an array of single values and needle is a value to search for inside the array.
+   */
+  function index(haystack, needle) {
+    for (idx in haystack) {
+      if (haystack[idx] == needle) return idx;
+    }
+    return -1;
+  }

This looks fine, but while you're at it, why don't you move the above function to woo.utils.js, where it would be more appropriate?

r+ with that change.
Attachment #501708 - Flags: review?(mcote) → review+
http://hg.mozilla.org/automation/orangefactor/pushloghtml?changeset=8443ef4af7bd
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Product: Testing → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: