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)
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•14 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•14 years ago
|
||
http://hg.mozilla.org/automation/orangefactor/pushloghtml?changeset=8443ef4af7bd
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•10 years ago
|
Product: Testing → Tree Management
Updated•4 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
•