mahara-contributors team mailing list archive

[Alan McNatty <717184@xxxxxxxxxxxxxxxxxx>]

Just a comment on the implementation.

It is a shame this was implemented in the view (function
pieform_element_filebrowser_upload) and not in the base file artefact
class (i.e. ArtefactTypeFile::save_uploaded_file method).

If a user has enabled image resizing then
ArtefactTypeFile::save_uploaded_file should resize the image on save
automatically. What has been implemented is a Fix to the web-form only -
the biggest culprit to uploading oversized images may be mobile devices
(as the mobile user often lacks the control to resize the content on the
device)

With this implementation the mobile API would need to reinvent the
wheel. The endable resize config option does not actual resize all
uploaded content.

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
https://bugs.launchpad.net/bugs/717184

Title:
  Resize images at upload time

Status in Mahara ePortfolio:
  Fix Committed

Bug description:
  I'm aware that the roadmap for 1.4 mentions resizing images on the fly
  for use in Views, however, I feel that it would be more appropriate to
  resize images when they are uploaded as this would avoid using too
  much of a user's quota. The way I envisage this is that the site
  administrator sets a maximum size and any image larger than this is
  automatically resized. Perhaps also there could be a checkbox when
  uploading to override this, in the rare case that an image needs to be
  full-resolution.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/717184/+subscriptions