feat: 4674 - immediate access to "other" pictures if possible

[monsieurtanuki]

What

• With off-dart 3.3.0 we now have a more direct access to "other" pictures.
• Basically, now we fetch and save in the local database product table "main" AND "other" pictures links.
• That means that most of the time we don't need to download the "other" image ids from the server, as we already have them.
• "most of the time"? Yes, because for the legacy data already in the local database we don't have the "other" pictures yet. In that case we keep the "get other pics" button and just refresh the product.
• En passant while checking I refactored a little the background tasks related to product changes.

Fixes bug(s)

• Fixes: Extra photos fail to load with an empty product #4911

Part of

• New feature: create a screen to view all photos from a product #4674

Files

New file:

• background_task_product_change.dart: Interface for background tasks that change a product.

Impacted files:

• background_task_crop.dart: minor refactoring
• background_task_details.dart: minor refactoring, implementing new interface BackgroundTaskProductChange
• background_task_image.dart: minor refactoring
• background_task_unselected.dart: minor refactoring, implementing new interface BackgroundTaskProductChange
• background_task_upload.dart: minor refactoring, implementing new interface BackgroundTaskProductChange
• product_cards_helper.dart: new method getRawProductImages
• product_image_gallery_other_view.dart: now we may have directly the raw images, and if needed we refresh the product instead of fetching specific image ids
• product_image_gallery_view.dart: minor refactoring
• product_image_other_page.dart: replaced deprecated method
• product_image_server_button.dart: now we may have directly the raw images, and if needed we refresh the product instead of fetching specific image ids
• product_image_viewer.dart: replaced deprecated method
• pubspec.lock: generated
• pubspec.yaml: upgraded to off-dart 3.3.0
• uploaded_image_gallery.dart: now using List<ProductImage> as input

[codecov-commenter]

Codecov Report

Attention: 126 lines in your changes are missing coverage. Please review.

Comparison is base (85784fa) 9.62% compared to head (ccd65e8) 9.60%.

Files	Patch %	Lines
.../pages/image/product_image_gallery_other_view.dart	0.00%	41 Missing ⚠️
...lib/pages/product/product_image_server_button.dart	0.00%	30 Missing ⚠️
...s/smooth_app/lib/helpers/product_cards_helper.dart	0.00%	17 Missing ⚠️
...oth_app/lib/background/background_task_upload.dart	0.00%	9 Missing ⚠️
...th_app/lib/pages/image/uploaded_image_gallery.dart	0.00%	7 Missing ⚠️
.../lib/pages/product/product_image_gallery_view.dart	0.00%	5 Missing ⚠️
...mooth_app/lib/background/background_task_crop.dart	0.00%	4 Missing ⚠️
...th_app/lib/background/background_task_details.dart	0.00%	3 Missing ⚠️
..._app/lib/pages/image/product_image_other_page.dart	0.00%	3 Missing ⚠️
...th_app/lib/pages/product/product_image_viewer.dart	0.00%	3 Missing ⚠️
... and 2 more

Additional details and impacted files

@@            Coverage Diff             @@
##           develop   #4928      +/-   ##
==========================================
- Coverage     9.62%   9.60%   -0.03%     
==========================================
  Files          323     323              
  Lines        16267   16302      +35     
==========================================
  Hits          1565    1565              
- Misses       14702   14737      +35     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[monsieurtanuki]

For the record we need this PR to be merged in order to implement new features for the prices API. It's using off-dart 3.3.0, which has changes that triggers lint warnings.
Or, we need a different PR that just deals with the 3.3.0 version induced warnings.
Or, we are blocked.

[teolemon]

@g123k @M123-dev @AshAman999 Could you have a look at this one ?

[M123-dev]

Looks about right, thanks @monsieurtanuki

[monsieurtanuki]

Thank you @M123-dev for your review!

Possible impact as there are many "raw" images and they are stored in a verbose mode.
Room for improvements if needed, we'll see.