Merge pull request #3564 from StoDevX/fix-course-search

hawkrives · web-flow · commit a3586987a024 · 2019-03-17T12:53:24.000-05:00
Fix course search
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -66,6 +66,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 - Fixed the dictionary editor and made it handle user input again (#3383, #3387)
 - Fixed DatePicker by removing an unnecessary call to getDerivedStateFromProps (#3382)
 - Fixed the balances view from refreshing if the user has not agreed to the alert (#3509)
+- Fixed course search crash (#3564)
 
 ### Removed
 - Removed the `prepare` script patching `ScrollEnabled` inside `RCTMultilineTextInputView` (#3337)
diff --git a/source/views/sis/course-search/lib/execute-search.js b/source/views/sis/course-search/lib/execute-search.js
@@ -52,5 +52,5 @@ export function sortAndGroupResults(results: Array<Course>) {
 
 	sortedAgain.reverse()
 
-	return sorted
+	return sortedAgain
 }

Original file line number	Diff line number	Diff line change
`@@ -52,5 +52,5 @@ export function sortAndGroupResults(results: Array<Course>) {`
`52`	`52`
`53`	`53`	`sortedAgain.reverse()`
`54`	`54`
`55`		`- return sorted`
	`55`	`+ return sortedAgain`
`56`	`56`	`}`