From 29d7b4ef243a838592c80bba6c7c577494131a3d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Mon, 13 Mar 2023 03:32:28 +0100 Subject: [PATCH 01/36] fix path for js and css loading --- src/Content/Conversation.php | 1 + src/Core/ACL.php | 2 + src/Module/Item/Compose.php | 2 +- view/templates/item/compose.tpl | 50 +++++++++++++++++++++- view/theme/frio/templates/comment_item.tpl | 48 ++++++++++++++++++++- view/theme/frio/templates/head.tpl | 3 ++ view/theme/frio/templates/jot.tpl | 48 ++++++++++++++++++++- 7 files changed, 148 insertions(+), 6 deletions(-) diff --git a/src/Content/Conversation.php b/src/Content/Conversation.php index 249190b1a..7cf207627 100644 --- a/src/Content/Conversation.php +++ b/src/Content/Conversation.php @@ -445,6 +445,7 @@ class Conversation $this->page->registerFooterScript(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput.js')); $this->page->registerStylesheet(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput.css')); $this->page->registerStylesheet(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput-typeahead.css')); + $this->page->registerStylesheet(Theme::getPathForFile('vendor/enyo/dropzone/dist/min/dropzone.min.css')); $live_update_div = ''; diff --git a/src/Core/ACL.php b/src/Core/ACL.php index 4ed8d602a..45f8e1128 100644 --- a/src/Core/ACL.php +++ b/src/Core/ACL.php @@ -59,8 +59,10 @@ class ACL $page->registerFooterScript(Theme::getPathForFile('asset/typeahead.js/dist/typeahead.bundle.js')); $page->registerFooterScript(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput.js')); + $page->registerFooterScript(Theme::getPathForFile('../vendor/enyo/dropzone/dist/min/dropzone.min.js')); $page->registerStylesheet(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput.css')); $page->registerStylesheet(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput-typeahead.css')); + $page->registerStylesheet(Theme::getPathForFile('../vendor/enyo/dropzone/dist/min/dropzone.min.css')); $contacts = self::getValidMessageRecipientsForUser(DI::userSession()->getLocalUserId()); diff --git a/src/Module/Item/Compose.php b/src/Module/Item/Compose.php index d44fe2cd7..1bb56a3f7 100644 --- a/src/Module/Item/Compose.php +++ b/src/Module/Item/Compose.php @@ -164,7 +164,7 @@ class Compose extends BaseModule $this->page->registerFooterScript(Theme::getPathForFile('js/ajaxupload.js')); $this->page->registerFooterScript(Theme::getPathForFile('js/linkPreview.js')); $this->page->registerFooterScript(Theme::getPathForFile('js/compose.js')); - + $this->page->registerFooterScript(Theme::getPathForFile('../vendor/enyo/dropzone/dist/min/dropzone.min.js')); $contact = Contact::getById($a->getContactId()); if ($this->pConfig->get(DI::userSession()->getLocalUserId(), 'system', 'set_creation_date')) { diff --git a/view/templates/item/compose.tpl b/view/templates/item/compose.tpl index 50e21b149..1143a5efb 100644 --- a/view/templates/item/compose.tpl +++ b/view/templates/item/compose.tpl @@ -1,10 +1,17 @@ -
+

{{$l10n.compose_title}}

{{if $l10n.always_open_compose}}

{{$l10n.always_open_compose nofilter}}

{{/if}}
-
+ + + {{**}} @@ -71,6 +78,7 @@ +

@@ -94,3 +102,41 @@
+ diff --git a/view/theme/frio/templates/comment_item.tpl b/view/theme/frio/templates/comment_item.tpl index 7bce14ee1..ee09b36f3 100644 --- a/view/theme/frio/templates/comment_item.tpl +++ b/view/theme/frio/templates/comment_item.tpl @@ -1,8 +1,8 @@ {{if $threaded}} -
+
{{else}} -
+
{{/if}}
@@ -61,6 +61,50 @@
+
+ diff --git a/view/theme/frio/templates/head.tpl b/view/theme/frio/templates/head.tpl index 7851dacbd..b506ad278 100644 --- a/view/theme/frio/templates/head.tpl +++ b/view/theme/frio/templates/head.tpl @@ -55,6 +55,8 @@ media="screen" /> + {{foreach $stylesheets as $stylesheetUrl => $media}} @@ -137,6 +139,7 @@ {{/if}} + {{* Include the strings which are needed for some js functions (e.g. translation) They are loaded into the html so that js functions can use them *}} diff --git a/view/theme/frio/templates/jot.tpl b/view/theme/frio/templates/jot.tpl index c38e8531c..285b8e8d7 100644 --- a/view/theme/frio/templates/jot.tpl +++ b/view/theme/frio/templates/jot.tpl @@ -70,7 +70,7 @@
-
+
@@ -152,6 +152,7 @@ +
{{if $content}}{{/if}}
@@ -178,3 +179,48 @@ can load different content into the jot moadl (e.g. the item edit jot) this.style.height = this.contentWindow.document.body.offsetHeight + 'px'; }); + + From 3bb801c2c1fcc0d2c6859c3674247fe9eb697430 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Mon, 13 Mar 2023 03:37:32 +0100 Subject: [PATCH 02/36] fix dropzone class in template --- view/templates/item/compose.tpl | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/view/templates/item/compose.tpl b/view/templates/item/compose.tpl index 1143a5efb..17582df44 100644 --- a/view/templates/item/compose.tpl +++ b/view/templates/item/compose.tpl @@ -1,17 +1,10 @@ -
+

{{$l10n.compose_title}}

{{if $l10n.always_open_compose}}

{{$l10n.always_open_compose nofilter}}

{{/if}}
- - -
+ {{**}} From 00c7e989b2791582fe77ce088f89b884915bba47 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Mon, 13 Mar 2023 03:49:20 +0100 Subject: [PATCH 03/36] add missing css --- view/theme/frio/templates/head.tpl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/view/theme/frio/templates/head.tpl b/view/theme/frio/templates/head.tpl index b506ad278..4f42f0937 100644 --- a/view/theme/frio/templates/head.tpl +++ b/view/theme/frio/templates/head.tpl @@ -55,7 +55,7 @@ media="screen" /> - {{foreach $stylesheets as $stylesheetUrl => $media}} From 5107d4418e564791e95a59c3660aca2bbef8d132 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Mon, 13 Mar 2023 03:49:50 +0100 Subject: [PATCH 04/36] add enyo/dropzone --- composer.json | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/composer.json b/composer.json index e4e6aa6f0..b21c94fde 100644 --- a/composer.json +++ b/composer.json @@ -72,7 +72,8 @@ "npm-asset/textcomplete": "^0.18.2", "npm-asset/typeahead.js": "^0.11.1", "kornrunner/blurhash": "^1.2", - "psr/clock": "^1.0" + "psr/clock": "^1.0", + "enyo/dropzone": "^5.9" }, "repositories": [ { From 6a7f17559e888b7ae88c4ce660915dad764528c1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Mon, 13 Mar 2023 04:06:59 +0100 Subject: [PATCH 05/36] fix composer dropzone --- view/templates/item/compose.tpl | 10 ++++++---- view/theme/frio/templates/comment_item.tpl | 3 +-- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/view/templates/item/compose.tpl b/view/templates/item/compose.tpl index 17582df44..449c3af4c 100644 --- a/view/templates/item/compose.tpl +++ b/view/templates/item/compose.tpl @@ -96,7 +96,8 @@
diff --git a/view/theme/frio/templates/comment_item.tpl b/view/theme/frio/templates/comment_item.tpl index ee09b36f3..90a7a2993 100644 --- a/view/theme/frio/templates/comment_item.tpl +++ b/view/theme/frio/templates/comment_item.tpl @@ -68,7 +68,6 @@ From 359ad6ff5ab4c178591af01e35b5f8bb40380b84 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Mon, 13 Mar 2023 10:37:12 +0100 Subject: [PATCH 06/36] fix a warning and suggestions for pullrequest * fix indentation to tabs * remove unneeded calls for dropzone.min.js and dropzone.min.css * removed debug output --- composer.json | 4 +- composer.lock | 38 ++++++++- src/Content/Conversation.php | 1 - src/Core/ACL.php | 2 - src/Module/Item/Compose.php | 1 - view/templates/item/compose.tpl | 97 +++++++++++----------- view/theme/frio/templates/comment_item.tpl | 79 +++++++++--------- view/theme/frio/templates/head.tpl | 2 +- view/theme/frio/templates/jot.tpl | 81 +++++++++--------- 9 files changed, 164 insertions(+), 141 deletions(-) diff --git a/composer.json b/composer.json index b21c94fde..92dce029f 100644 --- a/composer.json +++ b/composer.json @@ -29,6 +29,7 @@ "asika/simple-console": "^1.0", "bacon/bacon-qr-code": "^2.0.0", "divineomega/password_exposed": "^2.8", + "enyo/dropzone": "^5.9", "ezyang/htmlpurifier": "^4.7", "friendica/json-ld": "^1.0", "geekwright/po": "^2.0", @@ -72,8 +73,7 @@ "npm-asset/textcomplete": "^0.18.2", "npm-asset/typeahead.js": "^0.11.1", "kornrunner/blurhash": "^1.2", - "psr/clock": "^1.0", - "enyo/dropzone": "^5.9" + "psr/clock": "^1.0" }, "repositories": [ { diff --git a/composer.lock b/composer.lock index f9b96b1d9..3e8ec70de 100644 --- a/composer.lock +++ b/composer.lock @@ -4,7 +4,7 @@ "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies", "This file is @generated automatically" ], - "content-hash": "5af9ac9003f4653f3aa1860dd5a4d821", + "content-hash": "c9e005c79c8556215c30a66c470659eb", "packages": [ { "name": "asika/simple-console", @@ -550,6 +550,42 @@ "description": "This PHP package provides a `password_exposed` helper function, that uses the haveibeenpwned.com API to check if a password has been exposed in a data breach.", "time": "2019-01-25T12:00:28+00:00" }, + { + "name": "enyo/dropzone", + "version": "v5.9.3", + "source": { + "type": "git", + "url": "https://github.com/dropzone/dropzone-packagist.git", + "reference": "286b2dc1f1195bd12169e4c9d5f91cfbe46e245f" + }, + "dist": { + "type": "zip", + "url": "https://api.github.com/repos/dropzone/dropzone-packagist/zipball/286b2dc1f1195bd12169e4c9d5f91cfbe46e245f", + "reference": "286b2dc1f1195bd12169e4c9d5f91cfbe46e245f", + "shasum": "" + }, + "type": "library", + "notification-url": "https://packagist.org/downloads/", + "license": [ + "MIT" + ], + "authors": [ + { + "name": "Matias Meno", + "email": "m@tias.me", + "homepage": "http://www.yesmeno.com" + } + ], + "description": "Handles drag and drop of files for you.", + "homepage": "http://www.dropzonejs.com", + "keywords": [ + "drag and drop", + "dragndrop", + "file upload", + "upload" + ], + "time": "2021-09-21T17:03:36+00:00" + }, { "name": "ezyang/htmlpurifier", "version": "v4.14.0", diff --git a/src/Content/Conversation.php b/src/Content/Conversation.php index 7cf207627..249190b1a 100644 --- a/src/Content/Conversation.php +++ b/src/Content/Conversation.php @@ -445,7 +445,6 @@ class Conversation $this->page->registerFooterScript(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput.js')); $this->page->registerStylesheet(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput.css')); $this->page->registerStylesheet(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput-typeahead.css')); - $this->page->registerStylesheet(Theme::getPathForFile('vendor/enyo/dropzone/dist/min/dropzone.min.css')); $live_update_div = ''; diff --git a/src/Core/ACL.php b/src/Core/ACL.php index 45f8e1128..4ed8d602a 100644 --- a/src/Core/ACL.php +++ b/src/Core/ACL.php @@ -59,10 +59,8 @@ class ACL $page->registerFooterScript(Theme::getPathForFile('asset/typeahead.js/dist/typeahead.bundle.js')); $page->registerFooterScript(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput.js')); - $page->registerFooterScript(Theme::getPathForFile('../vendor/enyo/dropzone/dist/min/dropzone.min.js')); $page->registerStylesheet(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput.css')); $page->registerStylesheet(Theme::getPathForFile('js/friendica-tagsinput/friendica-tagsinput-typeahead.css')); - $page->registerStylesheet(Theme::getPathForFile('../vendor/enyo/dropzone/dist/min/dropzone.min.css')); $contacts = self::getValidMessageRecipientsForUser(DI::userSession()->getLocalUserId()); diff --git a/src/Module/Item/Compose.php b/src/Module/Item/Compose.php index 1bb56a3f7..f53158b1f 100644 --- a/src/Module/Item/Compose.php +++ b/src/Module/Item/Compose.php @@ -164,7 +164,6 @@ class Compose extends BaseModule $this->page->registerFooterScript(Theme::getPathForFile('js/ajaxupload.js')); $this->page->registerFooterScript(Theme::getPathForFile('js/linkPreview.js')); $this->page->registerFooterScript(Theme::getPathForFile('js/compose.js')); - $this->page->registerFooterScript(Theme::getPathForFile('../vendor/enyo/dropzone/dist/min/dropzone.min.js')); $contact = Contact::getById($a->getContactId()); if ($this->pConfig->get(DI::userSession()->getLocalUserId(), 'system', 'set_creation_date')) { diff --git a/view/templates/item/compose.tpl b/view/templates/item/compose.tpl index 449c3af4c..21d89d5c4 100644 --- a/view/templates/item/compose.tpl +++ b/view/templates/item/compose.tpl @@ -1,11 +1,11 @@

{{$l10n.compose_title}}

- {{if $l10n.always_open_compose}} + {{if $l10n.always_open_compose}}

{{$l10n.always_open_compose nofilter}}

{{/if}}
- {{**}} + {{**}} @@ -13,11 +13,11 @@
- {{if $l10n.placeholdercategory}} + {{if $l10n.placeholdercategory}}
- {{/if}} + {{/if}}

@@ -55,12 +55,12 @@ @@ -71,7 +71,7 @@ -

+

@@ -96,42 +96,41 @@
diff --git a/view/theme/frio/templates/comment_item.tpl b/view/theme/frio/templates/comment_item.tpl index 90a7a2993..b57bf8a07 100644 --- a/view/theme/frio/templates/comment_item.tpl +++ b/view/theme/frio/templates/comment_item.tpl @@ -61,49 +61,46 @@
-
+
diff --git a/view/theme/frio/templates/head.tpl b/view/theme/frio/templates/head.tpl index 4f42f0937..b506ad278 100644 --- a/view/theme/frio/templates/head.tpl +++ b/view/theme/frio/templates/head.tpl @@ -55,7 +55,7 @@ media="screen" /> - {{foreach $stylesheets as $stylesheetUrl => $media}} diff --git a/view/theme/frio/templates/jot.tpl b/view/theme/frio/templates/jot.tpl index 285b8e8d7..015d967be 100644 --- a/view/theme/frio/templates/jot.tpl +++ b/view/theme/frio/templates/jot.tpl @@ -152,7 +152,7 @@ -
+
{{if $content}}{{/if}}
@@ -180,47 +180,42 @@ can load different content into the jot moadl (e.g. the item edit jot) }); - From be2afac60d1108408bfae674b1594aa983d6c254 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Mon, 13 Mar 2023 13:47:09 +0100 Subject: [PATCH 07/36] take maxFilesize from systemsettings * does not work everywhere... --- mod/photos.php | 18 +++++++++++++++--- src/Content/Conversation.php | 5 +++++ src/Module/Item/Compose.php | 3 +++ src/Module/Post/Edit.php | 5 +++++ src/Object/Post.php | 4 +++- view/templates/item/compose.tpl | 4 +++- view/theme/frio/templates/comment_item.tpl | 4 +++- view/theme/frio/templates/jot.tpl | 5 +++-- 8 files changed, 40 insertions(+), 8 deletions(-) diff --git a/mod/photos.php b/mod/photos.php index 050a254af..a8b63d12d 100644 --- a/mod/photos.php +++ b/mod/photos.php @@ -1139,7 +1139,11 @@ function photos_content(App $a) '$preview' => DI::l10n()->t('Preview'), '$loading' => DI::l10n()->t('Loading...'), '$qcomment' => $qcomment, - '$rand_num' => Crypto::randomDigits(12) + '$rand_num' => Crypto::randomDigits(12), + // Dropzone + //'$max_imagesize' => DI::config()->get('system', 'maximagesize'), + // don't know, if DI::config.. does not work here, so it is set to a manual value + '$max_imagesize' => 100000, ]); } } @@ -1194,7 +1198,11 @@ function photos_content(App $a) '$submit' => DI::l10n()->t('Submit'), '$preview' => DI::l10n()->t('Preview'), '$qcomment' => $qcomment, - '$rand_num' => Crypto::randomDigits(12) + '$rand_num' => Crypto::randomDigits(12), + // Dropzone + //'$max_imagesize' => DI::config()->get('system', 'maximagesize'), + // don't know, if DI::config.. does not work here, so it is set to a manual value + '$max_imagesize' => 200000 ]); } @@ -1268,7 +1276,11 @@ function photos_content(App $a) '$submit' => DI::l10n()->t('Submit'), '$preview' => DI::l10n()->t('Preview'), '$qcomment' => $qcomment, - '$rand_num' => Crypto::randomDigits(12) + '$rand_num' => Crypto::randomDigits(12), + // Dropzone + //'$max_imagesize' => DI::config()->get('system', 'maximagesize'), + // don't know, if DI::config.. does not work here, so it is set to a manual value + '$max_imagesize' => 300000 ]); } } diff --git a/src/Content/Conversation.php b/src/Content/Conversation.php index 249190b1a..586b0bc54 100644 --- a/src/Content/Conversation.php +++ b/src/Content/Conversation.php @@ -408,6 +408,11 @@ class Conversation '$compose_link_title' => $this->l10n->t('Open Compose page'), '$always_open_compose' => $this->pConfig->get($this->session->getLocalUserId(), 'frio', 'always_open_compose', false), + // Dropzone + //'$max_imagesize' => DI::config()->get('system', 'maximagesize'), + // DI::config.. does not work here, so it is set to a manual value + '$max_imagesize' => 600000, + ]); diff --git a/src/Module/Item/Compose.php b/src/Module/Item/Compose.php index f53158b1f..30e2aa9d6 100644 --- a/src/Module/Item/Compose.php +++ b/src/Module/Item/Compose.php @@ -240,6 +240,9 @@ class Compose extends BaseModule 'deny_cid' => $contact_deny_list, 'deny_gid' => $group_deny_list, ]), + + // Dropzone + '$max_imagesize' => DI::config()->get('system', 'maximagesize') ]); } } diff --git a/src/Module/Post/Edit.php b/src/Module/Post/Edit.php index 5fb339e17..87a0c1b9c 100644 --- a/src/Module/Post/Edit.php +++ b/src/Module/Post/Edit.php @@ -182,6 +182,11 @@ class Edit extends BaseModule '$shortpermset' => $this->t('Permissions'), '$compose_link_title' => $this->t('Open Compose page'), + // Dropzone + //'$max_imagesize' => DI::config()->get('system', 'maximagesize'), + // DI::config.. does not work here, so it is set to a manual value + '$max_imagesize' => 500000, + ]); return $output; diff --git a/src/Object/Post.php b/src/Object/Post.php index 17c0c1f68..59617573e 100644 --- a/src/Object/Post.php +++ b/src/Object/Post.php @@ -1068,7 +1068,9 @@ class Post '$prompttext' => DI::l10n()->t('Please enter a image/video/audio/webpage URL:'), '$preview' => DI::l10n()->t('Preview'), '$indent' => $indent, - '$rand_num' => Crypto::randomDigits(12) + '$rand_num' => Crypto::randomDigits(12), + // Dropzone + '$max_imagesize' => DI::config()->get('system', 'maximagesize'), ]); } diff --git a/view/templates/item/compose.tpl b/view/templates/item/compose.tpl index 21d89d5c4..16ba7b477 100644 --- a/view/templates/item/compose.tpl +++ b/view/templates/item/compose.tpl @@ -97,9 +97,11 @@
+ {{/if}} - + + {{* Include the strings which are needed for some js functions (e.g. translation) They are loaded into the html so that js functions can use them *}} diff --git a/view/theme/frio/templates/jot.tpl b/view/theme/frio/templates/jot.tpl index 2d20fef44..707691094 100644 --- a/view/theme/frio/templates/jot.tpl +++ b/view/theme/frio/templates/jot.tpl @@ -180,11 +180,12 @@ can load different content into the jot modal (e.g. the item edit jot) - {{* Include the strings which are needed for some js functions (e.g. translation) They are loaded into the html so that js functions can use them *}} diff --git a/view/theme/frio/templates/jot.tpl b/view/theme/frio/templates/jot.tpl index 1ffe7dd3c..006095302 100644 --- a/view/theme/frio/templates/jot.tpl +++ b/view/theme/frio/templates/jot.tpl @@ -182,29 +182,27 @@ can load different content into the jot modal (e.g. the item edit jot) diff --git a/view/theme/frio/templates/head.tpl b/view/theme/frio/templates/head.tpl index 478f19056..13104aa6a 100644 --- a/view/theme/frio/templates/head.tpl +++ b/view/theme/frio/templates/head.tpl @@ -140,6 +140,8 @@ {{/if}} + + {{* Include the strings which are needed for some js functions (e.g. translation) They are loaded into the html so that js functions can use them *}} From d39781023eff8550b09021e586ea61b4daee278f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Wed, 15 Mar 2023 04:47:20 +0100 Subject: [PATCH 20/36] create factory --- view/js/dropzone-factory.js | 63 +++++++++++++++++++++++++++++++++++++ 1 file changed, 63 insertions(+) create mode 100644 view/js/dropzone-factory.js diff --git a/view/js/dropzone-factory.js b/view/js/dropzone-factory.js new file mode 100644 index 000000000..fe6c4448b --- /dev/null +++ b/view/js/dropzone-factory.js @@ -0,0 +1,63 @@ +var DzFactory = function () { + this.createDropzone = function(element, target, maxImagesize) { + return new Dropzone( element, { + paramName: 'userfile', // The name that will be used to transfer the file + maxFilesize: maxImagesize, // MB + url: '/media/photo/upload?response=url&album=', + accept: function(file, done) { + done(); + }, + init: function() { + this.on('success', function(file, serverResponse) { + var _target = $(target) + var resp = $(serverResponse).find('div#content').text() + if (_target.setRangeText) { + //if setRangeText function is supported by current browser + _target.setRangeText(' ' + $.trim(resp) + ' ') + } else { + _target.focus() + document.execCommand('insertText', false /*no UI*/, ' ' + $.trim(resp) + ' '); + } + }); + this.on('complete', function(file) { + var dz = this; + // Remove just uploaded file from dropzone, makes interface more clear. + // Image can be seen in posting-preview + // We need preview to get optical feedback about upload-progress. + // you see success, when the bb-code link for image is inserted + setTimeout(function(){ + dz.removeFile(file); + },5000); + }); + }, + paste: function(event){ + const items = (event.clipboardData || event.originalEvent.clipboardData).items; + items.forEach((item) => { + if (item.kind === 'file') { + // adds the file to your dropzone instance + dz.addFile(item.getAsFile()) + } + }) + }, + }); + }; + + this.copyPaste = function(event, dz) { + const items = (event.clipboardData || event.originalEvent.clipboardData).items; + items.forEach((item) => { + if (item.kind === 'file') { + // adds the file to your dropzone instance + dz.addFile(item.getAsFile()) + } + }) + }; + + this.setupDropzone = function(element, target, maxImagesize) { + var dropzone = this.createDropzone(element, target, maxImagesize) + $(element).on('paste', function(event) { + + dzFactory.copyPaste(event, dropzone); + }) + }; +} + From 1a1a1246b69ca79a0c7a92cbd56288afbe7f1b42 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Wed, 15 Mar 2023 05:52:41 +0100 Subject: [PATCH 21/36] attach dropzone only when opening comment --- view/theme/frio/templates/comment_item.tpl | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/view/theme/frio/templates/comment_item.tpl b/view/theme/frio/templates/comment_item.tpl index c4fcb9f62..5926e787f 100644 --- a/view/theme/frio/templates/comment_item.tpl +++ b/view/theme/frio/templates/comment_item.tpl @@ -67,5 +67,8 @@ From 512e5900e6d2612b830f11b85e41931fe49b6248 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Wed, 15 Mar 2023 14:33:42 +0100 Subject: [PATCH 22/36] use round instead of floor to make imagesize <1M possible --- mod/photos.php | 6 +++--- src/Content/Conversation.php | 2 +- src/Module/Item/Compose.php | 2 +- src/Module/Post/Edit.php | 2 +- src/Object/Post.php | 2 +- 5 files changed, 7 insertions(+), 7 deletions(-) diff --git a/mod/photos.php b/mod/photos.php index 06c0d1946..f9d4637c3 100644 --- a/mod/photos.php +++ b/mod/photos.php @@ -1141,7 +1141,7 @@ function photos_content(App $a) '$qcomment' => $qcomment, '$rand_num' => Crypto::randomDigits(12), // Dropzone - '$max_imagesize' => floor(\Friendica\Util\Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000), + '$max_imagesize' => round(\Friendica\Util\Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000,1), ]); } } @@ -1198,7 +1198,7 @@ function photos_content(App $a) '$qcomment' => $qcomment, '$rand_num' => Crypto::randomDigits(12), // Dropzone - '$max_imagesize' => floor(Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000), + '$max_imagesize' => round(Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000,1), ]); } @@ -1274,7 +1274,7 @@ function photos_content(App $a) '$qcomment' => $qcomment, '$rand_num' => Crypto::randomDigits(12), // Dropzone - '$max_imagesize' => floor(Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000), + '$max_imagesize' => round(Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000,1), ]); } } diff --git a/src/Content/Conversation.php b/src/Content/Conversation.php index b0fd60ae3..e660c9ff4 100644 --- a/src/Content/Conversation.php +++ b/src/Content/Conversation.php @@ -411,7 +411,7 @@ class Conversation '$always_open_compose' => $this->pConfig->get($this->session->getLocalUserId(), 'frio', 'always_open_compose', false), // Dropzone - '$max_imagesize' => floor(\Friendica\Util\Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000), + '$max_imagesize' => round(\Friendica\Util\Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000,1), ]); diff --git a/src/Module/Item/Compose.php b/src/Module/Item/Compose.php index 2f60eebc3..1c1687976 100644 --- a/src/Module/Item/Compose.php +++ b/src/Module/Item/Compose.php @@ -242,7 +242,7 @@ class Compose extends BaseModule ]), // Dropzone - '$max_imagesize' => floor(\Friendica\Util\Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000), + '$max_imagesize' => round(\Friendica\Util\Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000,1), ]); } } diff --git a/src/Module/Post/Edit.php b/src/Module/Post/Edit.php index e28d54c9b..403908cc0 100644 --- a/src/Module/Post/Edit.php +++ b/src/Module/Post/Edit.php @@ -186,7 +186,7 @@ class Edit extends BaseModule '$compose_link_title' => $this->t('Open Compose page'), // Dropzone - '$max_imagesize' => floor(Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000), + '$max_imagesize' => round(Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000,1), ]); return $output; } diff --git a/src/Object/Post.php b/src/Object/Post.php index 5fa9dfcf6..c6ef5e3ba 100644 --- a/src/Object/Post.php +++ b/src/Object/Post.php @@ -1070,7 +1070,7 @@ class Post '$indent' => $indent, '$rand_num' => Crypto::randomDigits(12), // Dropzone - '$max_imagesize' => floor(\Friendica\Util\Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000), + '$max_imagesize' => round(\Friendica\Util\Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000,1), ]); } From f876a053df0c87fc33bb33e9ab6e39e044b9405b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Wed, 15 Mar 2023 15:19:55 +0100 Subject: [PATCH 23/36] add dropzone also onclick on comment-fake-form --- view/theme/frio/templates/comment_item.tpl | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/view/theme/frio/templates/comment_item.tpl b/view/theme/frio/templates/comment_item.tpl index 5926e787f..c0f78eda9 100644 --- a/view/theme/frio/templates/comment_item.tpl +++ b/view/theme/frio/templates/comment_item.tpl @@ -67,8 +67,14 @@ From e62265b23424997c994ee75e77b2deabaf528394 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Wed, 15 Mar 2023 15:32:18 +0100 Subject: [PATCH 24/36] make errormessages visible --- view/templates/item/compose.tpl | 2 +- view/theme/frio/templates/comment_item.tpl | 2 +- view/theme/frio/templates/jot.tpl | 8 +++----- 3 files changed, 5 insertions(+), 7 deletions(-) diff --git a/view/templates/item/compose.tpl b/view/templates/item/compose.tpl index 88a19ac97..29319a51e 100644 --- a/view/templates/item/compose.tpl +++ b/view/templates/item/compose.tpl @@ -46,7 +46,7 @@

-
+

diff --git a/view/theme/frio/templates/comment_item.tpl b/view/theme/frio/templates/comment_item.tpl index c0f78eda9..c01e0c80b 100644 --- a/view/theme/frio/templates/comment_item.tpl +++ b/view/theme/frio/templates/comment_item.tpl @@ -38,7 +38,7 @@

-
+

diff --git a/view/theme/frio/templates/jot.tpl b/view/theme/frio/templates/jot.tpl index 006095302..76359b9da 100644 --- a/view/theme/frio/templates/jot.tpl +++ b/view/theme/frio/templates/jot.tpl @@ -99,11 +99,9 @@ {{/if}} {{* The jot text field in which the post text is inserted *}} - -
- -
- +
+ +
diff --git a/view/theme/frio/templates/comment_item.tpl b/view/theme/frio/templates/comment_item.tpl index c01e0c80b..b61c1d6f8 100644 --- a/view/theme/frio/templates/comment_item.tpl +++ b/view/theme/frio/templates/comment_item.tpl @@ -66,7 +66,6 @@
- + {{* Include the strings which are needed for some js functions (e.g. translation) They are loaded into the html so that js functions can use them *}} diff --git a/view/theme/frio/templates/jot.tpl b/view/theme/frio/templates/jot.tpl index 76359b9da..1b77ded42 100644 --- a/view/theme/frio/templates/jot.tpl +++ b/view/theme/frio/templates/jot.tpl @@ -180,46 +180,5 @@ can load different content into the jot modal (e.g. the item edit jot) From 16f065e2600c92354b6e1995385889873a840c24 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Wed, 15 Mar 2023 21:05:16 +0100 Subject: [PATCH 26/36] changes in case of codereview --- view/js/dropzone-factory.js | 27 +++++++------ view/templates/item/compose.tpl | 2 +- view/theme/frio/js/modal.js | 44 +--------------------- view/theme/frio/templates/comment_item.tpl | 12 +++--- view/theme/frio/templates/jot.tpl | 2 +- 5 files changed, 23 insertions(+), 64 deletions(-) diff --git a/view/js/dropzone-factory.js b/view/js/dropzone-factory.js index fe6c4448b..5aaab5036 100644 --- a/view/js/dropzone-factory.js +++ b/view/js/dropzone-factory.js @@ -1,6 +1,6 @@ var DzFactory = function () { - this.createDropzone = function(element, target, maxImagesize) { - return new Dropzone( element, { + this.createDropzone = function(dropSelector, textareaSelector, maxImagesize) { + return new Dropzone( dropSelector, { paramName: 'userfile', // The name that will be used to transfer the file maxFilesize: maxImagesize, // MB url: '/media/photo/upload?response=url&album=', @@ -9,14 +9,14 @@ var DzFactory = function () { }, init: function() { this.on('success', function(file, serverResponse) { - var _target = $(target) - var resp = $(serverResponse).find('div#content').text() - if (_target.setRangeText) { + const targetTextarea = document.getElementById(textareaSelector); + const bbcodeString = $(serverResponse).find('div#content').text(); + if (targetTextarea.setRangeText) { //if setRangeText function is supported by current browser - _target.setRangeText(' ' + $.trim(resp) + ' ') + targetTextarea.setRangeText(' ' + $.trim(bbcodeString) + ' '); } else { - _target.focus() - document.execCommand('insertText', false /*no UI*/, ' ' + $.trim(resp) + ' '); + targetTextarea.focus(); + document.execCommand('insertText', false /*no UI*/, '\n' + $.trim(bbcodeString) + '\n'); } }); this.on('complete', function(file) { @@ -35,7 +35,7 @@ var DzFactory = function () { items.forEach((item) => { if (item.kind === 'file') { // adds the file to your dropzone instance - dz.addFile(item.getAsFile()) + dz.addFile(item.getAsFile()); } }) }, @@ -47,15 +47,14 @@ var DzFactory = function () { items.forEach((item) => { if (item.kind === 'file') { // adds the file to your dropzone instance - dz.addFile(item.getAsFile()) + dz.addFile(item.getAsFile()); } }) }; - this.setupDropzone = function(element, target, maxImagesize) { - var dropzone = this.createDropzone(element, target, maxImagesize) - $(element).on('paste', function(event) { - + this.setupDropzone = function(dropSelector, textareaSelector, maxImagesize) { + var dropzone = this.createDropzone(dropSelector, textareaSelector, maxImagesize); + $(dropSelector).on('paste', function(event) { dzFactory.copyPaste(event, dropzone); }) }; diff --git a/view/templates/item/compose.tpl b/view/templates/item/compose.tpl index 07db40d9c..499dfaffb 100644 --- a/view/templates/item/compose.tpl +++ b/view/templates/item/compose.tpl @@ -96,5 +96,5 @@
diff --git a/view/theme/frio/js/modal.js b/view/theme/frio/js/modal.js index 705f472c4..85c258c92 100644 --- a/view/theme/frio/js/modal.js +++ b/view/theme/frio/js/modal.js @@ -293,48 +293,8 @@ function editpost(url) { // To make dropzone fileupload work on editing a comment, we need to // attach a new dropzone to modal - dropzoneJotEdit = new Dropzone( '#jot-text-wrap', { - paramName: 'userfile', // The name that will be used to transfer the file - maxFilesize: '{{$max_imagesize}}', // MB - url: '/media/photo/upload?response=url&album=', - accept: function(file, done) { - done(); - }, - init: function() { - this.on('success', function(file, serverResponse) { - var target = $('#profile-jot-text') - var resp = $(serverResponse).find('div#content').text() - if (target.setRangeText) { - //if setRangeText function is supported by current browser - target.setRangeText(' ' + $.trim(resp) + ' ') - } else { - target.focus() - document.execCommand('insertText', false /*no UI*/, ' ' + $.trim(resp) + ' '); - } - }); - this.on('complete', function(file) { - // Remove just uploaded file from dropzone, makes interface more clear. - // Image can be seen in posting-preview - // We need preview to get optical feedback about upload-progress. - // you see success, when the bb-code link for image is inserted - setTimeout(function(){ - dropzoneJotEdit.removeFile(file); - },5000); - }); - }, - }); - - // Enables Copy&Paste for this dropzone - $('#jot-text-wrap').on('paste', function(event){ - const items = (event.clipboardData || event.originalEvent.clipboardData).items; - items.forEach((item) => { - if (item.kind === 'file') { - // adds the file to your dropzone instance - dropzoneJotEdit.addFile(item.getAsFile()) - } - }) - }) - + console.log("modal.js max_imagesize",'{{$max_imagesize}}'); + dzFactory.setupDropzone('#jot-text-wrap', 'profile-jot-text', '{{$max_imagesize}}'); modal.show(); $("#jot-popup").show(); diff --git a/view/theme/frio/templates/comment_item.tpl b/view/theme/frio/templates/comment_item.tpl index b61c1d6f8..3d9860a10 100644 --- a/view/theme/frio/templates/comment_item.tpl +++ b/view/theme/frio/templates/comment_item.tpl @@ -67,13 +67,13 @@ diff --git a/view/theme/frio/templates/jot.tpl b/view/theme/frio/templates/jot.tpl index 1b77ded42..325c8d5b3 100644 --- a/view/theme/frio/templates/jot.tpl +++ b/view/theme/frio/templates/jot.tpl @@ -180,5 +180,5 @@ can load different content into the jot modal (e.g. the item edit jot) From 920dcca263edb51088381b7077ff336222e10bd9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Wed, 15 Mar 2023 21:48:17 +0100 Subject: [PATCH 27/36] render max_imagesize in header.tpl so it is reachable general from the whole website, and the dropzone-factory can use it also to create dropzones from modal.js --- mod/photos.php | 6 ------ src/App/Page.php | 14 ++++++++----- src/Content/Conversation.php | 3 --- src/Module/Item/Compose.php | 3 --- src/Module/Post/Edit.php | 2 -- src/Object/Post.php | 2 -- view/js/dropzone-factory.js | 8 ++++---- view/templates/item/compose.tpl | 2 +- view/theme/frio/js/dropzone-frio.js | 24 ---------------------- view/theme/frio/js/modal.js | 3 +-- view/theme/frio/templates/comment_item.tpl | 4 ++-- view/theme/frio/templates/head.tpl | 2 +- view/theme/frio/templates/jot.tpl | 2 +- 13 files changed, 19 insertions(+), 56 deletions(-) delete mode 100644 view/theme/frio/js/dropzone-frio.js diff --git a/mod/photos.php b/mod/photos.php index 5b225489e..a08b569ce 100644 --- a/mod/photos.php +++ b/mod/photos.php @@ -1139,8 +1139,6 @@ function photos_content(App $a) '$loading' => DI::l10n()->t('Loading...'), '$qcomment' => $qcomment, '$rand_num' => Crypto::randomDigits(12), - // Dropzone - '$max_imagesize' => round(\Friendica\Util\Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000, 1), ]); } } @@ -1196,8 +1194,6 @@ function photos_content(App $a) '$preview' => DI::l10n()->t('Preview'), '$qcomment' => $qcomment, '$rand_num' => Crypto::randomDigits(12), - // Dropzone - '$max_imagesize' => round(Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000, 1), ]); } @@ -1272,8 +1268,6 @@ function photos_content(App $a) '$preview' => DI::l10n()->t('Preview'), '$qcomment' => $qcomment, '$rand_num' => Crypto::randomDigits(12), - // Dropzone - '$max_imagesize' => round(Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000, 1), ]); } } diff --git a/src/App/Page.php b/src/App/Page.php index 965c04915..45ec6be0f 100644 --- a/src/App/Page.php +++ b/src/App/Page.php @@ -253,11 +253,15 @@ class Page implements ArrayAccess '$touch_icon' => $touch_icon, '$block_public' => intval($config->get('system', 'block_public')), '$stylesheets' => $this->stylesheets, - '$likeError' => $l10n->t('Like not successfull'), - '$dislikeError' => $l10n->t('Dislike not successfull'), - '$announceError' => $l10n->t('Sharing not successfull'), - '$srvError' => $l10n->t('Backend error'), - '$netError' => $l10n->t('Network error'), + '$likeError' => $l10n->t('Like not successfull'), + '$dislikeError' => $l10n->t('Dislike not successfull'), + '$announceError' => $l10n->t('Sharing not successfull'), + '$srvError' => $l10n->t('Backend error'), + '$netError' => $l10n->t('Network error'), + // Dropzone + '$max_imagesize' => round(\Friendica\Util\Strings::getBytesFromShorthand($config->get('system', 'maximagesize')) / 1000000, 1), + + ]) . $this->page['htmlhead']; } diff --git a/src/Content/Conversation.php b/src/Content/Conversation.php index 1d7430e19..f23333911 100644 --- a/src/Content/Conversation.php +++ b/src/Content/Conversation.php @@ -409,9 +409,6 @@ class Conversation '$compose_link_title' => $this->l10n->t('Open Compose page'), '$always_open_compose' => $this->pConfig->get($this->session->getLocalUserId(), 'frio', 'always_open_compose', false), - // Dropzone - '$max_imagesize' => round(\Friendica\Util\Strings::getBytesFromShorthand($this->config->get('system', 'maximagesize')) / 1000000, 1), - ]); diff --git a/src/Module/Item/Compose.php b/src/Module/Item/Compose.php index 19726eb71..f53158b1f 100644 --- a/src/Module/Item/Compose.php +++ b/src/Module/Item/Compose.php @@ -240,9 +240,6 @@ class Compose extends BaseModule 'deny_cid' => $contact_deny_list, 'deny_gid' => $group_deny_list, ]), - - // Dropzone - '$max_imagesize' => round(\Friendica\Util\Strings::getBytesFromShorthand($this->config->get('system', 'maximagesize')) / 1000000, 1), ]); } } diff --git a/src/Module/Post/Edit.php b/src/Module/Post/Edit.php index 403908cc0..5ddc250d8 100644 --- a/src/Module/Post/Edit.php +++ b/src/Module/Post/Edit.php @@ -185,8 +185,6 @@ class Edit extends BaseModule '$compose_link_title' => $this->t('Open Compose page'), - // Dropzone - '$max_imagesize' => round(Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000,1), ]); return $output; } diff --git a/src/Object/Post.php b/src/Object/Post.php index c6ef5e3ba..3f4d061a1 100644 --- a/src/Object/Post.php +++ b/src/Object/Post.php @@ -1069,8 +1069,6 @@ class Post '$preview' => DI::l10n()->t('Preview'), '$indent' => $indent, '$rand_num' => Crypto::randomDigits(12), - // Dropzone - '$max_imagesize' => round(\Friendica\Util\Strings::getBytesFromShorthand(DI::config()->get('system', 'maximagesize')) / 1000000,1), ]); } diff --git a/view/js/dropzone-factory.js b/view/js/dropzone-factory.js index 5aaab5036..bbf92625d 100644 --- a/view/js/dropzone-factory.js +++ b/view/js/dropzone-factory.js @@ -1,8 +1,8 @@ var DzFactory = function () { - this.createDropzone = function(dropSelector, textareaSelector, maxImagesize) { + this.createDropzone = function(dropSelector, textareaSelector) { return new Dropzone( dropSelector, { paramName: 'userfile', // The name that will be used to transfer the file - maxFilesize: maxImagesize, // MB + maxFilesize: max_imagesize, // MB url: '/media/photo/upload?response=url&album=', accept: function(file, done) { done(); @@ -52,8 +52,8 @@ var DzFactory = function () { }) }; - this.setupDropzone = function(dropSelector, textareaSelector, maxImagesize) { - var dropzone = this.createDropzone(dropSelector, textareaSelector, maxImagesize); + this.setupDropzone = function(dropSelector, textareaSelector) { + var dropzone = this.createDropzone(dropSelector, textareaSelector); $(dropSelector).on('paste', function(event) { dzFactory.copyPaste(event, dropzone); }) diff --git a/view/templates/item/compose.tpl b/view/templates/item/compose.tpl index 499dfaffb..09e42c90e 100644 --- a/view/templates/item/compose.tpl +++ b/view/templates/item/compose.tpl @@ -96,5 +96,5 @@
diff --git a/view/theme/frio/js/dropzone-frio.js b/view/theme/frio/js/dropzone-frio.js deleted file mode 100644 index c165633c4..000000000 --- a/view/theme/frio/js/dropzone-frio.js +++ /dev/null @@ -1,24 +0,0 @@ -function isInteger(value) { - return /^\d+$/.test(value); -} - -function getMBytes(value) { - var res; - if (isInteger(value)) { - res = value; - } else { - eh = value.slice(-1); - am = Number(value.slice(0, -1)); - switch (eh) { - case 'k': - res = am * 1024; - break; - case 'm': - res = am * 1024 * 1024; - break; - case 'g': - res = am * 1024 * 1024 * 1024; - } - } - return res / 1000000; -} diff --git a/view/theme/frio/js/modal.js b/view/theme/frio/js/modal.js index 85c258c92..905e78743 100644 --- a/view/theme/frio/js/modal.js +++ b/view/theme/frio/js/modal.js @@ -293,8 +293,7 @@ function editpost(url) { // To make dropzone fileupload work on editing a comment, we need to // attach a new dropzone to modal - console.log("modal.js max_imagesize",'{{$max_imagesize}}'); - dzFactory.setupDropzone('#jot-text-wrap', 'profile-jot-text', '{{$max_imagesize}}'); + dzFactory.setupDropzone('#jot-text-wrap', 'profile-jot-text'); modal.show(); $("#jot-popup").show(); diff --git a/view/theme/frio/templates/comment_item.tpl b/view/theme/frio/templates/comment_item.tpl index 3d9860a10..81664cf0f 100644 --- a/view/theme/frio/templates/comment_item.tpl +++ b/view/theme/frio/templates/comment_item.tpl @@ -67,12 +67,12 @@ - + {{* Include the strings which are needed for some js functions (e.g. translation) They are loaded into the html so that js functions can use them *}} diff --git a/view/theme/frio/templates/jot.tpl b/view/theme/frio/templates/jot.tpl index 325c8d5b3..5c18afa86 100644 --- a/view/theme/frio/templates/jot.tpl +++ b/view/theme/frio/templates/jot.tpl @@ -180,5 +180,5 @@ can load different content into the jot modal (e.g. the item edit jot) From c01f7f98ae88fd86331070ca24b6d640bbe90bca Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Thu, 16 Mar 2023 12:03:49 +0100 Subject: [PATCH 28/36] reinsert Friendica\DI (accidentially removed before) --- mod/photos.php | 1 + 1 file changed, 1 insertion(+) diff --git a/mod/photos.php b/mod/photos.php index a08b569ce..18db9c15d 100644 --- a/mod/photos.php +++ b/mod/photos.php @@ -32,6 +32,7 @@ use Friendica\Core\Renderer; use Friendica\Core\System; use Friendica\Database\DBA; use Friendica\Database\DBStructure; +use Friendica\DI; use Friendica\Model\Contact; use Friendica\Model\Item; use Friendica\Model\Photo; From fba33d3afc5cff66094f162cd5c29e59a034126f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Thu, 16 Mar 2023 12:04:15 +0100 Subject: [PATCH 29/36] add options allowedFiles and more --- view/js/dropzone-factory.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/view/js/dropzone-factory.js b/view/js/dropzone-factory.js index bbf92625d..a3a3e6c36 100644 --- a/view/js/dropzone-factory.js +++ b/view/js/dropzone-factory.js @@ -4,6 +4,9 @@ var DzFactory = function () { paramName: 'userfile', // The name that will be used to transfer the file maxFilesize: max_imagesize, // MB url: '/media/photo/upload?response=url&album=', + addRemoveLinks: true, + acceptedFiles: 'image/*', + clickable: true, accept: function(file, done) { done(); }, From 6436d2cd8ac17198704129208d1f0e499f1cd83c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Fri, 17 Mar 2023 10:49:12 +0100 Subject: [PATCH 30/36] changes in case of code-review --- src/Content/Conversation.php | 1 - view/js/dropzone-factory.js | 12 ++++++------ view/theme/frio/templates/comment_item.tpl | 8 ++++---- 3 files changed, 10 insertions(+), 11 deletions(-) diff --git a/src/Content/Conversation.php b/src/Content/Conversation.php index f23333911..249190b1a 100644 --- a/src/Content/Conversation.php +++ b/src/Content/Conversation.php @@ -408,7 +408,6 @@ class Conversation '$compose_link_title' => $this->l10n->t('Open Compose page'), '$always_open_compose' => $this->pConfig->get($this->session->getLocalUserId(), 'frio', 'always_open_compose', false), - ]); diff --git a/view/js/dropzone-factory.js b/view/js/dropzone-factory.js index a3a3e6c36..21ea5ebbc 100644 --- a/view/js/dropzone-factory.js +++ b/view/js/dropzone-factory.js @@ -1,6 +1,6 @@ var DzFactory = function () { - this.createDropzone = function(dropSelector, textareaSelector) { - return new Dropzone( dropSelector, { + this.createDropzone = function(dropSelector, textareaElementId) { + return new Dropzone(dropSelector, { paramName: 'userfile', // The name that will be used to transfer the file maxFilesize: max_imagesize, // MB url: '/media/photo/upload?response=url&album=', @@ -12,7 +12,7 @@ var DzFactory = function () { }, init: function() { this.on('success', function(file, serverResponse) { - const targetTextarea = document.getElementById(textareaSelector); + const targetTextarea = document.getElementById(textareaElementId); const bbcodeString = $(serverResponse).find('div#content').text(); if (targetTextarea.setRangeText) { //if setRangeText function is supported by current browser @@ -23,7 +23,7 @@ var DzFactory = function () { } }); this.on('complete', function(file) { - var dz = this; + const dz = this; // Remove just uploaded file from dropzone, makes interface more clear. // Image can be seen in posting-preview // We need preview to get optical feedback about upload-progress. @@ -55,8 +55,8 @@ var DzFactory = function () { }) }; - this.setupDropzone = function(dropSelector, textareaSelector) { - var dropzone = this.createDropzone(dropSelector, textareaSelector); + this.setupDropzone = function(dropSelector, textareaElementId) { + var dropzone = this.createDropzone(dropSelector, textareaElementId); $(dropSelector).on('paste', function(event) { dzFactory.copyPaste(event, dropzone); }) diff --git a/view/theme/frio/templates/comment_item.tpl b/view/theme/frio/templates/comment_item.tpl index 81664cf0f..2d977d807 100644 --- a/view/theme/frio/templates/comment_item.tpl +++ b/view/theme/frio/templates/comment_item.tpl @@ -68,12 +68,12 @@ From 8a3361dfbe041983e5547dfe774f8ca91dd7d8c3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Fri, 17 Mar 2023 10:54:31 +0100 Subject: [PATCH 31/36] remove upload-url parameter --- view/js/dropzone-factory.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/view/js/dropzone-factory.js b/view/js/dropzone-factory.js index 21ea5ebbc..a5010b95e 100644 --- a/view/js/dropzone-factory.js +++ b/view/js/dropzone-factory.js @@ -3,7 +3,7 @@ var DzFactory = function () { return new Dropzone(dropSelector, { paramName: 'userfile', // The name that will be used to transfer the file maxFilesize: max_imagesize, // MB - url: '/media/photo/upload?response=url&album=', + url: '/media/photo/upload?album=', addRemoveLinks: true, acceptedFiles: 'image/*', clickable: true, From 399ee74e05da635df548df4911f2cd904fe2e3f0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Fri, 17 Mar 2023 13:06:42 +0100 Subject: [PATCH 32/36] revert to original (no changes needed anymore) --- src/Module/Item/Compose.php | 1 + 1 file changed, 1 insertion(+) diff --git a/src/Module/Item/Compose.php b/src/Module/Item/Compose.php index f53158b1f..d44fe2cd7 100644 --- a/src/Module/Item/Compose.php +++ b/src/Module/Item/Compose.php @@ -164,6 +164,7 @@ class Compose extends BaseModule $this->page->registerFooterScript(Theme::getPathForFile('js/ajaxupload.js')); $this->page->registerFooterScript(Theme::getPathForFile('js/linkPreview.js')); $this->page->registerFooterScript(Theme::getPathForFile('js/compose.js')); + $contact = Contact::getById($a->getContactId()); if ($this->pConfig->get(DI::userSession()->getLocalUserId(), 'system', 'set_creation_date')) { From faa524faecdf5a8920fea9d169e60dfea32fb735 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Fri, 17 Mar 2023 14:22:32 +0100 Subject: [PATCH 33/36] revert changes, remove line (from code-review) --- src/App/Page.php | 1 - src/Module/Post/Edit.php | 4 +--- src/Object/Post.php | 2 +- 3 files changed, 2 insertions(+), 5 deletions(-) diff --git a/src/App/Page.php b/src/App/Page.php index 45ec6be0f..23f542f7d 100644 --- a/src/App/Page.php +++ b/src/App/Page.php @@ -261,7 +261,6 @@ class Page implements ArrayAccess // Dropzone '$max_imagesize' => round(\Friendica\Util\Strings::getBytesFromShorthand($config->get('system', 'maximagesize')) / 1000000, 1), - ]) . $this->page['htmlhead']; } diff --git a/src/Module/Post/Edit.php b/src/Module/Post/Edit.php index 5ddc250d8..5fb339e17 100644 --- a/src/Module/Post/Edit.php +++ b/src/Module/Post/Edit.php @@ -28,7 +28,6 @@ use Friendica\Core\Hook; use Friendica\Core\L10n; use Friendica\Core\Renderer; use Friendica\Core\Session\Capability\IHandleUserSessions; -use Friendica\DI; use Friendica\Model\Contact; use Friendica\Model\Post; use Friendica\Model\User; @@ -36,7 +35,6 @@ use Friendica\Module\Response; use Friendica\Navigation\SystemMessages; use Friendica\Network\HTTPException; use Friendica\Util\Crypto; -use Friendica\Util\Strings; use Friendica\Util\Profiler; use Psr\Log\LoggerInterface; @@ -184,8 +182,8 @@ class Edit extends BaseModule '$shortpermset' => $this->t('Permissions'), '$compose_link_title' => $this->t('Open Compose page'), - ]); + return $output; } diff --git a/src/Object/Post.php b/src/Object/Post.php index 3f4d061a1..17c0c1f68 100644 --- a/src/Object/Post.php +++ b/src/Object/Post.php @@ -1068,7 +1068,7 @@ class Post '$prompttext' => DI::l10n()->t('Please enter a image/video/audio/webpage URL:'), '$preview' => DI::l10n()->t('Preview'), '$indent' => $indent, - '$rand_num' => Crypto::randomDigits(12), + '$rand_num' => Crypto::randomDigits(12) ]); } From 57a7a3d6d803d0df59d17241e2d0bcce96abf275 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Fri, 17 Mar 2023 15:17:55 +0100 Subject: [PATCH 34/36] change responsehandling to be ready for PR #12892 --- view/js/dropzone-factory.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/view/js/dropzone-factory.js b/view/js/dropzone-factory.js index a5010b95e..1b2a29337 100644 --- a/view/js/dropzone-factory.js +++ b/view/js/dropzone-factory.js @@ -13,7 +13,7 @@ var DzFactory = function () { init: function() { this.on('success', function(file, serverResponse) { const targetTextarea = document.getElementById(textareaElementId); - const bbcodeString = $(serverResponse).find('div#content').text(); + const bbcodeString = serverResponse; if (targetTextarea.setRangeText) { //if setRangeText function is supported by current browser targetTextarea.setRangeText(' ' + $.trim(bbcodeString) + ' '); From 5040bd9a1a69d0956a67f15590a4acfeb29969f4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Fri, 17 Mar 2023 17:05:18 +0100 Subject: [PATCH 35/36] change response-handling (code-review) --- view/js/dropzone-factory.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/view/js/dropzone-factory.js b/view/js/dropzone-factory.js index 1b2a29337..c2246dfa2 100644 --- a/view/js/dropzone-factory.js +++ b/view/js/dropzone-factory.js @@ -13,13 +13,12 @@ var DzFactory = function () { init: function() { this.on('success', function(file, serverResponse) { const targetTextarea = document.getElementById(textareaElementId); - const bbcodeString = serverResponse; if (targetTextarea.setRangeText) { //if setRangeText function is supported by current browser - targetTextarea.setRangeText(' ' + $.trim(bbcodeString) + ' '); + targetTextarea.setRangeText(' ' + $.trim(serverResponse) + ' '); } else { targetTextarea.focus(); - document.execCommand('insertText', false /*no UI*/, '\n' + $.trim(bbcodeString) + '\n'); + document.execCommand('insertText', false /*no UI*/, '\n' + $.trim(serverResponse) + '\n'); } }); this.on('complete', function(file) { From cf63cbde72e5fd41d237a3fe3aef3de4d07c9f1b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakobus=20Sch=C3=BCrz?= Date: Sun, 19 Mar 2023 19:14:10 +0100 Subject: [PATCH 36/36] remove removelink (it is not useful at the moment) --- view/js/dropzone-factory.js | 1 - 1 file changed, 1 deletion(-) diff --git a/view/js/dropzone-factory.js b/view/js/dropzone-factory.js index c2246dfa2..ca77bb6d5 100644 --- a/view/js/dropzone-factory.js +++ b/view/js/dropzone-factory.js @@ -4,7 +4,6 @@ var DzFactory = function () { paramName: 'userfile', // The name that will be used to transfer the file maxFilesize: max_imagesize, // MB url: '/media/photo/upload?album=', - addRemoveLinks: true, acceptedFiles: 'image/*', clickable: true, accept: function(file, done) {