?
avatar brianteeman
brianteeman
24 Sep 2014

In a vanilla install of Joomla the default Maximum Size for an image upload is now 10MB but the default Maximum Size for a template upload is still only 2MB - would it make sense to at least increase the template size to match or maybe even better use the same value?

avatar brianteeman brianteeman - open - 24 Sep 2014
avatar Kubik-Rubik
Kubik-Rubik - comment - 24 Sep 2014

Yes, we could set the same value per default but in the end it depends on the PHP settings (upload_max_filesize and post_max_size) what size will be accepted or not from the server. I think that we should check whether the limit is lower than 10MB and then show the correct value in the Media Manager, e.g. 4MB or whatever and not just show a fixed number as we do now.

This comment was created with the J!Tracker Application at http://issues.joomla.org/.

avatar brianteeman
brianteeman - comment - 24 Sep 2014

I personally agree with you on that concept @Kubik-Rubik Do you think you could code that as a solution?

avatar Kubik-Rubik
Kubik-Rubik - comment - 24 Sep 2014

@brianteeman Sure! We could use for example some functions from Drupal: http://stackoverflow.com/a/25370978 ;-)

This comment was created with the J!Tracker Application at http://issues.joomla.org/.

avatar brianteeman
brianteeman - comment - 24 Sep 2014

I definitely think its an improvement to report the server setting

On 24 September 2014 21:38, Viktor Vogel notifications@github.com wrote:

@brianteeman https://github.com/brianteeman Sure! We could use for
example some functions from Drupal: http://stackoverflow.com/a/25370978
;-)

This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at
http://issues.joomla.org/http://issues.joomla.org/
http://issues.joomla.org/.

—
Reply to this email directly or view it on GitHub
#4337 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar beat
beat - comment - 24 Sep 2014

I agree it is a big improvement, and showing from where the limit comes (and thus giving a hint on how to change it if needed) is even better. This should also be ported to extensions by upload installations imho.

avatar brianteeman
brianteeman - comment - 24 Sep 2014

If we make this change then I would hope we would make it globally wherever
we have upload limits and if it is an API then extension developers can
easily use it

On 24 September 2014 22:03, beat notifications@github.com wrote:

I agree it is a big improvement, and showing from where the limit comes
(and thus giving a hint on how to change it if needed) is even better. This
should also be ported to extensions by upload installations imho.

—
Reply to this email directly or view it on GitHub
#4337 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar Kubik-Rubik
Kubik-Rubik - comment - 24 Sep 2014

@brianteeman Yes, we should add it into the library to have the possibility to call it globally.
@bakual I would also solve it in a similar way. Thanks for the input!

I will create something tomorrow! :-)

This comment was created with the J!Tracker Application at http://issues.joomla.org/.

avatar Kubik-Rubik Kubik-Rubik - change - 24 Sep 2014
Status New ⇒ Confirmed
avatar Kubik-Rubik
Kubik-Rubik - comment - 25 Sep 2014

Here we go: #4348

Please test first the library implementation. If everything is okay and we have it in the core, we can replace every hard-coded values (e.g. Media Manager).

avatar Kubik-Rubik
Kubik-Rubik - comment - 25 Sep 2014

This issue can be closed in favor of #4348

This comment was created with the J!Tracker Application at http://issues.joomla.org/.

avatar brianteeman brianteeman - change - 25 Sep 2014
Status Confirmed ⇒ Closed
avatar brianteeman brianteeman - close - 25 Sep 2014
avatar zero-24 zero-24 - close - 25 Sep 2014
avatar brianteeman brianteeman - change - 25 Sep 2014
Closed_Date 0000-00-00 00:00:00 ⇒ 2014-09-25 15:30:58
avatar zero-24 zero-24 - change - 7 Jul 2015
Labels Added: ?

Add a Comment

Login with GitHub to post a comment