Closed
Bug 623639
Opened 15 years ago
Closed 15 years ago
split woo.bugs.js into woo.bugs.js & woo.dataparser.js for easier database switching
Categories
(Tree Management Graveyard :: OrangeFactor, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jmaher, Unassigned)
Details
Attachments
(1 file)
|
12.41 KB,
patch
|
mcote
:
review+
|
Details | Diff | Splinter Review |
remove all the code and functions that depend on the raw data from a database query.
Attachment #501708 -
Flags: review?(mcote)
Comment 1•15 years ago
|
||
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+
| Reporter | ||
Comment 2•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•11 years ago
|
Product: Testing → Tree Management
Updated•5 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•