feat(ltFileDetails): save all

app/api/ltDataAuth.ts

@@ -21,9 +21,8 @@ export function getUserInfo() {
   }).json();
 }
 
-export function editLocation(title: CommonsFile, coordinates: LatLng) {
+export function editLocation(title: CommonsFile, coordinates: LatLng[]) {
   const {pageid} = title;
-  const {type, lat, lng} = coordinates;
   return useFetch<EditApiResponse>(
     '/edit',
     {
@@ -32,7 +31,7 @@ export function editLocation(title: CommonsFile, coordinates: LatLng) {
         'Content-Type': 'application/json',
         'X-XSRF-TOKEN': xsrfToken() || ''
       },
-      body: JSON.stringify({type, lat, lng, pageid})
+      body: JSON.stringify(coordinates.map(({type, lat, lng}) => ({type, lat, lng, pageid})))
     },
     {
       afterFetch(ctx) {

app/components/ltFileDetails.vue

@@ -1,5 +1,16 @@
 <template>
-  <div class="card">
+  <button
+    class="btn btn-success form-control"
+    type="button"
+    :disabled="!(file.coordinates?.isChanged || file.objectLocation?.isChanged)"
+    @click="editLocation()"
+  >
+    <svg class="octicon">
+      <use xlink:href="#git-commit"></use>
+    </svg>
+    {{ t('Save') }}
+  </button>
+  <div class="card mt-2">
     <div class="card-body p-2">
       <div>
         <label @click="collapseCameraLocation = !collapseCameraLocation">
@@ -117,17 +128,28 @@ const msgErrorStatusText = computed(() =>
 
 watch(props.file.coordinates, (c1, c2) => console.log(c1, c2));
 
-function editLocation(coordinates: LatLng) {
+function editLocation(coordinates: LatLng[]) {
   error.value = undefined;
+  if (coordinates === undefined) {
+    coordinates = [];
+    if (props.file?.coordinates?.isChanged) {
+      coordinates.push(props.file.coordinates);
+    }
+    if (props.file?.objectLocation?.isChanged) {
+      coordinates.push(props.file.objectLocation);
+    }
+  }
   return ltDataAuth.editLocation(props.file, coordinates).then(
     () => {
-      if (coordinates.type === 'Location') {
-        // eslint-disable-next-line vue/no-mutating-props
-        props.file.coordinates = coordinates.commit();
-      } else if (coordinates.type === 'Object location') {
-        // eslint-disable-next-line vue/no-mutating-props
-        props.file.objectLocation = coordinates.commit();
-      }
+      coordinates.forEach(c => {
+        if (c.type === 'Location') {
+          // eslint-disable-next-line vue/no-mutating-props
+          props.file.coordinates = c.commit();
+        } else if (c.type === 'Object location') {
+          // eslint-disable-next-line vue/no-mutating-props
+          props.file.objectLocation = c.commit();
+        }
+      });
     },
     e => {
       error.value = e;

app/components/ltLocationInput.vue

@@ -12,7 +12,7 @@
       "
     />
     <button
-      class="btn btn-secondary"
+      class="btn btn-outline-secondary"
       type="button"
       :title="t('Discard')"
       @click="modelValue && emit('update:modelValue', modelValue.rollback())"
@@ -22,7 +22,7 @@
       </svg>
     </button>
     <button
-      class="btn btn-success"
+      class="btn btn-outline-success"
       type="button"
       :title="t('Save')"
       :disabled="!modelValue?.isChanged"

backend/app.py

@@ -1,6 +1,7 @@
 import json
 import logging
 import os
+import typing
 
 from flask import Flask, abort, jsonify, request
 from flask_mwoauth import MWOAuth
@@ -49,12 +50,19 @@ def user():
     return r
 
 
+class EditRequest(typing.TypedDict):
+    type: typing.Literal["Location", "Object location"]
+    lat: int
+    lng: int
+    pageid: int
+
+
 @app.route("/edit", methods=["POST"])
 def edit():
     if not mwoauth.get_current_user():
         abort(401)
 
-    data = request.get_json()
+    data: list[EditRequest] = request.get_json()
     if (
         "pageid" not in data
         or "type" not in data