Create shares list report + send it by email + diff

README.md

@@ -4,7 +4,9 @@ This app allows generating reports of shares on the system.
 
 ## Usage
 
-### Command
+### Commands
+
+#### List
 
 ```sh
 ./occ sharing:list [-u|--user [USER]] [-p|--path [PATH]] [-t|--token [TOKEN]] [-f|--filter [FILTER]] [-o|--output FORMAT]
@@ -13,7 +15,7 @@ This app allows generating reports of shares on the system.
 Without options, the command yields the unfiltered list of all shares.\
 With options, the list is narrowed down using the filters set.
 
-### Options
+##### Options
 
 * `-u [USER]` or `--user [USER]`\
   List only shares of the given user.
@@ -27,6 +29,33 @@ With options, the list is narrowed down using the filters set.
 * `-o FORMAT` or `--output FORMAT`\
   Set the output format (json or csv, default is json).
 
+#### Send
+
+```sh
+./occ sharing:send [-u|--user USER] [-p|--path PATH] [-t|--token TOKEN] [-f|--filter FILTER] [-o|--output FORMAT]
+```
+
+Without options, the command yields the unfiltered list of all shares.\
+With options, the list is narrowed down using the filters set.
+
+##### Options
+
+* `-r` or `--recipients`\
+  Recipients users of generated reports.
+* `-x` or `--target-path`\
+  Generated reports will be stored on this path.
+* `-d` or `--diff`\
+  Create a differential report in json format from the last available report.
+* `-u [USER]` or `--user [USER]`\
+  List only shares of the given user.
+* `-p [PATH]` or `--path [PATH]`\
+  List only shares within the given path.
+* `-t [TOKEN]` or `--token [TOKEN]`\
+  List only shares that use a token that (at least partly) matches the argument.
+* `-f [FILTER]` or `--filter [FILTER]`\
+  List only shares where the TYPE matches the argument.\
+  Possible values for the filter argument: {owner, initiator, recipient}
+
 ## Examples
 
 To better illustrate how the app work see the examples below:

appinfo/info.xml

@@ -17,5 +17,6 @@
 
 	<commands>
 		<command>OCA\ShareListing\Command\ListShares</command>
+		<command>OCA\ShareListing\Command\SendShares</command>
 	</commands>
 </info>

composer.json

@@ -4,7 +4,8 @@
   "license": "MIT",
   "require": {
     "nikic/iter": "^2.2",
-    "symfony/serializer": "^5.4"
+    "symfony/serializer": "^5.4",
+    "swaggest/json-diff": "^3.9"
   },
   "require-dev": {
     "phpunit/phpunit": "^9",

lib/Command/AbstractCommand.php

@@ -0,0 +1,78 @@
+<?php
+declare(strict_types=1);
+/**
+ * @copyright Copyright (c) 2022 Solution Libre SAS
+ * @copyright Copyright (c) 2018 Roeland Jago Douma <[email protected]>
+ *
+ * @author Florent Poinsaut <[email protected]>
+ * @author Roeland Jago Douma <[email protected]>
+ * @author John Molakvoæ <[email protected]>
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ */
+
+namespace OCA\ShareListing\Command;
+
+use OCA\ShareListing\Service\SharesList;
+use OC\Core\Command\Base;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+
+abstract class AbstractCommand extends Base {
+	/** @var SharesList */
+	protected $sharesList;
+
+	public function __construct(SharesList $sharesList) {
+		parent::__construct();
+
+		$this->sharesList = $sharesList;
+	}
+
+	public function configure() {
+		$this->addOption(
+				'user',
+				'u',
+				InputOption::VALUE_OPTIONAL,
+				'Will list shares of the given user'
+			)
+			->addOption(
+				'path',
+				'p',
+				InputOption::VALUE_OPTIONAL,
+				'Will only consider the given path'
+			)->addOption(
+				'token',
+				't',
+				InputOption::VALUE_OPTIONAL,
+				'Will only consider the given token'
+			)->addOption(
+				'filter',
+				'f',
+				InputOption::VALUE_OPTIONAL,
+				'Filter shares, possible values: owner, initiator, recipient, token, has-expiration, no-expiration'
+			);
+	}
+
+	protected function getOptions(InputInterface $input): array {
+		$user = $input->getOption('user');
+		$path = $input->getOption('path');
+		$token = $input->getOption('token');
+		$filter = $this->sharesList->filterStringToInt($input->getOption('filter'));
+
+		return [$user, $path, $token, $filter];
+	}
+}

lib/Command/ListShares.php

@@ -27,69 +27,16 @@
 namespace OCA\ShareListing\Command;
 
 use iter;
-use OCA\ShareListing\Service\SharesList;
-use OCP\Files\IRootFolder;
-use OCP\IUser;
-use OCP\IUserManager;
-use OCP\Share\IManager as ShareManager;
-use OC\Core\Command\Base;
-use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class ListShares extends Base {
-
-	/** @var ShareManager */
-	private $shareManager;
-
-	/** @var IUserManager */
-	private $userManager;
-
-	/** @var IRootFolder */
-	private $rootFolder;
-
-	/** @var SharesList */
-	private $sharesList;
-
-	public function __construct(ShareManager $shareManager,
-								IUserManager $userManager,
-								IRootFolder $rootFolder,
-								SharesList $sharesList) {
-		parent::__construct();
-
-		$this->shareManager = $shareManager;
-		$this->userManager = $userManager;
-		$this->rootFolder = $rootFolder;
-		$this->sharesList = $sharesList;
-
-	}
-
+class ListShares extends AbstractCommand {
 	public function configure() {
+		parent::configure();
+
 		$this->setName('sharing:list')
 			->setDescription('List who has access to shares by owner')
-			->addOption(
-				'user',
-				'u',
-				InputOption::VALUE_OPTIONAL,
-				'Will list shares of the given user'
-			)
-			->addOption(
-				'path',
-				'p',
-				InputOption::VALUE_OPTIONAL,
-				'Will only consider the given path'
-			)->addOption(
-				'token',
-				't',
-				InputOption::VALUE_OPTIONAL,
-				'Will only consider the given token'
-			)->addOption(
-				'filter',
-				'f',
-				InputOption::VALUE_OPTIONAL,
-				'Filter shares, possible values: owner, initiator, recipient, token, has-expiration, no-expiration'
-			)
 			->addOption(
 				'output',
 				'o',
@@ -98,12 +45,9 @@ public function configure() {
 				'json'
 			);
 	}
-
+	
 	protected function execute(InputInterface $input, OutputInterface $output): int {
-		$user = $input->getOption('user');
-		$path = $input->getOption('path');
-		$token = $input->getOption('token');
-		$filter = $this->sharesList->filterStringToInt($input->getOption('filter'));
+		[$user, $path, $token, $filter] = $this->getOptions($input);
 		$outputOpt = $input->getOption('output');
 
 		$shares = iter\toArray($this->sharesList->getFormattedShares($user, $filter, $path, $token));