Bug 369455

Summary:	Version 16.08.1 breaks the Position and Zoom plugin
Product:	[Applications] kdenlive	Reporter:	ryeb
Component:	Video Effects & Transitions	Assignee:	Vincent PINON <vpinon>
Status:	RESOLVED FIXED
Severity:	normal	CC:	jb
Priority:	NOR
Version:	16.08.0
Target Milestone:	---
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:	http://commits.kde.org/kdenlive/aed3cd0a8a65f3021c71ec92df35ace52953d9d7	Version Fixed In:
Sentry Crash Report:

Description ryeb 2016-09-28 01:46:41 UTC

After updating to Kdenlive version 16.08.1 the Position and Zoom plugin no longer works.

Reproducible: Always

Steps to Reproduce:
1. Click on a clip in the timeline
2. In the video effects tab choose Crop and Transform => Position and Zoom
3. Add a couple of keyframes and then zoom in and out
4. Preview your clip (or render)

Actual Results:  
The red boarder and white dot will move, but the actual video clip is not effected.

Expected Results:  
A zoom effect in the video preview and exported file.

Running up to date Fedora 24. Kdenlive installed via RPM Fusion. If you downgrade Kdenlive to version 16.04.2, the plugin works.

Comment 1 Jean-Baptiste Mardelle 2016-09-28 08:52:07 UTC

I can confirm an issue when using a locale with a comma as decimal separator. Working on it. Can you please paste here the output of the following command in a terminal:

locale

Comment 2 Jean-Baptiste Mardelle 2016-09-28 09:08:14 UTC

Git commit aed3cd0a8a65f3021c71ec92df35ace52953d9d7 by Jean-Baptiste Mardelle.
Committed on 28/09/2016 at 09:07.
Pushed by mardelle into branch 'Applications/16.08'.

Fix several geometry effects broken on locale with comma separator (french, german,...)

M  +1    -1    data/effects/audiowaveform.xml
M  +1    -1    data/effects/dynamictext.xml
M  +1    -1    data/effects/pan_zoom.xml
M  +1    -1    data/effects/region.xml
M  +1    -1    data/effects/rotation.xml

http://commits.kde.org/kdenlive/aed3cd0a8a65f3021c71ec92df35ace52953d9d7

Comment 3 ryeb 2016-09-29 02:57:00 UTC

It looks like you don't need this anymore, but just in case, the output of locale is:

LANG=en_CA.UTF-8
LC_CTYPE="en_CA.UTF-8"
LC_NUMERIC="en_CA.UTF-8"
LC_TIME="en_CA.UTF-8"
LC_COLLATE="en_CA.UTF-8"
LC_MONETARY="en_CA.UTF-8"
LC_MESSAGES="en_CA.UTF-8"
LC_PAPER="en_CA.UTF-8"
LC_NAME="en_CA.UTF-8"
LC_ADDRESS="en_CA.UTF-8"
LC_TELEPHONE="en_CA.UTF-8"
LC_MEASUREMENT="en_CA.UTF-8"
LC_IDENTIFICATION="en_CA.UTF-8"
LC_ALL=