Manual backport - Respect alphabetical order of collections in "Move" modal (#31355) (#31549)

* Backport changes from #31355
- Skip tests because the setup depends on refactoring done in master
- See discussion: https://metaboat.slack.com/archives/C505ZNNH4/p1686648256978279

* Remove unused code

frontend/src/metabase/entities/collections/collections.ts

@@ -80,10 +80,13 @@ const Collections = createEntity({
 
   selectors: {
     getExpandedCollectionsById: createSelector(
-      [state => state.entities.collections || {}, getUserPersonalCollectionId],
+      [
+        state => Collections.selectors.getList(state),
+        getUserPersonalCollectionId,
+      ],
       (collections, currentUserPersonalCollectionId) =>
         getExpandedCollectionsById(
-          Object.values(collections),
+          collections || [],
           currentUserPersonalCollectionId,
         ),
     ),

frontend/src/metabase/entities/snippet-collections.js

@@ -45,9 +45,8 @@ const SnippetCollections = createEntity({
 
   selectors: {
     getExpandedCollectionsById: createSelector(
-      state => state.entities.snippetCollections || {},
-      collections =>
-        getExpandedCollectionsById(Object.values(collections), null),
+      state => SnippetCollections.selectors.getList(state) || [],
+      collections => getExpandedCollectionsById(collections, null),
     ),
   },