fix(ImageProviders): set size on request

56890720 · Ronan Abhamon · afceb67c · 56890720 · 56890720
Commit 56890720 authored Jun 23, 2017 by Ronan Abhamon
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 4 deletions

AvatarProvider.cpp src/app/providers/AvatarProvider.cpp +4 -2

ThumbnailProvider.cpp src/app/providers/ThumbnailProvider.cpp +4 -2

No files found.
--- a/src/app/providers/AvatarProvider.cpp
+++ b/src/app/providers/AvatarProvider.cpp
@@ -36,6 +36,8 @@ AvatarProvider::AvatarProvider () : QQuickImageProvider(
  mAvatarsPath = ::Utils::coreStringToAppString(Paths::getAvatarsDirPath());
 }
-QImage AvatarProvider::requestImage (const QString &id, QSize *, const QSize &) {
+QImage AvatarProvider::requestImage (const QString &id, QSize *size, const QSize &) {
-  return QImage(mAvatarsPath + id);
+  QImage image(mAvatarsPath + id);
+  *size = image.size();
+  return image;
 }
--- a/src/app/providers/ThumbnailProvider.cpp
+++ b/src/app/providers/ThumbnailProvider.cpp
@@ -36,6 +36,8 @@ ThumbnailProvider::ThumbnailProvider () : QQuickImageProvider(
  mThumbnailsPath = ::Utils::coreStringToAppString(Paths::getThumbnailsDirPath());
 }
-QImage ThumbnailProvider::requestImage (const QString &id, QSize *, const QSize &) {
+QImage ThumbnailProvider::requestImage (const QString &id, QSize *size, const QSize &) {
-  return QImage(mThumbnailsPath + id);
+  QImage image(mThumbnailsPath + id);
+  *size = image.size();
+  return image;
 }