fix: When sorting by filename the extension should only be considered…

… if the basename is equal

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

__tests__/utils/fileSorting.spec.ts

@@ -224,4 +224,42 @@ describe('sortNodes', () => {
 
 		expect(sortNodes(array, { sortingMode: FilesSortingMode.Modified, sortingOrder: 'desc' })).toEqual(['a', 'b', 'c'])
 	})
+
+	test('Names with underscores are sorted correctly', () => {
+		const array = [
+			file('file_1.txt'),
+			file('file_3.txt'),
+			file('file.txt'),
+			file('file_2.txt'),
+		] as const
+
+		expect(
+			sortNodes(
+				array,
+				{
+					sortingMode: FilesSortingMode.Name,
+					sortingOrder: 'asc',
+				},
+			),
+		).toEqual(['file.txt', 'file_1.txt', 'file_2.txt', 'file_3.txt'])
+	})
+
+	test('Names are sorted correctly by extension', () => {
+		const array = [
+			file('file.b'),
+			file('file.c'),
+			file('file.a'),
+			file('file.d'),
+		] as const
+
+		expect(
+			sortNodes(
+				array,
+				{
+					sortingMode: FilesSortingMode.Name,
+					sortingOrder: 'asc',
+				},
+			),
+		).toEqual(['file.a', 'file.b', 'file.c', 'file.d'])
+	})
 })

lib/utils/fileSorting.ts

@@ -37,7 +37,7 @@ export interface FilesSortingOptions {
  * @param nodes Nodes to sort
  * @param options Sorting options
  */
-export function sortNodes(nodes: Node[], options: FilesSortingOptions = {}): Node[] {
+export function sortNodes(nodes: readonly Node[], options: FilesSortingOptions = {}): Node[] {
 	const sortingOptions = {
 		// Default to sort by name
 		sortingMode: FilesSortingMode.Name,
@@ -46,6 +46,12 @@ export function sortNodes(nodes: Node[], options: FilesSortingOptions = {}): Nod
 		...options,
 	}
 
+	/**
+	 * Get the basename without any extension
+	 * @param name The filename to extract the basename from
+	 */
+	const basename = (name: string) => name.lastIndexOf('.') > 0 ? name.slice(0, name.lastIndexOf('.')) : name
+
 	const identifiers = [
 		// 1: Sort favorites first if enabled
 		...(sortingOptions.sortFavoritesFirst ? [(v: Node) => v.attributes?.favorite !== 1] : []),
@@ -54,7 +60,7 @@ export function sortNodes(nodes: Node[], options: FilesSortingOptions = {}): Nod
 		// 3: Use sorting mode if NOT basename (to be able to use displayName too)
 		...(sortingOptions.sortingMode !== FilesSortingMode.Name ? [(v: Node) => v[sortingOptions.sortingMode]] : []),
 		// 4: Use displayName if available, fallback to name
-		(v: Node) => v.attributes?.displayName || v.basename,
+		(v: Node) => basename(v.attributes?.displayName || v.basename),
 		// 5: Finally, use basename if all previous sorting methods failed
 		(v: Node) => v.basename,
 	]