From a23a2d7d9c6948c8f83231955afc31233820663e Mon Sep 17 00:00:00 2001
From: "P. J. Reed" <phillipreed@hatchbed.com>
Date: Wed, 3 Nov 2021 14:42:44 -0500
Subject: [PATCH] Fix AdminController scanning functions

These got accidentally broken during the storage backend refactoring.
---
 CHANGELOG.md                                                | 3 +++
 pom.xml                                                     | 2 +-
 .../com/github/swrirobotics/bags/storage/BagScanner.java    | 3 +++
 .../bags/storage/filesystem/FilesystemBagStorageImpl.java   | 6 +-----
 src/main/webapp/resources/js/views/AboutWindow.js           | 2 +-
 5 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index bdc1aaf3..24ae3477 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,8 @@
 # Bag Database changelog
 
+3.4.2
+- Fix admin functions for forcing bag scanning
+
 3.4.1
 - Increase maximum bag file name length to 255 characters
 - Don't prevent adding bags if the bag removal task fails
diff --git a/pom.xml b/pom.xml
index 3b7d4d68..36b15d7b 100644
--- a/pom.xml
+++ b/pom.xml
@@ -7,7 +7,7 @@
     <groupId>com.github.swri-robotics</groupId>
     <artifactId>bag-database</artifactId>
     <packaging>war</packaging>
-    <version>3.4.1</version>
+    <version>3.4.2</version>
     <name>Bag Database</name>
     <properties>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
diff --git a/src/main/java/com/github/swrirobotics/bags/storage/BagScanner.java b/src/main/java/com/github/swrirobotics/bags/storage/BagScanner.java
index ef2c3cbc..c979ed33 100644
--- a/src/main/java/com/github/swrirobotics/bags/storage/BagScanner.java
+++ b/src/main/java/com/github/swrirobotics/bags/storage/BagScanner.java
@@ -44,6 +44,7 @@
 import com.github.swrirobotics.remote.GeocodingService;
 import com.github.swrirobotics.status.Status;
 import com.github.swrirobotics.status.StatusProvider;
+import com.google.common.base.Joiner;
 import com.google.common.collect.Maps;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -118,6 +119,7 @@ public void initialize() {
         myBagService.deleteUnownedBags();
 
         for (BagStorage storage : storages) {
+            myBagStorages.put(storage.getStorageId(), storage);
             storage.addChangeListener(this);
             scanStorage(storage, false);
         }
@@ -340,6 +342,7 @@ public void updateAllTags() {
     }
 
     public void scanAllStorages(boolean forceUpdate) {
+        myLogger.info("Scanning all storage backends: " + Joiner.on(',').join(myBagStorages.keySet()));
         for (BagStorage storage : myBagStorages.values()) {
             scanStorage(storage, forceUpdate);
         }
diff --git a/src/main/java/com/github/swrirobotics/bags/storage/filesystem/FilesystemBagStorageImpl.java b/src/main/java/com/github/swrirobotics/bags/storage/filesystem/FilesystemBagStorageImpl.java
index d37cf1f4..e263bc1c 100644
--- a/src/main/java/com/github/swrirobotics/bags/storage/filesystem/FilesystemBagStorageImpl.java
+++ b/src/main/java/com/github/swrirobotics/bags/storage/filesystem/FilesystemBagStorageImpl.java
@@ -35,7 +35,6 @@
 import com.github.swrirobotics.bags.storage.*;
 import com.github.swrirobotics.bags.storage.filesystem.watcher.DefaultRecursiveWatcher;
 import com.github.swrirobotics.bags.storage.filesystem.watcher.RecursiveWatcher;
-import com.github.swrirobotics.config.ConfigService;
 import com.github.swrirobotics.persistence.Bag;
 import com.github.swrirobotics.persistence.BagRepository;
 import com.github.swrirobotics.status.Status;
@@ -70,7 +69,6 @@ public class FilesystemBagStorageImpl extends StatusProvider implements BagStora
 
     private final ApplicationContext applicationContext;
     private final BagRepository bagRepository;
-    private final ConfigService configService;
     private BagService bagService;
 
     private final DirectoryStream.Filter<Path> myDirFilter = path -> path.toFile().isDirectory();
@@ -79,11 +77,9 @@ public class FilesystemBagStorageImpl extends StatusProvider implements BagStora
     private RecursiveWatcher myWatcher = null;
 
 
-    public FilesystemBagStorageImpl(ApplicationContext applicationContext, BagRepository bagRepository,
-                                    ConfigService configService) {
+    public FilesystemBagStorageImpl(ApplicationContext applicationContext, BagRepository bagRepository) {
         this.applicationContext = applicationContext;
         this.bagRepository = bagRepository;
-        this.configService = configService;
     }
 
     @Override
diff --git a/src/main/webapp/resources/js/views/AboutWindow.js b/src/main/webapp/resources/js/views/AboutWindow.js
index 09366576..ffbc67a1 100644
--- a/src/main/webapp/resources/js/views/AboutWindow.js
+++ b/src/main/webapp/resources/js/views/AboutWindow.js
@@ -35,7 +35,7 @@ Ext.define('BagDatabase.views.AboutWindow', {
     layout: 'fit',
     bodyPadding: 5,
     constrainHeader: true,
-    html: "<h2>Bag Database 3.4.1</h2>" +
+    html: "<h2>Bag Database 3.4.2</h2>" +
         "<p>Documentation: <a href='https://swri-robotics.github.io/bag-database/'>https://swri-robotics.github.io/bag-database/</a></p>" +
         "<p>Source Code: <a href='https://github.com/swri-robotics/bag-database'>https://github.com/swri-robotics/bag-database</a></p>" +
         "<p>Copyright 2015-2020 Southwest Research Institute</p>" +