From 8f188ffa8498424653a3e6086ef99bbe1745cd29 Mon Sep 17 00:00:00 2001 From: Marien Fressinaud Date: Thu, 30 Apr 2020 12:11:51 +0200 Subject: Fix filenames for files exported with CLI (#2932) Filenames were created with the username of the current user. However, when we export the files with the CLI, the current user is "_". This commit makes the username always required in the `exportFile` method so we make sure to always manipulate a real value. Consequently, the filenames can be formatted correctly. Obviously, this has absolutely no impacts since the CLI doesn't consider the HTTP headers. It just makes things a bit more clear. It's a first step to remove the concept of "default user". --- cli/export-opml-for-user.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'cli/export-opml-for-user.php') diff --git a/cli/export-opml-for-user.php b/cli/export-opml-for-user.php index 8e7ed08f7..79bf1f1b3 100755 --- a/cli/export-opml-for-user.php +++ b/cli/export-opml-for-user.php @@ -19,7 +19,7 @@ fwrite(STDERR, 'FreshRSS exporting OPML for user “' . $username . "”…\n"); $importController = new FreshRSS_importExport_Controller(); $ok = false; -$ok = $importController->exportFile(true, false, false, array(), 0, $username); +$ok = $importController->exportFile($username, true, false, false, array(), 0); invalidateHttpCache($username); -- cgit v1.2.3