Apply suggestions from code review

Co-authored-by: Hypolite Petovan <hypolite@mrpetovan.com>

diff --git a/src/Module/Media/Attachment/Browser.php b/src/Module/Media/Attachment/Browser.php
index ed9ae74a3fd8cc2c4be4a6a2353718275a520887..86ea00faf86e87200ee5fc8780591131478f91b3 100644 (file)
--- a/src/Module/Media/Attachment/Browser.php
+++ b/src/Module/Media/Attachment/Browser.php
@@ -66,7 +66,6 @@ class Browser extends BaseModule
 
                $files = Attach::selectToArray(['id', 'filename', 'filetype'], ['uid' => $this->session->getLocalUserId()]);
 
-
                $fileArray = array_map([$this, 'map_files'], $files);
 
                $tpl    = Renderer::getMarkupTemplate('media/browser.tpl');

diff --git a/src/Module/Media/Attachment/Upload.php b/src/Module/Media/Attachment/Upload.php
index b11469f5af22ee52430341ae9316188ded0eab25..0bc95fe79299c2c526a9d5c4365c56014e9671d4 100644 (file)
--- a/src/Module/Media/Attachment/Upload.php
+++ b/src/Module/Media/Attachment/Upload.php
@@ -74,7 +74,6 @@ class Upload extends \Friendica\BaseModule
                }
 
                $owner = User::getOwnerDataById($this->userSession->getLocalUserId());
-
                if (!$owner) {
                        $this->logger->warning('Owner not found.', ['uid' => $this->userSession->getLocalUserId()]);
                        return $this->return(401, $this->t('Invalid request.'));

diff --git a/view/js/module/media/browser.js b/view/js/module/media/browser.js
index 7790e2588b8ac60624c9a63f5cf6d3c72eb121cf..c78a7f13b6b013040534a573f53a4a16bd074f4b 100644 (file)
--- a/view/js/module/media/browser.js
+++ b/view/js/module/media/browser.js
@@ -67,8 +67,6 @@ const Browser = {
                        Browser.id = h.split('-')[1];
                }
 
-               console.log('FileBrowser:', nickname, type, Browser.event, Browser.id);
-
                $('.error a.close').on('click', function (e) {
                        e.preventDefault();
                        $('.error').addClass('hidden');
@@ -90,7 +88,6 @@ const Browser = {
                        if (Browser.type === "attachment") {
                                embed = '[attachment]' + this.dataset.link + '[/attachment]';
                        }
-                       console.log(Browser.event, this.dataset.filename, embed, Browser.id);
                        parent.$('body').trigger(Browser.event, [
                                this.dataset.filename,
                                embed,

diff --git a/view/theme/frio/js/module/media/browser.js b/view/theme/frio/js/module/media/browser.js
index c89f426ebd51b24bd92c6230d309007797763b72..ccd2e1b5bef4cd26e7db3a8668c49035989c207c 100644 (file)
--- a/view/theme/frio/js/module/media/browser.js
+++ b/view/theme/frio/js/module/media/browser.js
@@ -56,7 +56,7 @@
  * IMPORTANT
  *
  *  This is a modified version to work with
- *  the frio theme.and bootstrap modals
+ *  the frio theme and Bootstrap modals
  *
  *  The original file is under:
  *  js/module/media/browser.js
@@ -86,8 +86,6 @@ var Browser = {
                        }
                }
 
-               console.log('FileBrowser: ' + nickname, type, Browser.event, Browser.id);
-
                Browser.postLoad();
 
                $('.error .close').on('click', function (e) {