minor changes

trash.sh

@@ -1,18 +1,21 @@
-#!/bin/bash
+#!/usr/bin/env bash
-# Trash Functions
+# Trash Bin Functionss
+# On Github: https://github.com/kimdre/trash.sh
 
-TRASH_DIR="$HOME"/.trash
+TRASH_DIR="$HOME/.trash"
+FILE_PATHS_FILE="$TRASH_DIR/.filepaths"
 
 [ ! -d "$TRASH_DIR" ] && mkdir "$TRASH_DIR"
-FILE_PATHS_FILE="$TRASH_DIR/.filepaths"
 [ ! -f "$FILE_PATHS_FILE" ] && touch "$FILE_PATHS_FILE"
 
 usage() {
   local func_args="$1"
 
   if [[ $func_args == 0 ]];
-    then echo -e "At least one argument required:\ncommand FILE1  DIR2 ..."
+    then echo -e "At least one file/directory as argument required:\nExamples: FILE.txt /DIR/FILE.txt SUB_DIR/FILE.* /DIR/* DIR/DIR ..."
-    exit
+    return 1
+  else
+    return 0
   fi
 }
 
@@ -21,13 +24,13 @@ trash_list() {
 }
 
 trash_put() {
-  usage "$#"
+  usage "$#" && {
-
+    for FILE in $@; do
-  for FILE in $@; do
+      # Store old filepath for later restore
-    # Store old filepath for later restore
+      echo "$FILE $(readlink -f "$FILE")" >>"$FILE_PATHS_FILE"
-    echo "$FILE $(readlink -f "$FILE")" >>"$FILE_PATHS_FILE"
+    done
-  done
+    mv -v $* "$TRASH_DIR/"
-  mv -v $* "$TRASH_DIR/"
+  }
 }
 
 trash_empty() {
@@ -36,34 +39,34 @@ trash_empty() {
 }
 
 trash_restore() {
-  usage "$#"
+  usage "$#" && {
+    for FILE_TO_RESTORE in "$@"; do
+      # Get old filepath for restore
+      OLD_PATH=$(grep "$FILE_TO_RESTORE" "$FILE_PATHS_FILE" | cut -d ' ' -f2)
 
-  for FILE_TO_RESTORE in "$@"; do
+      # Move file to old filepath
-    # Get old filepath for restore
+      mv -v "${TRASH_DIR}/${FILE_TO_RESTORE}" "$OLD_PATH"
-    OLD_PATH=$(grep "$FILE_TO_RESTORE" "$FILE_PATHS_FILE" | cut -d ' ' -f2)
 
-    # Move file to old filepath
+      # Replace / with \/ from string and remove filepath from file after move was successful
-    mv -v "${TRASH_DIR}/${FILE_TO_RESTORE}" "$OLD_PATH"
+      sed -i "/${FILE_TO_RESTORE////\\/}/d" "$FILE_PATHS_FILE"
-
+    done
-    # Replace / with \/ from string and remove filepath from file after move was successful
+  }
-    sed -i "/${FILE_TO_RESTORE////\\/}/d" "$FILE_PATHS_FILE"
-  done
 }
 
 trash_rm() {
-  usage "$#"
+  usage "$#" && {
-
+    for FILE_TO_REMOVE in "$@"; do
-  for FILE_TO_REMOVE in "$@"; do
+      if ! echo "$FILE_TO_REMOVE" | grep "$HOME/.trash"
-    if ! echo "$FILE_TO_REMOVE" | grep "$HOME/.trash"
+        then FILEPATH_TO_REMOVE="$HOME/.trash/$FILE_TO_REMOVE"
-      then FILEPATH_TO_REMOVE="$HOME/.trash/$FILE_TO_REMOVE"
+         else FILEPATH_TO_REMOVE="$FILE_TO_REMOVE"
-      else FILEPATH_TO_REMOVE="$FILE_TO_REMOVE"
+      fi
-    fi
+      if [ -f "$FILEPATH_TO_REMOVE" ]; then
-    if [ -f "$FILEPATH_TO_REMOVE" ]; then
+        \rm -irf "$FILEPATH_TO_REMOVE" && sed -i "/$FILE_TO_REMOVE/d" "$FILE_PATHS_FILE"
-      \rm -irf "$FILEPATH_TO_REMOVE" && sed -i "/$FILE_TO_REMOVE/d" "$FILE_PATHS_FILE"
+      else
-    else
+        echo "$FILE_TO_REMOVE is not in Trash." && false
-      echo "$FILE_TO_REMOVE is not in Trash." && false
+      fi
-    fi
+   done
-  done
+  }
 }
 
 alias trash='trash_put'