From 3240d52564adbf1035c13534010a52d6f9b6bd36 Mon Sep 17 00:00:00 2001
From: Anik Das <anikdas0811@gmail.com>
Date: Sat, 4 Sep 2021 04:17:04 +0530
Subject: [PATCH] feat: added eslint eqeqeq rule, closes #5238 (#5614)

* chore: add eslint eqeqeq rule

Signed-off-by: Anik Das <anikdas0811@gmail.com>

* fix: === in place of == from eslint

changed != to !== as well. some vue components also used == in v-if
which did not show up in eslint for some reason? changed them as well

Signed-off-by: Anik Das <anikdas0811@gmail.com>
---
 .eslintrc.json                                |  1 +
 openlibrary/components/LibraryExplorer.vue    |  2 +-
 .../LibraryExplorer/components/BookRoom.vue   |  8 +-
 .../components/ClassSlider.vue                |  6 +-
 .../components/FlatBookCover.vue              |  4 +-
 .../components/LibraryToolbar.vue             | 10 +-
 .../LibraryExplorer/components/OLCarousel.vue | 16 ++--
 .../LibraryExplorer/components/Shelf.vue      |  4 +-
 .../LibraryExplorer/components/ShelfIndex.vue |  4 +-
 .../LibraryExplorer/components/ShelfLabel.vue |  2 +-
 .../components/ShelfProgressBar.vue           |  4 +-
 .../components/LibraryExplorer/configs.js     |  4 +-
 .../components/LibraryExplorer/utils.js       |  8 +-
 .../components/LibraryExplorer/utils/lcc.js   |  2 +-
 .../plugins/openlibrary/js/SearchBar.js       |  6 +-
 .../plugins/openlibrary/js/SearchUtils.js     | 10 +-
 .../plugins/openlibrary/js/add_new_field.js   |  6 +-
 .../openlibrary/js/carousel/Carousel.js       |  4 +-
 openlibrary/plugins/openlibrary/js/edit.js    | 16 ++--
 .../openlibrary/js/goodreads_import.js        |  6 +-
 .../plugins/openlibrary/js/graphs/plot.js     |  6 +-
 openlibrary/plugins/openlibrary/js/i18n.js    |  4 +-
 openlibrary/plugins/openlibrary/js/index.js   |  2 +-
 .../plugins/openlibrary/js/jquery.repeat.js   |  2 +-
 openlibrary/plugins/openlibrary/js/jsdef.js   | 12 +--
 .../openlibrary/js/markdown-editor/index.js   |  2 +-
 .../js/page-barcodescanner/LazyBookCard.js    | 10 +-
 .../js/page-barcodescanner/index.js           |  4 +-
 .../openlibrary/js/readinglog_stats.js        | 10 +-
 .../js/realtime_account_validation.js         |  6 +-
 openlibrary/plugins/openlibrary/js/search.js  |  8 +-
 .../plugins/openlibrary/js/subjects.js        | 12 +--
 .../plugins/openlibrary/js/template.js        |  2 +-
 scripts/2009/01/olpc/src/js/carousel.js       | 30 +++---
 scripts/2009/01/olpc/src/js/jquery.cluetip.js | 42 ++++-----
 .../2009/01/olpc/src/js/jquery.easing.1.3.js  |  6 +-
 .../2009/01/olpc/src/js/jquery.hoverIntent.js | 16 ++--
 scripts/2009/01/olpc/src/js/olpcbook.js       |  2 +-
 tests/unit/js/test.search.js                  |  4 +-
 vendor/js/jquery-form/jquery.form.js          | 92 +++++++++----------
 40 files changed, 198 insertions(+), 197 deletions(-)

diff --git a/.eslintrc.json b/.eslintrc.json
index 241aa77363b..e1e289b19e2 100644
--- a/.eslintrc.json
+++ b/.eslintrc.json
@@ -19,6 +19,7 @@
   "globals": {},
   "rules": {
     "prefer-template": "error",
+    "eqeqeq": ["error", "always"],
     "quotes": ["error", "single"],
     "eol-last": ["error", "always"],
     "indent": 2,
diff --git a/openlibrary/components/LibraryExplorer.vue b/openlibrary/components/LibraryExplorer.vue
index 139f92c5fe5..9b9cd7e1ba3 100644
--- a/openlibrary/components/LibraryExplorer.vue
+++ b/openlibrary/components/LibraryExplorer.vue
@@ -102,7 +102,7 @@ export default {
         let jumpTo = null;
         if (urlParams.has('jumpTo')) {
             const [classificationName, classificationString] = urlParams.get('jumpTo').split(':');
-            selectedClassification = classifications.find(c => c.field == classificationName);
+            selectedClassification = classifications.find(c => c.field === classificationName);
             jumpTo = selectedClassification.toQueryFormat(classificationString);
         }
         return {
diff --git a/openlibrary/components/LibraryExplorer/components/BookRoom.vue b/openlibrary/components/LibraryExplorer/components/BookRoom.vue
index 30dab5c8cac..30d65e8b3cb 100644
--- a/openlibrary/components/LibraryExplorer/components/BookRoom.vue
+++ b/openlibrary/components/LibraryExplorer/components/BookRoom.vue
@@ -2,7 +2,7 @@
   <div class="book-room" :class="{'expanding-animation': expandingAnimation}">
     <!-- <div class="room-breadcrumbs">
       <span v-for="(node, i) of breadcrumbs" :key="i">
-        <button @click="goUpTo(i)">{{i == 0 ? 'Home' : node.name}}</button>
+        <button @click="goUpTo(i)">{{i === 0 ? 'Home' : node.name}}</button>
         &gt;
       </span>
       <span v-if="breadcrumbs.length">{{activeRoom.name}}</span>
@@ -177,7 +177,7 @@ export default {
             });
 
             // Find the offset of the predecessor of the requested item in its shelf
-            const predecessor = decrementStringSolr(this.jumpToData.classification, false, this.classification.field == 'ddc');
+            const predecessor = decrementStringSolr(this.jumpToData.classification, false, this.classification.field === 'ddc');
             const shelf_query = `${this.classification.field}_sort:${this.jumpToData.shelf.query} ${this.filter}`;
             /** @type {number} */
             const offset = await fetch(`${CONFIGS.OL_BASE_SEARCH}/search.json?${new URLSearchParams({
@@ -219,7 +219,7 @@ export default {
             this.expandingAnimation = false;
             this.breadcrumbs.push(this.activeRoom);
             this.activeRoom = bookshelf;
-            const nodeToScrollTo = shelf?.position == 'root' ? shelf :
+            const nodeToScrollTo = shelf?.position === 'root' ? shelf :
                 shelf?.children && shelf?.position ? shelf.children[shelf.position]
                     : (shelf || bookshelf);
             await Vue.nextTick();
@@ -240,7 +240,7 @@ export default {
             this.roomWidth = max(1, this.$el.querySelector('.book-room-shelves').scrollWidth);
             this.viewportWidth = max(1, this.$el.getBoundingClientRect().width);
 
-            if (this.roomWidth == 1 || this.viewportWidth == 1) {
+            if (this.roomWidth === 1 || this.viewportWidth === 1) {
                 setTimeout(this.updateWidths, 100);
             }
         },
diff --git a/openlibrary/components/LibraryExplorer/components/ClassSlider.vue b/openlibrary/components/LibraryExplorer/components/ClassSlider.vue
index be85c542a09..03f8b030419 100644
--- a/openlibrary/components/LibraryExplorer/components/ClassSlider.vue
+++ b/openlibrary/components/LibraryExplorer/components/ClassSlider.vue
@@ -43,7 +43,7 @@ export default {
 
     watch: {
         async index(newVal, oldVal) {
-            if (typeof oldVal != 'number') return;
+            if (typeof oldVal !== 'number') return;
             this.direction = newVal > oldVal ? 'slide-right' : 'slide-left';
             await new Promise(res => setTimeout(res, 200));
             this.direction = null;
@@ -53,10 +53,10 @@ export default {
     computed: {
         index: {
             get() {
-                return this.node.position == 'root' ? 0 : this.node.position + 1;
+                return this.node.position === 'root' ? 0 : this.node.position + 1;
             },
             set(newVal) {
-                this.node.position = newVal == 0 ? 'root' : newVal - 1;
+                this.node.position = newVal === 0 ? 'root' : newVal - 1;
             }
         },
         sections() {
diff --git a/openlibrary/components/LibraryExplorer/components/FlatBookCover.vue b/openlibrary/components/LibraryExplorer/components/FlatBookCover.vue
index e19d7923d7a..34d9a7fed8b 100644
--- a/openlibrary/components/LibraryExplorer/components/FlatBookCover.vue
+++ b/openlibrary/components/LibraryExplorer/components/FlatBookCover.vue
@@ -1,6 +1,6 @@
 <template>
   <img
-    v-if="cover == 'image' && coverMultiresUrl"
+    v-if="cover === 'image' && coverMultiresUrl"
     class="cover"
     loading="lazy"
     @load="$emit('load', $event)"
@@ -37,7 +37,7 @@ export default {
         coverMultiresUrl() {
             const { cover_i, lending_edition_s } = this.book;
             const fullUrl = lending_edition_s ? this.olCoverUrl(lending_edition_s, 'olid') :
-                cover_i && cover_i != -1 ? this.olCoverUrl(cover_i) :
+                cover_i && cover_i !== -1 ? this.olCoverUrl(cover_i) :
                     null;
 
             if (fullUrl) {
diff --git a/openlibrary/components/LibraryExplorer/components/LibraryToolbar.vue b/openlibrary/components/LibraryExplorer/components/LibraryToolbar.vue
index dadb6d40abe..1a17f89dc10 100644
--- a/openlibrary/components/LibraryExplorer/components/LibraryToolbar.vue
+++ b/openlibrary/components/LibraryExplorer/components/LibraryToolbar.vue
@@ -249,8 +249,8 @@ export default {
 
     watch: {
         quickLanguageSelect(newVal) {
-            if (newVal == '') this.filterState.languages = [];
-            else if (newVal == 'custom') this.filterState.languages = this.fullLanguageSelect;
+            if (newVal === '') this.filterState.languages = [];
+            else if (newVal === 'custom') this.filterState.languages = this.fullLanguageSelect;
             else this.filterState.languages = [newVal];
         },
 
@@ -282,7 +282,7 @@ export default {
 
         computedFilters() {
             const parts = this.filterState.solrQueryParts();
-            const computedParts = parts[0] == this.filterState.filter ? parts.slice(1) : parts;
+            const computedParts = parts[0] === this.filterState.filter ? parts.slice(1) : parts;
             return computedParts.length ? ` AND ${computedParts.join(' AND ')}` : '';
         },
 
@@ -295,7 +295,7 @@ export default {
         },
 
         inDebugMode() {
-            return new URLSearchParams(location.search).get('debug') == 'true';
+            return new URLSearchParams(location.search).get('debug') === 'true';
         },
 
         styles() {
@@ -323,7 +323,7 @@ export default {
 
         toggleTab(tabName) {
             const index = this.openTabs.indexOf(tabName);
-            if (index == -1) {
+            if (index === -1) {
                 this.openTabs.push(tabName);
                 if (this.openTabs.length > this.maxTabs) {
                     this.openTabs.shift();
diff --git a/openlibrary/components/LibraryExplorer/components/OLCarousel.vue b/openlibrary/components/LibraryExplorer/components/OLCarousel.vue
index 637423a07fe..6cb4348ce08 100644
--- a/openlibrary/components/LibraryExplorer/components/OLCarousel.vue
+++ b/openlibrary/components/LibraryExplorer/components/OLCarousel.vue
@@ -1,6 +1,6 @@
 <template>
   <div class="ol-carousel">
-    <BooksCarousel v-if="status == 'Loaded' || results.length" :books="results">
+    <BooksCarousel v-if="status === 'Loaded' || results.length" :books="results">
       <template v-slot:cover-label="{book}">
         <slot name="cover-label" v-bind:book="book"/>
       </template>
@@ -13,23 +13,23 @@
         <slot name="book-end-start"/>
         <div class="book-end" v-if="offset > 0">
           <small>{{offset}}-{{offset + results.length}} of {{numFound}}</small>
-          <button class="load-more" @click="loadPrevPage" :disabled="offset == 0">Load previous</button>
+          <button class="load-more" @click="loadPrevPage" :disabled="offset === 0">Load previous</button>
           <a class="view-all" :href="olUrl" target="_blank">View all</a>
         </div>
       </template>
       <template #book-end>
         <div class="book-end">
           <small>{{offset}}-{{offset + results.length}} of {{numFound}}</small>
-          <button class="load-more" @click="loadNextPage" :disabled="offset + results.length == numFound">Load next</button>
+          <button class="load-more" @click="loadNextPage" :disabled="offset + results.length === numFound">Load next</button>
           <a class="view-all" :href="olUrl" target="_blank">View all</a>
         </div>
       </template>
     </BooksCarousel>
     <transition>
-      <div class="status-text" v-if="status == 'Errored'">Something went wrong... <button @click="reloadResults('reload')">Retry</button></div>
+      <div class="status-text" v-if="status === 'Errored'">Something went wrong... <button @click="reloadResults('reload')">Retry</button></div>
     </transition>
     <transition>
-      <div class="status-text" v-if="status == 'Loading'">Loading...</div>
+      <div class="status-text" v-if="status === 'Loading'">Loading...</div>
     </transition>
   </div>
 </template>
@@ -61,7 +61,7 @@ class CarouselCoordinator {
 
     registerOnscreenCarousel(carousel) {
         const index = this.currentlyRenderedOffscreen.indexOf(carousel);
-        if (index != -1) {
+        if (index !== -1) {
             // console.log('CarouselCoordinator', `Carousel no longer offscreen -- ${carousel.query}`);
             this.currentlyRenderedOffscreen.splice(index, 1);
         }
@@ -195,7 +195,7 @@ export default {
 
         async loadResults(offset, cache='force-cache') {
             // Don't re-fetch if already there
-            if (offset == this.offset && this.results.length) return;
+            if (offset === this.offset && this.results.length) return;
 
             this.lastFetchAbortController?.abort();
             if ('AbortController' in window) this.lastFetchAbortController = new AbortController();
@@ -230,7 +230,7 @@ export default {
 
         async _loadOffset(newOffset) {
             await this.loadResults(newOffset);
-            if (this.status == 'Loaded') this.offset = newOffset;
+            if (this.status === 'Loaded') this.offset = newOffset;
         },
 
         async loadPageContainingOffset(offset) {
diff --git a/openlibrary/components/LibraryExplorer/components/Shelf.vue b/openlibrary/components/LibraryExplorer/components/Shelf.vue
index 6c134e2b9a3..1ede0dee352 100644
--- a/openlibrary/components/LibraryExplorer/components/Shelf.vue
+++ b/openlibrary/components/LibraryExplorer/components/Shelf.vue
@@ -4,12 +4,12 @@
       class="shelf-label"
       :node="node"
       :key="node.short"
-      :is="features.shelfLabel == 'slider' ? 'ClassSlider' : 'ShelfLabel'"
+      :is="features.shelfLabel === 'slider' ? 'ClassSlider' : 'ShelfLabel'"
     >
       <template #extra-actions>
         <button
           :title="`See a list of the subsections of ${node.short}: ${node.name}`"
-          v-if="features.shelfLabel == 'slider' && node.children"
+          v-if="features.shelfLabel === 'slider' && node.children"
           :class="{selected: showShelfIndex}"
           @click="showShelfIndex = !showShelfIndex"
         >
diff --git a/openlibrary/components/LibraryExplorer/components/ShelfIndex.vue b/openlibrary/components/LibraryExplorer/components/ShelfIndex.vue
index 51a727f0f8d..9c74002f465 100644
--- a/openlibrary/components/LibraryExplorer/components/ShelfIndex.vue
+++ b/openlibrary/components/LibraryExplorer/components/ShelfIndex.vue
@@ -2,7 +2,7 @@
   <ol class="shelf-label--subclasses">
     <li>
       <a
-        :class="{ selected: index == 'root' }"
+        :class="{ selected: index === 'root' }"
         href="#"
         @click.prevent="index = 'root'"
         >
@@ -13,7 +13,7 @@
     </li>
     <li v-for="(child, i) of node.children || []" :key="i">
       <a
-        :class="{ selected: index == i }"
+        :class="{ selected: index === i }"
         href="#"
         @click.prevent="index = i"
         >
diff --git a/openlibrary/components/LibraryExplorer/components/ShelfLabel.vue b/openlibrary/components/LibraryExplorer/components/ShelfLabel.vue
index a73fbc76bbd..592e87667a9 100644
--- a/openlibrary/components/LibraryExplorer/components/ShelfLabel.vue
+++ b/openlibrary/components/LibraryExplorer/components/ShelfLabel.vue
@@ -3,7 +3,7 @@
     <details class="shelf-label--classes">
       <summary>
         <RightArrowIcon class="shelf-label--right-arrow"/>
-        <div class="shelf-label--name">{{index == 'root' ? node.name : node.children[index].name}}</div>
+        <div class="shelf-label--name">{{index === 'root' ? node.name : node.children[index].name}}</div>
       </summary>
       <ShelfIndex :node="node" />
     </details>
diff --git a/openlibrary/components/LibraryExplorer/components/ShelfProgressBar.vue b/openlibrary/components/LibraryExplorer/components/ShelfProgressBar.vue
index 9d539e74b94..8bea8f0d918 100644
--- a/openlibrary/components/LibraryExplorer/components/ShelfProgressBar.vue
+++ b/openlibrary/components/LibraryExplorer/components/ShelfProgressBar.vue
@@ -5,10 +5,10 @@
       :key="section.short"
       :title="section.name"
       :style="{ flex: section.count }"
-      :class="{ active: index == i }"
+      :class="{ active: index === i }"
     >
       <div
-        v-if="index == i"
+        v-if="index === i"
         class="marker"
         :style="{ left: `${(100 * section.offset) / section.count}%` }"
       ></div>
diff --git a/openlibrary/components/LibraryExplorer/configs.js b/openlibrary/components/LibraryExplorer/configs.js
index a7a26020146..c0b58c16c0a 100644
--- a/openlibrary/components/LibraryExplorer/configs.js
+++ b/openlibrary/components/LibraryExplorer/configs.js
@@ -1,6 +1,6 @@
 const urlParams = new URLSearchParams(location.search);
 
-const IS_VUE_APP =  document.title == 'Vue App';
+const IS_VUE_APP =  document.title === 'Vue App';
 const OL_BASE_DEFAULT = urlParams.get('ol_base') || (IS_VUE_APP ? 'openlibrary.org' : '');
 
 const CONFIGS = {
@@ -9,7 +9,7 @@ const CONFIGS = {
     OL_BASE_BOOKS: urlParams.get('ol_base_books') || OL_BASE_DEFAULT || '',
     OL_BASE_LANGS: urlParams.get('ol_base_langs') || OL_BASE_DEFAULT || '',
     OL_BASE_PUBLIC: urlParams.get('ol_base') || 'openlibrary.org',
-    DEBUG_MODE: urlParams.get('debug') == 'true',
+    DEBUG_MODE: urlParams.get('debug') === 'true',
 };
 
 for (const key of ['OL_BASE_COVERS', 'OL_BASE_SEARCH', 'OL_BASE_BOOKS', 'OL_BASE_LANGS']) {
diff --git a/openlibrary/components/LibraryExplorer/utils.js b/openlibrary/components/LibraryExplorer/utils.js
index f774d25e5b8..a259295aebb 100644
--- a/openlibrary/components/LibraryExplorer/utils.js
+++ b/openlibrary/components/LibraryExplorer/utils.js
@@ -73,10 +73,10 @@ export function decrementStringSolr(string, caseSensitive=true, numeric=false) {
     // Also append Z; this is the equivalent of going back one, and then expanding (e.g. 0.123 decremented is not 0.122, it's 0.12999999)
     const maxTail = (numeric ? '9' : 'z').repeat(5);
     const newLastChar = (
-        lastChar == '.' ? '' :
-            lastChar == '0' ? `.${maxTail}` :
-                lastChar == 'A' ? `9${maxTail}` :
-                    lastChar == 'a' ? `Z${maxTail}` :
+        lastChar === '.' ? '' :
+            lastChar === '0' ? `.${maxTail}` :
+                lastChar === 'A' ? `9${maxTail}` :
+                    lastChar === 'a' ? `Z${maxTail}` :
                         `${String.fromCharCode(lastChar.charCodeAt(0) - 1)}${maxTail}`);
 
     return string.slice(0, -1) + newLastChar;
diff --git a/openlibrary/components/LibraryExplorer/utils/lcc.js b/openlibrary/components/LibraryExplorer/utils/lcc.js
index c9a6ec96dd9..03e8deada57 100644
--- a/openlibrary/components/LibraryExplorer/utils/lcc.js
+++ b/openlibrary/components/LibraryExplorer/utils/lcc.js
@@ -27,7 +27,7 @@ export function short_lcc_to_sortable_lcc(lcc) {
     // E.g. "CPB Box no. 1516 vol. 17"
     // Although this might be useful to search by, it's not really an LCC,
     // so considering it invalid here.
-    if (letters == 'CPB') return null;
+    if (letters === 'CPB') return null;
 
     return `${letters}${number.toFixed(8).padStart(13, '0')}${cutter1}${rest}`;
 }
diff --git a/openlibrary/plugins/openlibrary/js/SearchBar.js b/openlibrary/plugins/openlibrary/js/SearchBar.js
index 0a8ebe1ae44..c79516b585a 100644
--- a/openlibrary/plugins/openlibrary/js/SearchBar.js
+++ b/openlibrary/plugins/openlibrary/js/SearchBar.js
@@ -100,7 +100,7 @@ export class SearchBar {
 
         if (urlParams.q) {
             let q = urlParams.q.replace(/\+/g, ' ');
-            if (this.facet.read() === 'title' && q.indexOf('title:') != -1) {
+            if (this.facet.read() === 'title' && q.indexOf('title:') !== -1) {
                 const parts = q.split('"');
                 if (parts.length === 3) {
                     q = parts[1];
@@ -220,7 +220,7 @@ export class SearchBar {
      * @return {String}
      */
     static marshalBookSearchQuery(q) {
-        if (q && q.indexOf(':') == -1 && q.indexOf('"') == -1) {
+        if (q && q.indexOf(':') === -1 && q.indexOf('"') === -1) {
             q = `title: "${q}"`;
         }
         return q;
@@ -299,7 +299,7 @@ export class SearchBar {
     handleFacetSelectChange(event) {
         const newFacet = event.target.value;
         // We don't want to persist advanced becaues it behaves like a button
-        if (newFacet == 'advanced') {
+        if (newFacet === 'advanced') {
             event.preventDefault();
             this.navigateTo('/advancedsearch');
         } else {
diff --git a/openlibrary/plugins/openlibrary/js/SearchUtils.js b/openlibrary/plugins/openlibrary/js/SearchUtils.js
index 38ba1ace34d..f935e978c05 100644
--- a/openlibrary/plugins/openlibrary/js/SearchUtils.js
+++ b/openlibrary/plugins/openlibrary/js/SearchUtils.js
@@ -50,7 +50,7 @@ export class PersistentValue {
         this.options = Object.assign({}, PersistentValue.DEFAULT_OPTIONS, options);
         this._listeners = [];
 
-        const noValue = this.read() == null;
+        const noValue = this.read() === null;
         const isValid = () => !this.options.initValidation || this.options.initValidation(this.read());
         if (noValue || !isValid()) {
             this.write(this.options.default);
@@ -76,13 +76,13 @@ export class PersistentValue {
             toWrite = this.options.writeTransformation(newValue, oldValue);
         }
 
-        if (toWrite == null) {
+        if (toWrite === null) {
             localStorage.removeItem(this.key);
         } else {
             localStorage.setItem(this.key, toWrite);
         }
 
-        if (oldValue != toWrite) {
+        if (oldValue !== toWrite) {
             this._emit(toWrite);
         }
     }
@@ -120,10 +120,10 @@ const DEFAULT_MODE = 'everything';
 /** Search mode; {@see MODES} */
 export const mode = new PersistentValue('mode', {
     default: DEFAULT_MODE,
-    initValidation: mode => MODES.indexOf(mode) != -1,
+    initValidation: mode => MODES.indexOf(mode) !== -1,
     writeTransformation(newValue, oldValue) {
         const mode = (newValue && newValue.toLowerCase()) || oldValue;
-        const isValidMode = MODES.indexOf(mode) != -1;
+        const isValidMode = MODES.indexOf(mode) !== -1;
         return isValidMode ? mode : DEFAULT_MODE;
     }
 });
diff --git a/openlibrary/plugins/openlibrary/js/add_new_field.js b/openlibrary/plugins/openlibrary/js/add_new_field.js
index 75412049473..c2f16ab6834 100644
--- a/openlibrary/plugins/openlibrary/js/add_new_field.js
+++ b/openlibrary/plugins/openlibrary/js/add_new_field.js
@@ -28,7 +28,7 @@ export default function($){
 
             $this.on('change', function(){
                 var value = $this.val();
-                if (value == '__add__') {
+                if (value === '__add__') {
                     if (options.onshow) {
                         options.onshow.apply($this, []);
                     }
@@ -45,7 +45,7 @@ export default function($){
             // handle cancel
             $href.on('cbox_closed', function() {
 
-                if ($this.val() == '__add__') {
+                if ($this.val() === '__add__') {
                     $this.val('');
                     $this.trigger('focus');
                 }
@@ -68,7 +68,7 @@ export default function($){
                 }
 
                 // validate
-                if (options.validate && options.validate.apply($this, [d]) == false) {
+                if (options.validate && options.validate.apply($this, [d]) === false) {
                     return;
                 }
 
diff --git a/openlibrary/plugins/openlibrary/js/carousel/Carousel.js b/openlibrary/plugins/openlibrary/js/carousel/Carousel.js
index 1eac587770b..9ea89f0934e 100644
--- a/openlibrary/plugins/openlibrary/js/carousel/Carousel.js
+++ b/openlibrary/plugins/openlibrary/js/carousel/Carousel.js
@@ -101,7 +101,7 @@ const Carousel = {
             const availabilityStatus = availabilityStatuses[availability.status] || availabilityStatuses.error;
             const cls = availabilityStatus.cls;
             const cta = availabilityStatus.cta;
-            const url = cls == 'cta-btn--available' ? `/borrow/ia/${ocaid}` : work.key;
+            const url = cls === 'cta-btn--available' ? `/borrow/ia/${ocaid}` : work.key;
 
             if (!cover.id && ocaid) {
                 cover.type = 'ia';
@@ -151,7 +151,7 @@ const Carousel = {
                     loadMore.locked = true; // lock for critical section
                     slick.addSlide('<div class="carousel__item carousel__loading-end">Loading...</div>');
 
-                    if (loadMore.pageMode == 'page') {
+                    if (loadMore.pageMode === 'page') {
                         // for first time, we're on page 1 already so initialize as page 2
                         // otherwise advance to next page
                         loadMore.page = loadMore.page ? loadMore.page + 1 : 2;
diff --git a/openlibrary/plugins/openlibrary/js/edit.js b/openlibrary/plugins/openlibrary/js/edit.js
index f63e3327116..98e0dd91918 100644
--- a/openlibrary/plugins/openlibrary/js/edit.js
+++ b/openlibrary/plugins/openlibrary/js/edit.js
@@ -58,10 +58,10 @@ export function initRoleValidation() {
     $('#roles').repeat({
         vars: {prefix: 'edition--'},
         validate: function (data) {
-            if (data.role == '' || data.role == '---') {
+            if (data.role === '' || data.role === '---') {
                 return error('#role-errors', '#select-role', dataConfig['Please select a role.']);
             }
-            if (data.name == '') {
+            if (data.name === '') {
                 return error('#role-errors', '#role-name', dataConfig['You need to give this ROLE a name.'].replace(/ROLE/, data.role));
             }
             $('#role-errors').hide();
@@ -75,20 +75,20 @@ export function initIdentifierValidation() {
     $('#identifiers').repeat({
         vars: {prefix: 'edition--'},
         validate: function (data) {
-            if (data.name == '' || data.name == '---') {
+            if (data.name === '' || data.name === '---') {
                 return error('#id-errors', 'select-id', dataConfig['Please select an identifier.'])
             }
             const label = $('#select-id').find(`option[value='${data.name}']`).html();
-            if (data.value == '') {
+            if (data.value === '') {
                 return error('#id-errors', 'id-value', dataConfig['You need to give a value to ID.'].replace(/ID/, label));
             }
             if (['ocaid'].includes(data.name) && /\s/g.test(data.value)) {
                 return error('#id-errors', 'id-value', dataConfig['ID ids cannot contain whitespace.'].replace(/ID/, label));
             }
-            if (data.name == 'isbn_10' && data.value.length != 10) {
+            if (data.name === 'isbn_10' && data.value.length !== 10) {
                 return error('#id-errors', 'id-value', dataConfig['ID must be exactly 10 characters [0-9] or X.'].replace(/ID/, label));
             }
-            if (data.name == 'isbn_13' && data.value.replace(/-/g, '').length != 13) {
+            if (data.name === 'isbn_13' && data.value.replace(/-/g, '').length !== 13) {
                 return error('#id-errors', 'id-value', dataConfig['ID must be exactly 13 digits [0-9]. For example: 978-1-56619-909-4'].replace(/ID/, label));
             }
             $('id-errors').hide();
@@ -102,10 +102,10 @@ export function initClassificationValidation() {
     $('#classifications').repeat({
         vars: {prefix: 'edition--'},
         validate: function (data) {
-            if (data.name == '' || data.name == '---') {
+            if (data.name === '' || data.name === '---') {
                 return error('#classification-errors', '#select-classification', dataConfig['Please select a classification.']);
             }
-            if (data.value == '') {
+            if (data.value === '') {
                 const label = $('#select-classification').find(`option[value='${data.name}']`).html();
                 return error('#classification-errors', '#classification-value', dataConfig['You need to give a value to CLASS.'].replace(/CLASS/, label));
             }
diff --git a/openlibrary/plugins/openlibrary/js/goodreads_import.js b/openlibrary/plugins/openlibrary/js/goodreads_import.js
index 9f0481ad577..d5033d4d69d 100644
--- a/openlibrary/plugins/openlibrary/js/goodreads_import.js
+++ b/openlibrary/plugins/openlibrary/js/goodreads_import.js
@@ -64,7 +64,7 @@ export function initGoodreadsImport() {
             if (shelves[shelf]){
                 shelf_id = shelves[shelf];
             }
-            if (checked && shelf_id != 0) {
+            if (checked && shelf_id !== 0) {
                 prevPromise = prevPromise.then(function () { // prevPromise changes in each iteration
                     $(`[isbn=${value['ISBN']}]`).addClass('selected');
                     return getWork(value['ISBN']); // return a new Promise
@@ -80,7 +80,7 @@ export function initGoodreadsImport() {
                         },
                         datatype: 'json',
                         success: function () {
-                            if (value['My Rating'] != '0') {
+                            if (value['My Rating'] !== '0') {
                                 $.ajax({
                                     url: `${obj['works'][0].key}/ratings.json`,
                                     type: 'POST',
@@ -117,7 +117,7 @@ export function initGoodreadsImport() {
                     func1(++count);
                 });
             }
-            else if (checked && shelf_id == 0) {
+            else if (checked && shelf_id === 0) {
                 $(`[isbn=${value['ISBN']}]`).append('<td class="error-imported">Error</td><td class="error-imported">Book in different Shelf</td>');
                 func1(++count);
             }
diff --git a/openlibrary/plugins/openlibrary/js/graphs/plot.js b/openlibrary/plugins/openlibrary/js/graphs/plot.js
index 0c12a69d4d6..288b00e2361 100644
--- a/openlibrary/plugins/openlibrary/js/graphs/plot.js
+++ b/openlibrary/plugins/openlibrary/js/graphs/plot.js
@@ -72,12 +72,12 @@ export function loadEditionsGraph() {
         $('#x').text(pos.x.toFixed(0));
         $('#y').text(pos.y.toFixed(0));
         if (item) {
-            if (previousPoint != item.datapoint) {
+            if (previousPoint !== item.datapoint) {
                 previousPoint = item.datapoint;
                 $('#chartLabel').remove();
                 x = item.datapoint[0].toFixed(0);
                 y = item.datapoint[1].toFixed(0);
-                if (y == 1) {
+                if (y === 1) {
                     showTooltip(item.pageX, item.pageY,
                         `${y} edition in ${x}`);
                 } else {
@@ -97,7 +97,7 @@ export function loadEditionsGraph() {
 
     $('.chartYaxis').css({top: '60px', left: '-60px'});
 
-    if (dateFrom == (dateTo - 1)) {
+    if (dateFrom === (dateTo - 1)) {
         $('.clickdata').text(`Published in ${dateFrom}`);
     } else {
         $('.clickdata').text(`Published between ${dateFrom} & ${dateTo-1}.`);
diff --git a/openlibrary/plugins/openlibrary/js/i18n.js b/openlibrary/plugins/openlibrary/js/i18n.js
index 000bf3267ea..2110502ff3a 100644
--- a/openlibrary/plugins/openlibrary/js/i18n.js
+++ b/openlibrary/plugins/openlibrary/js/i18n.js
@@ -3,7 +3,7 @@ export function sprintf(s) {
     var args = arguments;
     var i = 1;
     return s.replace(/%[%s]/g, function(match) {
-        if (match == '%%')
+        if (match === '%%')
             return '%';
         else
             return args[i++];
@@ -19,5 +19,5 @@ export function ugettext(s) {
 
 // used in templates/borrow/read.html
 export function ungettext(s1, s2, n) {
-    return n == 1? s1 : s2;
+    return n === 1? s1 : s2;
 }
diff --git a/openlibrary/plugins/openlibrary/js/index.js b/openlibrary/plugins/openlibrary/js/index.js
index 413e3194fe4..08e2ffae6b4 100644
--- a/openlibrary/plugins/openlibrary/js/index.js
+++ b/openlibrary/plugins/openlibrary/js/index.js
@@ -196,7 +196,7 @@ jQuery(function () {
         import(/* webpackChunkName: "carousel" */ './carousel')
             .then((module) => { module.init($carouselElements);
                 $('.slick-slide').each(function () {
-                    if ($(this).attr('aria-describedby') != undefined) {
+                    if ($(this).attr('aria-describedby') !== undefined) {
                         $(this).attr('id',$(this).attr('aria-describedby'));
                     }
                 });
diff --git a/openlibrary/plugins/openlibrary/js/jquery.repeat.js b/openlibrary/plugins/openlibrary/js/jquery.repeat.js
index 1f96a0fb253..8139905e402 100644
--- a/openlibrary/plugins/openlibrary/js/jquery.repeat.js
+++ b/openlibrary/plugins/openlibrary/js/jquery.repeat.js
@@ -73,7 +73,7 @@ export default function($){
             data = formdata();
             data.index = nextRowId;
 
-            if (options.validate && options.validate(data) == false) {
+            if (options.validate && options.validate(data) === false) {
                 return;
             }
 
diff --git a/openlibrary/plugins/openlibrary/js/jsdef.js b/openlibrary/plugins/openlibrary/js/jsdef.js
index 6eadb7b1789..956a6e2c371 100644
--- a/openlibrary/plugins/openlibrary/js/jsdef.js
+++ b/openlibrary/plugins/openlibrary/js/jsdef.js
@@ -20,7 +20,7 @@
 export function range(begin, end, step) {
     var r, i;
     step = step || 1;
-    if (end == undefined) {
+    if (end === undefined) {
         end = begin;
         begin = 0;
     }
@@ -75,11 +75,11 @@ ForLoop.prototype.next = function() {
     this.index0 = i;
     this.index = i+1;
 
-    this.first = (i == 0);
-    this.last = (i == this.length-1);
+    this.first = (i === 0);
+    this.last = (i === this.length-1);
 
-    this.odd = (this.index % 2 == 1);
-    this.even = (this.index % 2 == 0);
+    this.odd = (this.index % 2 === 1);
+    this.even = (this.index % 2 === 0);
     this.parity = ['even', 'odd'][this.index % 2];
 
     this.revindex0 = this.length - i;
@@ -114,7 +114,7 @@ export function websafe(value) {
     // Safari 6 is failing with weird javascript error in this function.
     // Added try-catch to avoid it.
     try {
-        if (value == null || value == undefined) {
+        if (value === null || value === undefined) {
             return '';
         }
         else {
diff --git a/openlibrary/plugins/openlibrary/js/markdown-editor/index.js b/openlibrary/plugins/openlibrary/js/markdown-editor/index.js
index 245dec48b00..67af03eb33f 100644
--- a/openlibrary/plugins/openlibrary/js/markdown-editor/index.js
+++ b/openlibrary/plugins/openlibrary/js/markdown-editor/index.js
@@ -9,7 +9,7 @@ export function initMarkdownEditor($textareas) {
     $textareas.on('focus', function (){
         // reveal the previous when the user focuses on the textarea for the first time
         $('.wmd-preview').show();
-        if ($('#prevHead').length == 0) {
+        if ($('#prevHead').length === 0) {
             $('.wmd-preview').before('<h3 id="prevHead">Preview</h3>');
         }
     }).wmd({
diff --git a/openlibrary/plugins/openlibrary/js/page-barcodescanner/LazyBookCard.js b/openlibrary/plugins/openlibrary/js/page-barcodescanner/LazyBookCard.js
index b4e114a05b7..9b57dab731c 100644
--- a/openlibrary/plugins/openlibrary/js/page-barcodescanner/LazyBookCard.js
+++ b/openlibrary/plugins/openlibrary/js/page-barcodescanner/LazyBookCard.js
@@ -49,24 +49,24 @@ export default class LazyBookCard {
         newState = Object.assign({}, oldState, newState);
 
         if (this.ui) {
-            if (oldState.link != newState.link) {
+            if (oldState.link !== newState.link) {
                 this.ui.attr('href', newState.link);
             }
 
-            if (oldState.coverSrc != newState.coverSrc) {
+            if (oldState.coverSrc !== newState.coverSrc) {
                 this.ui.find('.cover img').attr('src', newState.coverSrc);
             }
 
             const textFields = ['title', 'byline', 'identifier'];
             for (const field of textFields) {
-                if (oldState[field] != newState[field]) {
+                if (oldState[field] !== newState[field]) {
                     this.ui.find(`.${field}`).text(newState[field]);
                 }
             }
 
             const classFields = ['loading', 'errored'];
             for (const field of classFields) {
-                if (oldState[field] != newState[field]) {
+                if (oldState[field] !== newState[field]) {
                     this.ui.toggleClass(field, newState[field]);
                 }
             }
@@ -95,7 +95,7 @@ export default class LazyBookCard {
                 });
 
                 if (editionRecord.covers) {
-                    const coverId = editionRecord.covers.find(x => x != -1);
+                    const coverId = editionRecord.covers.find(x => x !== -1);
                     if (coverId) {
                         cardEl.updateState({
                             coverSrc: `http://covers.openlibrary.org/b/id/${coverId}-M.jpg`,
diff --git a/openlibrary/plugins/openlibrary/js/page-barcodescanner/index.js b/openlibrary/plugins/openlibrary/js/page-barcodescanner/index.js
index 16857fa13cc..453039034b4 100644
--- a/openlibrary/plugins/openlibrary/js/page-barcodescanner/index.js
+++ b/openlibrary/plugins/openlibrary/js/page-barcodescanner/index.js
@@ -32,7 +32,7 @@ export function init() {
             drawingCtx.clearRect(0, 0, canvasWidth, canvasHeight);
 
             result.boxes.forEach(box => {
-                if (box != result.box) {
+                if (box !== result.box) {
                     Quagga.ImageDebug.drawPath(box, {x: 0, y: 1}, drawingCtx, BOX_STYLE);
                 }
             });
@@ -50,7 +50,7 @@ export function init() {
     let lastResult = null;
     Quagga.onDetected(result => {
         const code = result.codeResult.code;
-        if (!isBarcodeISBN(code) || code == lastResult) return;
+        if (!isBarcodeISBN(code) || code === lastResult) return;
         lastResult = code;
 
         const isbn = code;
diff --git a/openlibrary/plugins/openlibrary/js/readinglog_stats.js b/openlibrary/plugins/openlibrary/js/readinglog_stats.js
index 2d1a350ddc0..7487b19dca5 100644
--- a/openlibrary/plugins/openlibrary/js/readinglog_stats.js
+++ b/openlibrary/plugins/openlibrary/js/readinglog_stats.js
@@ -124,7 +124,7 @@ export function init(config) {
         return `
             SELECT DISTINCT ?x ?xLabel ?olid
                 ${
-    SPARQL_FIELDS.map(f => `?${f.name} ${f.type == 'uri' ? `?${f.name}Label ` : ''}`).join('')
+    SPARQL_FIELDS.map(f => `?${f.name} ${f.type === 'uri' ? `?${f.name}Label ` : ''}`).join('')
 }
             WHERE {
               VALUES ?olids { ${authors.map(a => `"${a.key.split('/')[2]}"`).join(' ')} }
@@ -155,7 +155,7 @@ export function init(config) {
                 const record = { qid, olids: uniq(bindings.map(x => x.olid.value)) };
                 // { qid: Q123, olids: [ { value: }, {value: }], blah: [ {value:}, {value:} ], blahLabel: [{value:}, {value:},
                 for (const {name, type} of SPARQL_FIELDS) {
-                    if (type == 'uri') {
+                    if (type === 'uri') {
                     // need to dedupe whilst keeping labels in mind
                         const deduped = uniqBy(
                             bindings
@@ -190,9 +190,9 @@ export function init(config) {
         const canvas = document.createElement('canvas');
         container.append(canvas);
 
-        if (chartConfig.type == 'work-chart') {
+        if (chartConfig.type === 'work-chart') {
             createWorkChart(config, chartConfig, container, canvas);
-        } else if (chartConfig.type == 'wd-chart') {
+        } else if (chartConfig.type === 'wd-chart') {
             wdPromise.then(() => createWorkChart(config, chartConfig, container, canvas));
         }
     }
@@ -210,7 +210,7 @@ function getPath(obj, key) {
      * @return {any}
      */
     function main(obj, [head, ...rest]) {
-        if (typeof(obj) == 'undefined') return undefined;
+        if (typeof(obj) === 'undefined') return undefined;
         if (!head) return obj;
         if (head.endsWith('[]')) return obj[head.slice(0, -2)].flatMap(x => main(x, rest));
         else return main(obj[head], rest);
diff --git a/openlibrary/plugins/openlibrary/js/realtime_account_validation.js b/openlibrary/plugins/openlibrary/js/realtime_account_validation.js
index 9bf55668142..ee0f57367c6 100644
--- a/openlibrary/plugins/openlibrary/js/realtime_account_validation.js
+++ b/openlibrary/plugins/openlibrary/js/realtime_account_validation.js
@@ -9,7 +9,7 @@ export function initRealTimeValidation() {
 
     function validateUsername() {
         var value = $(this).val();
-        if (!value=='') {
+        if (!value === '') {
             $.ajax({
                 url: `/account/validate?username=${value}`,
                 type: 'GET',
@@ -35,7 +35,7 @@ export function initRealTimeValidation() {
 
     function validateEmail() {
         var value_email = $(this).val();
-        if (!value_email=='') {
+        if (!value_email === '') {
             $.ajax({
                 url: `/account/validate?email=${encodeURIComponent(value_email)}`,
                 type: 'GET',
@@ -63,7 +63,7 @@ export function initRealTimeValidation() {
         var value = document.getElementById('password').value;
         var value2 = document.getElementById('password2').value;
         if (value && value2) {
-            if (value2==value) {
+            if (value2 === value) {
                 $('#password2Message').removeClass().addClass('darkgreen').text('');
                 $('label[for="password2"]').removeClass();
                 $(document.getElementById('password2')).removeClass().addClass('required');
diff --git a/openlibrary/plugins/openlibrary/js/search.js b/openlibrary/plugins/openlibrary/js/search.js
index 1f254ecaf13..f8e180de272 100644
--- a/openlibrary/plugins/openlibrary/js/search.js
+++ b/openlibrary/plugins/openlibrary/js/search.js
@@ -13,7 +13,7 @@ export function more(header, start_facet_count, facet_inc) {
     const facetEntry = `div.${header} div.facetEntry`
     const shown = $(`${facetEntry}:not(:hidden)`).length
     const total = $(facetEntry).length
-    if (shown == start_facet_count) {
+    if (shown === start_facet_count) {
         $(`#${header}_less`).show();
         $(`#${header}_bull`).show();
     }
@@ -36,13 +36,13 @@ export function less(header, start_facet_count, facet_inc) {
     const shown = $(`${facetEntry}:not(:hidden)`).length
     const total = $(facetEntry).length
     const increment_extra = (shown - start_facet_count) % facet_inc;
-    const facet_dec = (increment_extra == 0) ? facet_inc:increment_extra;
+    const facet_dec = (increment_extra === 0) ? facet_inc:increment_extra;
     const next_shown = Math.max(start_facet_count, shown - facet_dec);
-    if (shown == total) {
+    if (shown === total) {
         $(`#${header}_more`).show();
         $(`#${header}_bull`).show();
     }
-    if (next_shown == start_facet_count) {
+    if (next_shown === start_facet_count) {
         $(`#${header}_less`).hide();
         $(`#${header}_bull`).hide();
     }
diff --git a/openlibrary/plugins/openlibrary/js/subjects.js b/openlibrary/plugins/openlibrary/js/subjects.js
index 3d6b476e29d..771233b335c 100644
--- a/openlibrary/plugins/openlibrary/js/subjects.js
+++ b/openlibrary/plugins/openlibrary/js/subjects.js
@@ -43,7 +43,7 @@ export function renderTag(tag, keyvals, child) {
     var key, val;
     for (key in keyvals) {
         val =  keyvals[key];
-        if (val == '') {
+        if (val === '') {
             html += `${key} `;
         } else {
             html += `${key}="${val}" `;
@@ -79,7 +79,7 @@ Subject.prototype = {
         // cache already visited pages
         //@@ Can't this be handled by HTTP caching?
         this._pages = {};
-        if (this.has_fulltext != 'true')
+        if (this.has_fulltext !== 'true')
             this._pages[0] = {works: slice(data.works, 0, this.settings.pagesize)};
 
     // TODO: initialize additional pages when there are more works.
@@ -90,7 +90,7 @@ Subject.prototype = {
     },
 
     getPageCount: function() {
-        return this.has_fulltext == 'true'? this.epage_count : this.page_count;
+        return this.has_fulltext === 'true'? this.epage_count : this.page_count;
     },
 
     renderWork: function(work) {
@@ -177,7 +177,7 @@ Subject.prototype = {
     setFilter: function(filter, callback) {
         var k, _this;
         for (k in filter) {
-            if (filter[k] == null) {
+            if (filter[k] === null) {
                 delete filter[k];
             }
         }
@@ -191,7 +191,7 @@ Subject.prototype = {
     },
 
     setSort: function(sort_order) {
-        if (this.sort == sort_order) {
+        if (this.sort === sort_order) {
             return; // shouldn't happen
         }
         this.sort = sort_order;
@@ -199,7 +199,7 @@ Subject.prototype = {
     },
 
     setFulltext: function(has_fulltext) {
-        if (this.has_fulltext == has_fulltext) {
+        if (this.has_fulltext === has_fulltext) {
             return;
         }
         this.has_fulltext = has_fulltext;
diff --git a/openlibrary/plugins/openlibrary/js/template.js b/openlibrary/plugins/openlibrary/js/template.js
index 0641d46d7cd..2d69a31f151 100644
--- a/openlibrary/plugins/openlibrary/js/template.js
+++ b/openlibrary/plugins/openlibrary/js/template.js
@@ -25,7 +25,7 @@ export default function Template(tmpl_text) {
     for (i=1; i < tokens.length; i++) {
         t = tokens[i].split('%>');
 
-        if (t[0][0] == '=') {
+        if (t[0][0] === '=') {
             addExpr(t[0].substr(1));
         }
         else {
diff --git a/scripts/2009/01/olpc/src/js/carousel.js b/scripts/2009/01/olpc/src/js/carousel.js
index c527dcf7ab8..8abc2a18f08 100644
--- a/scripts/2009/01/olpc/src/js/carousel.js
+++ b/scripts/2009/01/olpc/src/js/carousel.js
@@ -137,8 +137,8 @@ YAHOO.extension.Carousel.prototype = {
 				oThis.reload();
 			},
 			validator: function(orientation) {
-			    if(typeof orientation == "string") {
-			        return ("horizontal,vertical".indexOf(orientation.toLowerCase()) != -1);
+			    if(typeof orientation === "string") {
+			        return ("horizontal,vertical".indexOf(orientation.toLowerCase()) !== -1);
 			    } else {
 					return false;
 				}
@@ -322,7 +322,7 @@ YAHOO.extension.Carousel.prototype = {
 					YAHOO.util.Event.removeListener(oThis._carouselPrev, "click", oThis._scrollPrev);
 				} 
 				oThis._prevElementID = args[0];
-				if(oThis._prevElementID == null) {
+				if(oThis._prevElementID === null) {
 					oThis._carouselPrev = YAHOO.util.Dom.getElementsByClassName(carouselPrevClass, 
 														"div", oThis.carouselElem)[0];
 				} else {
@@ -349,7 +349,7 @@ YAHOO.extension.Carousel.prototype = {
 					YAHOO.util.Event.removeListener(oThis._carouselPrev, "click", oThis._scrollPrev);
 				} 
 				oThis._prevElementID = args[0];
-				if(oThis._prevElementID == null) {
+				if(oThis._prevElementID === null) {
 					oThis._carouselPrev = YAHOO.util.Dom.getElementsByClassName(carouselPrevClass, 
 														"div", oThis.carouselElem)[0];
 				} else {
@@ -367,7 +367,7 @@ YAHOO.extension.Carousel.prototype = {
 					YAHOO.util.Event.removeListener(oThis._carouselNext, "click", oThis._scrollNext);
 				} 
 				oThis._nextElementID = args[0];
-				if(oThis._nextElementID == null) {
+				if(oThis._nextElementID === null) {
 					oThis._carouselNext = YAHOO.util.Dom.getElementsByClassName(carouselNextClass, 
 														"div", oThis.carouselElem);
 				} else {
@@ -396,7 +396,7 @@ YAHOO.extension.Carousel.prototype = {
 					YAHOO.util.Event.removeListener(oThis._carouselNext, "click", oThis._scrollNext);
 				} 
 				oThis._nextElementID = args[0];
-				if(oThis._nextElementID == null) {
+				if(oThis._nextElementID === null) {
 					oThis._carouselNext = YAHOO.util.Dom.getElementsByClassName(carouselNextClass, 
 														"div", oThis.carouselElem);
 				} else {
@@ -589,14 +589,14 @@ YAHOO.extension.Carousel.prototype = {
  		this.carouselList = YAHOO.util.Dom.getElementsByClassName(carouselListClass, 
 												"ul", this.carouselElem)[0];
 							
-		if(this._nextElementID == null) {
+		if(this._nextElementID === null) {
 			this._carouselNext = YAHOO.util.Dom.getElementsByClassName(carouselNextClass, 
 												"div", this.carouselElem)[0];
 		} else {
 			this._carouselNext = YAHOO.util.Dom.get(this._nextElementID);
 		}
 
-		if(this._prevElementID == null) {
+		if(this._prevElementID === null) {
  			this._carouselPrev = YAHOO.util.Dom.getElementsByClassName(carouselPrevClass, 
 												"div", this.carouselElem)[0];
 		} else {
@@ -915,7 +915,7 @@ YAHOO.extension.Carousel.prototype = {
 	 * Returns whether the carousel's orientation is set to vertical.
 	 */
 	isVertical: function() {
-		return (this.cfg.getProperty("orientation") != "horizontal");
+		return (this.cfg.getProperty("orientation") !== "horizontal");
 	},
 	
 	
@@ -960,7 +960,7 @@ YAHOO.extension.Carousel.prototype = {
 		for(var i=0; i<ulKids.length; i++) {
 		
 			li = ulKids[i];
-			if(li.tagName == "LI" || li.tagName == "li") {
+			if(li.tagName === "LI" || li.tagName === "li") {
 				break;
 			}
 		}
@@ -1173,7 +1173,7 @@ YAHOO.extension.Carousel.prototype = {
 
 		var refItem = this.getItem(refIdx);
 		var size = this.cfg.getProperty("size");
-		if(size != this.UNBOUNDED_SIZE) {
+		if(size !== this.UNBOUNDED_SIZE) {
 			this.cfg.setProperty("size", size + 1, true);
 		}
 				
@@ -1288,7 +1288,7 @@ YAHOO.extension.Carousel.prototype = {
 
 		var scrollExtent = this._calculateAllowableScrollExtent();
 		
-		if(this.cfg.getProperty("wrap") && currEnd == scrollExtent.end) {
+		if(this.cfg.getProperty("wrap") && currEnd === scrollExtent.end) {
 			this.scrollTo(scrollExtent.start); // might need to check animation is on or not
 			return;
 		}
@@ -1585,10 +1585,10 @@ YAHOO.extension.Carousel.prototype = {
 		
 	_isValidObj: function(obj) {
 
-		if (null == obj) {
+		if (null === obj) {
 			return false;
 		}
-		if ("undefined" == typeof(obj) ) {
+		if ("undefined" === typeof(obj) ) {
 			return false;
 		}
 		return true;
@@ -1661,7 +1661,7 @@ var getRandom = function(max, last) {
 	var randomIndex;
 	do {
 		randomIndex = Math.floor(Math.random()*max);
-	} while(randomIndex == last);
+	} while(randomIndex === last);
 	
 	return randomIndex;
 };
diff --git a/scripts/2009/01/olpc/src/js/jquery.cluetip.js b/scripts/2009/01/olpc/src/js/jquery.cluetip.js
index 1336f261a72..25175a23cdb 100644
--- a/scripts/2009/01/olpc/src/js/jquery.cluetip.js
+++ b/scripts/2009/01/olpc/src/js/jquery.cluetip.js
@@ -60,7 +60,7 @@
 
   var $cluetip, $cluetipInner, $cluetipOuter, $cluetipTitle, $cluetipArrows, $dropShadow, imgCount;
   $.fn.cluetip = function(js, options) {
-    if (typeof js == 'object') {
+    if (typeof js === 'object') {
       options = js;
       js = null;
     }
@@ -114,9 +114,9 @@
             
       // parse the title
       var tipParts;
-      var tipTitle = (opts.attribute != 'title') ? $this.attr(opts.titleAttribute) : '';
+      var tipTitle = (opts.attribute !== 'title') ? $this.attr(opts.titleAttribute) : '';
       if (opts.splitTitle) {
-        if(tipTitle == undefined) {tipTitle = '';}
+        if(tipTitle === undefined) {tipTitle = '';}
         tipParts = tipTitle.split(opts.splitTitle);
         tipTitle = tipParts.shift();
       }
@@ -133,7 +133,7 @@
       }
       isActive = true;
       $cluetip.removeClass().css({width: tipInnerWidth});
-      if (tipAttribute == $this.attr('href')) {
+      if (tipAttribute === $this.attr('href')) {
         $this.css('cursor', opts.cursor);
       }
       $this.attr('title','');
@@ -144,12 +144,12 @@
       linkLeft = $this.offset().left;
       mouseX = event.pageX;
       mouseY = event.pageY;
-      if ($this[0].tagName.toLowerCase() != 'area') {
+      if ($this[0].tagName.toLowerCase() !== 'area') {
         sTop = $(document).scrollTop();
         winWidth = $(window).width();
       }
 // position clueTip horizontally
-      if (opts.positionBy == 'fixed') {
+      if (opts.positionBy === 'fixed') {
         posX = linkWidth + linkLeft + lOffset;
         $cluetip.css({left: posX});
       } else {
@@ -157,7 +157,7 @@
           || linkLeft + linkWidth + tipWidth + lOffset > winWidth 
           ? linkLeft - tipWidth - lOffset 
           : linkWidth + linkLeft + lOffset;
-        if ($this[0].tagName.toLowerCase() == 'area' || opts.positionBy == 'mouse' || linkWidth + tipWidth > winWidth) { // position by mouse
+        if ($this[0].tagName.toLowerCase() === 'area' || opts.positionBy === 'mouse' || linkWidth + tipWidth > winWidth) { // position by mouse
           if (mouseX + 20 + tipWidth > winWidth) {  
             $cluetip.addClass(' cluetip-' + ctClass);
             posX = (mouseX - tipWidth - lOffset) >= 0 ? mouseX - tipWidth - lOffset - parseInt($cluetip.css('marginLeft'),10) + parseInt($cluetipInner.css('marginRight'),10) :  mouseX - (tipWidth/2);
@@ -166,7 +166,7 @@
           }
         }
         var pY = posX < 0 ? event.pageY + tOffset : event.pageY;
-        $cluetip.css({left: (posX > 0 && opts.positionBy != 'bottomTop') ? posX : (mouseX + (tipWidth/2) > winWidth) ? winWidth/2 - tipWidth/2 : Math.max(mouseX - (tipWidth/2),0)});
+        $cluetip.css({left: (posX > 0 && opts.positionBy !== 'bottomTop') ? posX : (mouseX + (tipWidth/2) > winWidth) ? winWidth/2 - tipWidth/2 : Math.max(mouseX - (tipWidth/2),0)});
       }
         wHeight = $(window).height();
 
@@ -186,7 +186,7 @@
       else if (tipParts) {
         var tpl = tipParts.length;
         for (var i=0; i < tpl; i++){
-          if (i == 0) {
+          if (i === 0) {
             $cluetipInner.html(tipParts[i]);
           } else { 
             $cluetipInner.append('<div class="split-body">' + tipParts[i] + '</div>');
@@ -198,7 +198,7 @@
 * load external file via ajax          
 ***************************************/
 
-      else if (!opts.local && tipAttribute.indexOf('#') != 0) {
+      else if (!opts.local && tipAttribute.indexOf('#') !== 0) {
         if (cluetipContents && opts.ajaxCache) {
           $cluetipInner.html(cluetipContents);
           cluetipShow(pY);
@@ -266,7 +266,7 @@
       tipTitle ? $cluetipTitle.show().html(tipTitle) : (opts.showTitle) ? $cluetipTitle.show().html('&nbsp;') : $cluetipTitle.hide();
       if (opts.sticky) {
         var $closeLink = $('<div id="cluetip-close"><a href="#">' + opts.closeText + '</a></div>');
-        (opts.closePosition == 'bottom') ? $closeLink.appendTo($cluetipInner) : (opts.closePosition == 'title') ? $closeLink.prependTo($cluetipTitle) : $closeLink.prependTo($cluetipInner);
+        (opts.closePosition === 'bottom') ? $closeLink.appendTo($cluetipInner) : (opts.closePosition === 'title') ? $closeLink.prependTo($cluetipTitle) : $closeLink.prependTo($cluetipInner);
         $closeLink.on('click', function() {
           cluetipClose();
           return false;
@@ -288,13 +288,13 @@
       }
 // now that content is loaded, finish the positioning 
       var direction = '';
-      $cluetipOuter.css({overflow: defHeight == 'auto' ? 'visible' : 'auto', height: defHeight});
-      tipHeight = defHeight == 'auto' ? Math.max($cluetip.outerHeight(),$cluetip.height()) : parseInt(defHeight,10);   
+      $cluetipOuter.css({overflow: defHeight === 'auto' ? 'visible' : 'auto', height: defHeight});
+      tipHeight = defHeight === 'auto' ? Math.max($cluetip.outerHeight(),$cluetip.height()) : parseInt(defHeight,10);   
       tipY = posY;
       baseline = sTop + wHeight;
-      if (opts.positionBy == 'fixed') {
+      if (opts.positionBy === 'fixed') {
         tipY = posY - opts.dropShadowSteps + tOffset;
-      } else if ( (posX < mouseX && Math.max(posX, 0) + tipWidth > mouseX) || opts.positionBy == 'bottomTop') {
+      } else if ( (posX < mouseX && Math.max(posX, 0) + tipWidth > mouseX) || opts.positionBy === 'bottomTop') {
         if (posY + tipHeight + tOffset > baseline && mouseY - sTop > tipHeight + tOffset) { 
           tipY = mouseY - tipHeight - tOffset;
           direction = 'top';
@@ -304,12 +304,12 @@
         }
       } else if ( posY + tipHeight + tOffset > baseline ) {
         tipY = (tipHeight >= wHeight) ? sTop : baseline - tipHeight - tOffset;
-      } else if ($this.css('display') == 'block' || $this[0].tagName.toLowerCase() == 'area' || opts.positionBy == "mouse") {
+      } else if ($this.css('display') === 'block' || $this[0].tagName.toLowerCase() === 'area' || opts.positionBy === "mouse") {
         tipY = bpY - tOffset;
       } else {
         tipY = posY - opts.dropShadowSteps;
       }
-      if (direction == '') {
+      if (direction === '') {
         posX < linkLeft ? direction = 'left' : direction = 'right';
       }
       $cluetip.css({top: tipY + 'px'}).removeClass().addClass('clue-' + direction + '-' + ctClass).addClass(' cluetip-' + ctClass);
@@ -322,7 +322,7 @@
 
 // (first hide, then) ***SHOW THE CLUETIP***
       $dropShadow.hide();
-      $cluetip.hide()[opts.fx.open](opts.fx.open != 'show' && opts.fx.openSpeed);
+      $cluetip.hide()[opts.fx.open](opts.fx.open !== 'show' && opts.fx.openSpeed);
       if (opts.dropShadow) $dropShadow.css({height: tipHeight, width: tipInnerWidth}).show();
       if ($.fn.bgiframe) { $cluetip.bgiframe(); }
       // trigger the optional onShow function
@@ -379,7 +379,7 @@ clearTimeout(closeOnDelay);
           return false;
         });
   // activate by focus; inactivate by blur    
-      } else if (opts.activation == 'focus') {
+      } else if (opts.activation === 'focus') {
         $this.focus(function(event) {
           activate(event);
         });
@@ -390,13 +390,13 @@ clearTimeout(closeOnDelay);
     // clicking is returned false if cluetip url is same as href url
       } else {
         $this.on('click', function() {
-          if ($this.attr('href') && $this.attr('href') == tipAttribute && !opts.clickThrough) {
+          if ($this.attr('href') && $this.attr('href') === tipAttribute && !opts.clickThrough) {
             return false;
           }
         });
         //set up mouse tracking
         var mouseTracks = function(evt) {
-          if (opts.tracking == true) {
+          if (opts.tracking === true) {
             var trackX = posX - evt.pageX;
             var trackY = tipY ? tipY - evt.pageY : posY - evt.pageY;
             $this.mousemove(function(evt) {
diff --git a/scripts/2009/01/olpc/src/js/jquery.easing.1.3.js b/scripts/2009/01/olpc/src/js/jquery.easing.1.3.js
index ef743210795..7ed322d21a0 100644
--- a/scripts/2009/01/olpc/src/js/jquery.easing.1.3.js
+++ b/scripts/2009/01/olpc/src/js/jquery.easing.1.3.js
@@ -139,15 +139,15 @@ jQuery.extend( jQuery.easing,
 		return a*Math.pow(2,-10*(t-=1)) * Math.sin( (t*d-s)*(2*Math.PI)/p )*.5 + c + b;
 	},
 	easeInBack: function (x, t, b, c, d, s) {
-		if (s == undefined) s = 1.70158;
+		if (s === undefined) s = 1.70158;
 		return c*(t/=d)*t*((s+1)*t - s) + b;
 	},
 	easeOutBack: function (x, t, b, c, d, s) {
-		if (s == undefined) s = 1.70158;
+		if (s === undefined) s = 1.70158;
 		return c*((t=t/d-1)*t*((s+1)*t + s) + 1) + b;
 	},
 	easeInOutBack: function (x, t, b, c, d, s) {
-		if (s == undefined) s = 1.70158; 
+		if (s === undefined) s = 1.70158; 
 		if ((t/=d/2) < 1) return c/2*(t*t*(((s*=(1.525))+1)*t - s)) + b;
 		return c/2*((t-=2)*t*(((s*=(1.525))+1)*t + s) + 2) + b;
 	},
diff --git a/scripts/2009/01/olpc/src/js/jquery.hoverIntent.js b/scripts/2009/01/olpc/src/js/jquery.hoverIntent.js
index dfcdf262d81..c69c9eb491c 100644
--- a/scripts/2009/01/olpc/src/js/jquery.hoverIntent.js
+++ b/scripts/2009/01/olpc/src/js/jquery.hoverIntent.js
@@ -77,9 +77,9 @@
 		// A private function for handling mouse 'hovering'
 		var handleHover = function(e) {
 			// next three lines copied from jQuery.hover, ignore children onMouseOver/onMouseOut
-			var p = (e.type == "mouseover" ? e.fromElement : e.toElement) || e.relatedTarget;
-			while ( p && p != this ) { try { p = p.parentNode; } catch(e) { p = this; } }
-			if ( p == this ) { return false; }
+			var p = (e.type === "mouseover" ? e.fromElement : e.toElement) || e.relatedTarget;
+			while ( p && p !== this ) { try { p = p.parentNode; } catch(e) { p = this; } }
+			if ( p === this ) { return false; }
 
 			// copy objects to be passed into t (required for event object to be passed in IE)
 			var ev = jQuery.extend({},e);
@@ -88,21 +88,21 @@
 			// cancel hoverIntent timer if it exists
 			if (ob.hoverIntent_t) { ob.hoverIntent_t = clearTimeout(ob.hoverIntent_t); }
 
-			// else e.type == "onmouseover"
-			if (e.type == "mouseover") {
+			// else e.type === "onmouseover"
+			if (e.type === "mouseover") {
 				// set "previous" X and Y position based on initial entry point
 				pX = ev.pageX; pY = ev.pageY;
 				// update "current" X and Y position based on mousemove
 				$(ob).bind("mousemove",track);
 				// start polling interval (self-calling timeout) to compare mouse coordinates over time
-				if (ob.hoverIntent_s != 1) { ob.hoverIntent_t = setTimeout( function(){compare(ev,ob);} , cfg.interval );}
+				if (ob.hoverIntent_s !== 1) { ob.hoverIntent_t = setTimeout( function(){compare(ev,ob);} , cfg.interval );}
 
-			// else e.type == "onmouseout"
+			// else e.type === "onmouseout"
 			} else {
 				// unbind expensive mousemove event
 				$(ob).unbind("mousemove",track);
 				// if hoverIntent state is true, then call the mouseOut function after the specified delay
-				if (ob.hoverIntent_s == 1) { ob.hoverIntent_t = setTimeout( function(){delay(ev,ob);} , cfg.timeout );}
+				if (ob.hoverIntent_s === 1) { ob.hoverIntent_t = setTimeout( function(){delay(ev,ob);} , cfg.timeout );}
 			}
 		};
 
diff --git a/scripts/2009/01/olpc/src/js/olpcbook.js b/scripts/2009/01/olpc/src/js/olpcbook.js
index 272a8f27680..bbdfcbc3d1e 100644
--- a/scripts/2009/01/olpc/src/js/olpcbook.js
+++ b/scripts/2009/01/olpc/src/js/olpcbook.js
@@ -18,7 +18,7 @@ gb.getPageURI = function(index) {
 }
 
 gb.getPageSide = function(index) {
-    if (0 == (index & 0x1)) {
+    if (0 === (index & 0x1)) {
         return 'R';
     } else {
         return 'L';
diff --git a/tests/unit/js/test.search.js b/tests/unit/js/test.search.js
index 7a4ef19cc01..3be9c4627fd 100644
--- a/tests/unit/js/test.search.js
+++ b/tests/unit/js/test.search.js
@@ -43,11 +43,11 @@ function createSearchFacets(totalFacet = 2, visibleFacet = 2, minVisibleFacet =
         </div>
     `;
 
-    if (visibleFacet == minVisibleFacet) {
+    if (visibleFacet === minVisibleFacet) {
         divTestFacet.querySelector('#test_bull').style.display = 'none';
         divTestFacet.querySelector('#test_less').style.display = 'none';
     }
-    if (visibleFacet == totalFacet) {
+    if (visibleFacet === totalFacet) {
         divTestFacet.querySelector('#test_more').style.display = 'none';
         divTestFacet.querySelector('#test_bull').style.display = 'none';
     }
diff --git a/vendor/js/jquery-form/jquery.form.js b/vendor/js/jquery-form/jquery.form.js
index 2a1e0e22bc3..cce28781dfc 100644
--- a/vendor/js/jquery-form/jquery.form.js
+++ b/vendor/js/jquery-form/jquery.form.js
@@ -50,7 +50,7 @@ $.fn.ajaxSubmit = function(options) {
 		return this;
 	}
 
-	if (typeof options == 'function')
+	if (typeof options === 'function')
 		options = { success: options };
 
 	var url = $.trim(this.attr('action'));
@@ -109,7 +109,7 @@ $.fn.ajaxSubmit = function(options) {
 
 	var q = $.param(a);
 
-	if (options.type.toUpperCase() == 'GET') {
+	if (options.type.toUpperCase() === 'GET') {
 		options.url += (options.url.indexOf('?') >= 0 ? '&' : '?') + q;
 		options.data = null;  // data is null for 'get'
 	}
@@ -144,7 +144,7 @@ $.fn.ajaxSubmit = function(options) {
 
 	var multipart = false;
 //	var mp = 'multipart/form-data';
-//	multipart = ($form.attr('enctype') == mp || $form.attr('encoding') == mp);
+//	multipart = ($form.attr('enctype') === mp || $form.attr('encoding') === mp);
 
 	// options.iframe allows user to force iframe mode
 	// 06-NOV-09: now defaulting to iframe mode if file input is detected
@@ -219,7 +219,7 @@ $.fn.ajaxSubmit = function(options) {
 			if (n && !sub.disabled) {
 				options.extraData = options.extraData || {};
 				options.extraData[n] = sub.value;
-				if (sub.type == "image") {
+				if (sub.type === "image") {
 					options.extraData[name+'.x'] = form.clk_x;
 					options.extraData[name+'.y'] = form.clk_y;
 				}
@@ -233,9 +233,9 @@ $.fn.ajaxSubmit = function(options) {
 
 			// update form attrs in IE friendly way
 			form.setAttribute('target',id);
-			if (form.getAttribute('method') != 'POST')
+			if (form.getAttribute('method') !== 'POST')
 				form.setAttribute('method', 'POST');
-			if (form.getAttribute('action') != opts.url)
+			if (form.getAttribute('action') !== opts.url)
 				form.setAttribute('action', opts.url);
 
 			// ie borks in some cases when setting encoding
@@ -287,9 +287,9 @@ $.fn.ajaxSubmit = function(options) {
 
 				doc = io.contentWindow ? io.contentWindow.document : io.contentDocument ? io.contentDocument : io.document;
 				
-				var isXml = opts.dataType == 'xml' || doc.XMLDocument || $.isXMLDoc(doc);
+				var isXml = opts.dataType === 'xml' || doc.XMLDocument || $.isXMLDoc(doc);
 				log('isXml='+isXml);
-				if (!isXml && (doc.body == null || doc.body.innerHTML == '')) {
+				if (!isXml && (doc.body === null || doc.body.innerHTML === '')) {
 				 	if (--domCheckCount) {
 						// in some browsers (Opera) the iframe DOM is not always traversable when
 						// the onload callback fires, so we loop a bit to accommodate
@@ -308,7 +308,7 @@ $.fn.ajaxSubmit = function(options) {
 					return headers[header];
 				};
 
-				if (opts.dataType == 'json' || opts.dataType == 'script') {
+				if (opts.dataType === 'json' || opts.dataType === 'script') {
 					// see if user embedded response in textarea
 					var ta = doc.getElementsByTagName('textarea')[0];
 					if (ta)
@@ -320,7 +320,7 @@ $.fn.ajaxSubmit = function(options) {
 							xhr.responseText = pre.innerHTML;
 					}			  
 				}
-				else if (opts.dataType == 'xml' && !xhr.responseXML && xhr.responseText != null) {
+				else if (opts.dataType === 'xml' && !xhr.responseXML && xhr.responseText !== null) {
 					xhr.responseXML = toXml(xhr.responseText);
 				}
 				data = $.httpData(xhr, opts.dataType);
@@ -354,7 +354,7 @@ $.fn.ajaxSubmit = function(options) {
 			}
 			else
 				doc = (new DOMParser()).parseFromString(s, 'text/xml');
-			return (doc && doc.documentElement && doc.documentElement.tagName != 'parsererror') ? doc : null;
+			return (doc && doc.documentElement && doc.documentElement.tagName !== 'parsererror') ? doc : null;
 		};
 	};
 };
@@ -384,17 +384,17 @@ $.fn.ajaxForm = function(options) {
 		if (!($el.is(":submit,input:image"))) {
 			// is this a child element of the submit el?  (ex: a span within a button)
 			var t = $el.closest(':submit');
-			if (t.length == 0)
+			if (t.length === 0)
 				return;
 			target = t[0];
 		}
 		var form = this;
 		form.clk = target;
-		if (target.type == 'image') {
-			if (e.offsetX != undefined) {
+		if (target.type === 'image') {
+			if (e.offsetX !== undefined) {
 				form.clk_x = e.offsetX;
 				form.clk_y = e.offsetY;
-			} else if (typeof $.fn.offset == 'function') { // try to use dimensions plugin
+			} else if (typeof $.fn.offset === 'function') { // try to use dimensions plugin
 				var offset = $el.offset();
 				form.clk_x = e.pageX - offset.left;
 				form.clk_y = e.pageY - offset.top;
@@ -426,7 +426,7 @@ $.fn.ajaxFormUnbind = function() {
  */
 $.fn.formToArray = function(semantic) {
 	var a = [];
-	if (this.length == 0) return a;
+	if (this.length === 0) return a;
 
 	var form = this[0];
 	var els = semantic ? form.getElementsByTagName('*') : form.elements;
@@ -436,9 +436,9 @@ $.fn.formToArray = function(semantic) {
 		var n = el.name;
 		if (!n) continue;
 
-		if (semantic && form.clk && el.type == "image") {
-			// handle image inputs on the fly when semantic == true
-			if(!el.disabled && form.clk == el) {
+		if (semantic && form.clk && el.type === "image") {
+			// handle image inputs on the fly when semantic === true
+			if(!el.disabled && form.clk === el) {
 				a.push({name: n, value: $(el).val()});
 				a.push({name: n+'.x', value: form.clk_x}, {name: n+'.y', value: form.clk_y});
 			}
@@ -446,18 +446,18 @@ $.fn.formToArray = function(semantic) {
 		}
 
 		var v = $.fieldValue(el, true);
-		if (v && v.constructor == Array) {
+		if (v && v.constructor === Array) {
 			for(var j=0, jmax=v.length; j < jmax; j++)
 				a.push({name: n, value: v[j]});
 		}
-		else if (v !== null && typeof v != 'undefined')
+		else if (v !== null && typeof v !== 'undefined')
 			a.push({name: n, value: v});
 	}
 
 	if (!semantic && form.clk) {
 		// input type=='image' are not found in elements array! handle it here
 		var $input = $(form.clk), input = $input[0], n = input.name;
-		if (n && !input.disabled && input.type == 'image') {
+		if (n && !input.disabled && input.type === 'image') {
 			a.push({name: n, value: $input.val()});
 			a.push({name: n+'.x', value: form.clk_x}, {name: n+'.y', value: form.clk_y});
 		}
@@ -484,11 +484,11 @@ $.fn.fieldSerialize = function(successful) {
 		var n = this.name;
 		if (!n) return;
 		var v = $.fieldValue(this, successful);
-		if (v && v.constructor == Array) {
+		if (v && v.constructor === Array) {
 			for (var i=0,max=v.length; i < max; i++)
 				a.push({name: n, value: v[i]});
 		}
-		else if (v !== null && typeof v != 'undefined')
+		else if (v !== null && typeof v !== 'undefined')
 			a.push({name: this.name, value: v});
 	});
 	//hand off to jQuery.param for proper encoding
@@ -509,21 +509,21 @@ $.fn.fieldSerialize = function(successful) {
  *
  *  var v = $(':text').fieldValue();
  *  // if no values are entered into the text inputs
- *  v == ['','']
+ *  v === ['','']
  *  // if values entered into the text inputs are 'foo' and 'bar'
- *  v == ['foo','bar']
+ *  v === ['foo','bar']
  *
  *  var v = $(':checkbox').fieldValue();
  *  // if neither checkbox is checked
  *  v === undefined
  *  // if both checkboxes are checked
- *  v == ['B1', 'B2']
+ *  v === ['B1', 'B2']
  *
  *  var v = $(':radio').fieldValue();
  *  // if neither radio is checked
  *  v === undefined
  *  // if first radio is checked
- *  v == ['C1']
+ *  v === ['C1']
  *
  * The successful argument controls whether or not the field element must be 'successful'
  * (per http://www.w3.org/TR/html4/interact/forms.html#successful-controls).
@@ -537,9 +537,9 @@ $.fn.fieldValue = function(successful) {
 	for (var val=[], i=0, max=this.length; i < max; i++) {
 		var el = this[i];
 		var v = $.fieldValue(el, successful);
-		if (v === null || typeof v == 'undefined' || (v.constructor == Array && !v.length))
+		if (v === null || typeof v === 'undefined' || (v.constructor === Array && !v.length))
 			continue;
-		v.constructor == Array ? $.merge(val, v) : val.push(v);
+		v.constructor === Array ? $.merge(val, v) : val.push(v);
 	}
 	return val;
 };
@@ -549,19 +549,19 @@ $.fn.fieldValue = function(successful) {
  */
 $.fieldValue = function(el, successful) {
 	var n = el.name, t = el.type, tag = el.tagName.toLowerCase();
-	if (typeof successful == 'undefined') successful = true;
+	if (typeof successful === 'undefined') successful = true;
 
-	if (successful && (!n || el.disabled || t == 'reset' || t == 'button' ||
-		(t == 'checkbox' || t == 'radio') && !el.checked ||
-		(t == 'submit' || t == 'image') && el.form && el.form.clk != el ||
-		tag == 'select' && el.selectedIndex == -1))
+	if (successful && (!n || el.disabled || t === 'reset' || t === 'button' ||
+		(t === 'checkbox' || t === 'radio') && !el.checked ||
+		(t === 'submit' || t === 'image') && el.form && el.form.clk !== el ||
+		tag === 'select' && el.selectedIndex === -1))
 			return null;
 
-	if (tag == 'select') {
+	if (tag === 'select') {
 		var index = el.selectedIndex;
 		if (index < 0) return null;
 		var a = [], ops = el.options;
-		var one = (t == 'select-one');
+		var one = (t === 'select-one');
 		var max = (one ? index+1 : ops.length);
 		for(var i=(one ? index : 0); i < max; i++) {
 			var op = ops[i];
@@ -598,11 +598,11 @@ $.fn.clearForm = function() {
 $.fn.clearFields = $.fn.clearInputs = function() {
 	return this.each(function() {
 		var t = this.type, tag = this.tagName.toLowerCase();
-		if (t == 'text' || t == 'password' || tag == 'textarea')
+		if (t === 'text' || t === 'password' || tag === 'textarea')
 			this.value = '';
-		else if (t == 'checkbox' || t == 'radio')
+		else if (t === 'checkbox' || t === 'radio')
 			this.checked = false;
-		else if (tag == 'select')
+		else if (tag === 'select')
 			this.selectedIndex = -1;
 	});
 };
@@ -614,7 +614,7 @@ $.fn.resetForm = function() {
 	return this.each(function() {
 		// guard against an input with the name of 'reset'
 		// note that IE reports the reset function as an 'object'
-		if (typeof this.reset == 'function' || (typeof this.reset == 'object' && !this.reset.nodeType))
+		if (typeof this.reset === 'function' || (typeof this.reset === 'object' && !this.reset.nodeType))
 			this.reset();
 	});
 };
@@ -623,7 +623,7 @@ $.fn.resetForm = function() {
  * Enables or disables any matching elements.
  */
 $.fn.enable = function(b) {
-	if (b == undefined) b = true;
+	if (b === undefined) b = true;
 	return this.each(function() {
 		this.disabled = !b;
 	});
@@ -634,14 +634,14 @@ $.fn.enable = function(b) {
  * selects/deselects and matching option elements.
  */
 $.fn.selected = function(select) {
-	if (select == undefined) select = true;
+	if (select === undefined) select = true;
 	return this.each(function() {
 		var t = this.type;
-		if (t == 'checkbox' || t == 'radio')
+		if (t === 'checkbox' || t === 'radio')
 			this.checked = select;
-		else if (this.tagName.toLowerCase() == 'option') {
+		else if (this.tagName.toLowerCase() === 'option') {
 			var $sel = $(this).parent('select');
-			if (select && $sel[0] && $sel[0].type == 'select-one') {
+			if (select && $sel[0] && $sel[0].type === 'select-one') {
 				// deselect all other options
 				$sel.find('option').selected(false);
 			}