Albums: Reload pictures when the album sort order has changed #405

Signed-off-by: Michael Mayer <michael@photoprism.app>

Author: lastzero

frontend/src/page/album/photos.vue

@@ -105,12 +105,11 @@ export default {
     const uid = this.$route.params.album;
     const query = this.$route.query;
     const routeName = this.$route.name;
-    const order = query["order"] ? query["order"] : "oldest";
     const camera = query["camera"] ? parseInt(query["camera"]) : 0;
     const q = query["q"] ? query["q"] : "";
     const country = query["country"] ? query["country"] : "";
     const view = this.getViewType();
-    const filter = { country: country, camera: camera, order: order, q: q };
+    const filter = { country: country, camera: camera, q: q };
     const settings = { view: view };
     const batchSize = Photo.batchSize();
 
@@ -136,6 +135,7 @@ export default {
       settings: settings,
       filter: filter,
       lastFilter: {},
+      lastParams: {},
       routeName: routeName,
       collectionRoute: this.$route.meta?.collectionRoute ? this.$route.meta.collectionRoute : "albums",
       loading: true,
@@ -185,7 +185,7 @@ export default {
       const routeChanged = this.routeName !== this.$route.name;
 
       if (routeChanged) {
-        this.lastFilter = {};
+        this.resetLastFilter();
       }
 
       this.routeName = this.$route.name;
@@ -230,6 +230,9 @@ export default {
     this.$view.leave(this);
   },
   methods: {
+    resetLastFilter() {
+      this.lastFilter = {};
+    },
     onCtrl(ev) {
       if (!ev || !(ev instanceof KeyboardEvent) || !ev.ctrlKey || !this.$view.isActive(this)) {
         return;
@@ -263,6 +266,10 @@ export default {
 
       return "cards";
     },
+    getSortOrder() {
+      const query = this.$route.query;
+      return query["order"] ? query["order"] : this.model?.Order;
+    },
     openDate(index) {
       if (!this.canEdit) {
         return this.openPhoto(index);
@@ -348,6 +355,7 @@ export default {
         offset: offset,
         s: this.uid,
         merged: true,
+        order: this.getSortOrder(),
       };
 
       Object.assign(params, this.lastFilter);
@@ -356,6 +364,8 @@ export default {
         Object.assign(params, this.staticFilter);
       }
 
+      this.lastParams = params;
+
       Photo.search(params)
         .then((response) => {
           this.results = Photo.mergeResponse(this.results, response);
@@ -436,10 +446,6 @@ export default {
     updateQuery(props) {
       this.updateFilter(props);
 
-      if (this.model.Order !== this.filter.order) {
-        this.model.Order = this.filter.order;
-      }
-
       if (this.loading) {
         return;
       }
@@ -468,6 +474,7 @@ export default {
         offset: this.offset,
         s: this.uid,
         merged: true,
+        order: this.getSortOrder(),
       };
 
       Object.assign(params, this.filter);
@@ -481,15 +488,17 @@ export default {
     refresh(props) {
       this.updateSettings(props);
 
-      if (this.loading) return;
+      if (this.loading) {
+        return;
+      }
 
       this.loading = true;
       this.page = 0;
       this.dirty = true;
       this.complete = false;
       this.scrollDisabled = false;
 
-      this.loadMore();
+      this.loadMore(true);
     },
     search() {
       /**
@@ -519,6 +528,8 @@ export default {
 
       const params = this.searchParams();
 
+      this.lastParams = params;
+
       Photo.search(params)
         .then((response) => {
           // Hide search toolbar expansion panel when matching pictures were found.
@@ -564,7 +575,6 @@ export default {
         .then((m) => {
           this.model = m;
 
-          this.filter.order = m.Order;
           window.document.title = `${this.$config.get("siteTitle")}: ${this.model.Title}`;
 
           return Promise.resolve(this.model);
@@ -575,7 +585,9 @@ export default {
         });
     },
     onAlbumsUpdated(ev, data) {
-      if (!this.listen) return;
+      if (!this.listen) {
+        return;
+      }
 
       if (!data || !data.entities || !Array.isArray(data.entities)) {
         return;
@@ -597,10 +609,8 @@ export default {
           this.complete = false;
           this.scrollDisabled = false;
 
-          if (this.filter.order !== this.model.Order) {
-            this.filter.order = this.model.Order;
+          if (this.lastParams?.order !== this.model?.Order) {
             this.updateQuery();
-          } else {
             this.loadMore(true);
           }
 

frontend/src/page/photos.vue

@@ -570,7 +570,9 @@ export default {
     updateQuery(props) {
       this.updateFilter(props);
 
-      if (this.loading) return;
+      if (this.loading) {
+        return;
+      }
 
       const query = {
         view: this.settings.view,
@@ -608,15 +610,17 @@ export default {
     refresh(props) {
       this.updateSettings(props);
 
-      if (this.loading) return;
+      if (this.loading) {
+        return;
+      }
 
       this.loading = true;
       this.page = 0;
       this.dirty = true;
       this.complete = false;
       this.scrollDisabled = false;
 
-      this.loadMore();
+      this.loadMore(true);
     },
     search() {
       /**