Error message pops up when moving mouse

[Michael Rudolph]

[ Impact ]

One way to reproduce the error message stated below is to create a new project and try to add a picture. Maybe it's sufficient under certain circumstances to just move the mouse cursor.

A bunch of upstream commits have been applied in the proposed upload, which turns the application back to a working state.

[ Test Plan ]

* Install photofilmstrip from jammy-proposed.

* Open the application and create a new project.

* Find that you successfully can add a picture.

[ Where problems could occur ]

Two of the patches consist of upstream commits which were specifically intended to address this issue in Ubuntu 22.04. But in order to make those patches apply, a couple of previous commits were patched too. So the risk for surprise side effects is not zero, but OTOH the alternative is a completely broken package.

[ Original description ]

The message is:

Traceback (most recent call last):
   File "/usr/lib/python3/dist-packages/wx/core.py", line 3282, in <lambda>
     lambda event: event.callable(*event.args, **event.kw) )
   File "/usr/lib/python3/dist-packages/photofilmstrip/lib/common/ObserverPattern.py", line 26, in Notify
     observer.ObservableUpdate(self, arg)
   File "/usr/lib/python3/dist-packages/photofilmstrip/gui/ImageSectionEditor.py", line 92, in ObservableUpdate
     self.__Scale()
   File "/usr/lib/python3/dist-packages/photofilmstrip/gui/ImageSectionEditor.py", line 122, in __Scale
     self._imgProxy.Scale(newWidth, newHeight)
   File "/usr/lib/python3/dist-packages/photofilmstrip/gui/ImageSectionEditor.py", line 675, in Scale
     img = self._wxImg.Scale(width, height)
 TypeError: Image.Scale(): argument 2 has unexpected type 'float'

This problem can be solved by installing version 4.0.0 of photofilmstrip. This version should be in the distribution instead of version 3.7.3

[Gunnar Hjalmarsson (gunnarhj)]

A proposed fix has been uploaded to the jammy unapproved queue. If someone wants to test before it reaches jammy-proposed, it's also available in this PPA:

https://launchpad.net/~gunnarhj/+archive/ubuntu/photofilmstrip

[Timo Aaltonen (tjaalton)]

Hello Michael, or anyone else affected,

Accepted photofilmstrip into jammy-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/photofilmstrip/3.7.3-2ubuntu22.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-jammy to verification-done-jammy. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-jammy. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

[Gunnar Hjalmarsson (gunnarhj)]

I accomplished the steps in the test plan using photofilmstrip 3.7.3-2ubuntu22.04.1 from jammy-proposed.

[Robie Basak (racb)]

The verification of the Stable Release Update for photofilmstrip has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

[Launchpad Janitor (janitor)]

This bug was fixed in the package photofilmstrip - 3.7.3-2ubuntu22.04.1

---------------
photofilmstrip (3.7.3-2ubuntu22.04.1) jammy; urgency=medium

  * Fix issue likely caused by post release python 3.10 update
    - d/p/Draw_rule_of_thirds_guides_in_rectangle_editor.patch
    - d/p/use_own_cursor_for_moving_rectangle.patch
    - d/p/round_floats_to_int_before_passing_them_to_wx-api.patch
    - d/p/round_floats_to_int_in_picture_rectangles.patch
    - LP: #2024190

 -- Gunnar Hjalmarsson <email address hidden> Sat, 01 Jul 2023 22:59:46 +0200