#82 adds rest endpoint and services methods to handle deletion of a snapshot

Author: robfrank

src/main/java/com/arcadeanalytics/provider/FileSystemDataProvider.java

@@ -9,9 +9,9 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -109,6 +109,22 @@ public void deleteAllSnapshots(Widget widget) {
 
     }
 
+    public boolean deleteSnapshot(Widget widget, String filename) {
+
+        final Long id = widget.getId();
+
+        final Path file = widgets.resolve(id.toString()).resolve(filename);
+        log.info("deleting snapshot :: {}", file);
+        try {
+            return Files.deleteIfExists(file);
+        } catch (IOException e) {
+            log.error("unable to delete snapshot {} due to {} ", filename, e.getMessage());
+            return false;
+        }
+
+    }
+
+
     public List<String> getAllSnapshots(Widget widget) {
         final Long id = widget.getId();
         log.info("getting all snapshots of widget:: {} ", id);

src/main/java/com/arcadeanalytics/repository/FileSystemRepository.java

@@ -9,9 +9,9 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -81,6 +81,19 @@ public byte[] read(String path) {
     }
 
 
+    public boolean delete(String path) {
+        final Path file = resolve(path);
+        log.info("delete :: {} ", file.toAbsolutePath());
+
+        try {
+            return Files.deleteIfExists(file);
+        } catch (IOException e) {
+            log.error("unable to delete :: {} due to {} ", file.toAbsolutePath(), e.getMessage());
+        }
+
+        return false;
+    }
+
     public Path resolve(String path) {
 
         return rootPath.resolve(path);

src/main/java/com/arcadeanalytics/service/WidgetService.java

@@ -9,9 +9,9 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -382,6 +382,15 @@ public boolean saveSnapshot(Long id, String snapshotData) {
 
     }
 
+    @Transactional
+    public boolean deleteSnapshot(Long id, String fileName) {
+
+        return getWidgetIfAllowed(id)
+                .map(widget -> new FileSystemDataProvider(fsRepository)
+                        .deleteSnapshot(widget, SNAPSHOT_PREFIX + fileName))
+                .orElse(false);
+    }
+
     public String layout(final String layoutType, final String json) {
 
         return new LayoutUtils().applyLayout(layoutType, json);

src/main/java/com/arcadeanalytics/web/rest/WidgetResource.java

@@ -9,9 +9,9 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -135,7 +135,7 @@ public ResponseEntity<List<WidgetDTO>> getAllWidgets(@ApiParam Pageable pageable
     /**
      * GET  /widgets : get all the widgets.
      *
-     * @param id {@link com.arcadeanalytics.domain.Dashboard} id
+     * @param id       {@link com.arcadeanalytics.domain.Dashboard} id
      * @param pageable the pagination information
      * @return the ResponseEntity with status 200 (OK) and the list of widgets in body
      */
@@ -242,6 +242,14 @@ public ResponseEntity<String> getLatestWidgetSnapshot(@PathVariable Long id, @Re
         return ResponseUtil.wrapOrNotFound(data);
     }
 
+    @DeleteMapping(value = "/widgets/snapshot/{id}", produces = MediaType.APPLICATION_JSON_UTF8_VALUE)
+    @Timed
+    public ResponseEntity<Boolean> deleteWidgetSnapshot(@PathVariable Long id, @RequestParam(defaultValue = "last") String fileName) {
+        log.debug("REST request to delete Widget snapshot: {} ", fileName);
+
+        return ResponseUtil.wrapOrNotFound(Optional.of(widgetService.deleteSnapshot(id, fileName)));
+    }
+
 
     @GetMapping(value = "/widgets/snapshots/{id}", produces = MediaType.APPLICATION_JSON_UTF8_VALUE)
     @Timed