Skip to content

Commit

Permalink
Remove asciiOnlyToLowerCase
Browse files Browse the repository at this point in the history
  • Loading branch information
advplyr committed Oct 8, 2024
1 parent e6db149 commit 0adceaa
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 25 deletions.
3 changes: 1 addition & 2 deletions server/controllers/LibraryController.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ const libraryItemsBookFilters = require('../utils/queries/libraryItemsBookFilter
const libraryItemFilters = require('../utils/queries/libraryItemFilters')
const seriesFilters = require('../utils/queries/seriesFilters')
const fileUtils = require('../utils/fileUtils')
const { asciiOnlyToLowerCase } = require('../utils/index')
const { createNewSortInstance } = require('../libs/fastSort')
const naturalSort = createNewSortInstance({
comparer: new Intl.Collator(undefined, { numeric: true, sensitivity: 'base' }).compare
Expand Down Expand Up @@ -809,7 +808,7 @@ class LibraryController {
}

const limit = req.query.limit || 12
const query = asciiOnlyToLowerCase(req.query.q.trim())
const query = req.query.q.trim()

const matches = await libraryItemFilters.search(req.user, req.library, query, limit)
res.json(matches)
Expand Down
23 changes: 0 additions & 23 deletions server/utils/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -194,29 +194,6 @@ module.exports.getTitlePrefixAtEnd = (title) => {
return prefix ? `${sort}, ${prefix}` : title
}

/**
* to lower case for only ascii characters
* used to handle sqlite that doesnt support unicode lower
* @see https://github.com/advplyr/audiobookshelf/issues/2187
*
* @param {string} str
* @returns {string}
*/
module.exports.asciiOnlyToLowerCase = (str) => {
if (!str) return ''

let temp = ''
for (let chars of str) {
let value = chars.charCodeAt()
if (value >= 65 && value <= 90) {
temp += String.fromCharCode(value + 32)
} else {
temp += chars
}
}
return temp
}

/**
* Escape string used in RegExp
* @see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions#escaping
Expand Down

0 comments on commit 0adceaa

Please sign in to comment.