bend path effect doesn't work in 0.47 but does in 0.46

[Will Pittenger]

I am having troubles in 0.47 with bent paths. My test object was a vertical text object that I converted to a path. As noted below:

* Setting the width to units of the length has no effect
* Turning on Original Path is vertical causes only the bottommost character in my string ("Boulevard Albert ler") to be between the nodes.
* If I paste a path into the effect or import a bent path (also text) that 0.46 created, it renders each character a bigger than the previous one. In the case of the imported path, the characters were so large that Inkscape could only show me one at a time.

[Will Pittenger (will-pittenger1)]

Almost forgot: Undo does nothing when a Bend Path operation would be undone. I tried both the Edit menu and the Undo History dialog.

[su_v (suv-lp)]

Could you please attach sample files created with Inkscape 0.46 and with 0.47?
… and add information about Inkscape version and OS used.

[Will Pittenger (will-pittenger1)]

My OS is Vista 64-bit SP2.

[su_v (suv-lp)]

1) 'Bent path as created by 0.46.svg': Bend LPE rendered correctly in 0.46, but not in 0.47
Confirmed with Inkscape 0.46-2, Inkscape 0.47pre2-3 and Inkscape 0.46+devel r22288 on OS X 10.5.8

2) 'Bent path doesn't work in 0.47.svg': option 'Original Path is vertical' seems currently broken:
besides not rendering LPE's from 0.46 correctly Inkscape 0.47pre2-3 and r22288 crash repeatedly while node-editing the skeleton path when this option is checked. It doesn't matter whether LPE is applied to a group or a single path.

[jazzynico (jazzynico)]

Also confirmed on Ubuntu 9.04, Inkscape rev. 22293.
When path editing, Inkscape send dozens of "How to handle helperpaths for non-shapes?" in the console.
I also tried to move a node, and it led me to a total freeze (I couldn't even access to another window, and had to kill Inkscape from an xterm).

Backtrace attached.

[su_v (suv-lp)]

> "How to handle helperpaths for non-shapes?"
I see them too - tons of them ;-) but only when LPE is applied to group of paths (maybe shapes as well?).

found this in the Inkscape Wiki (old GSoC 2007 page):
° No helperpath on shapes other than paths
<http://wiki.inkscape.org/wiki/index.php/LivePathEffects#BUGS:>

[Johan Engelen (johanengelen)]

Here a patch to get rid of the annoying message about the helperpath

[Johan Engelen (johanengelen)]

Think I've found the bug , working on it now.

[Johan Engelen (johanengelen)]

This patch fixes lpe-Bend path.
I've also added a testcase to the testsuite, however i stumbled on another problem with closed/unclosed paths. For this testcase, 0.47 bend outputs closed paths, while 0.46 doesn't .

[jazzynico (jazzynico)]

Tested on Ubuntu 9.04 with the two attached files, and the patch looks good. All the reported bugs seem to be fixed.
Thanks Johan!

[su_v (suv-lp)]

I have a problem repeating the vertical path effect from 0.46 in 0.47: It is rendered correctly with the patch applied, but I cannot create a similarly bent vertical text, even with Johan's patch, with 0.46+devel r22293: after setting the option 'Original Path is vertical' and node-editing the bend path on the 90° rotated path, the rendering of the path with LPE applied isn't restored to its expected vertical position.

User error?

[ScislaC (scislac)]

Thanks Johan! Committed in r22298.

[su_v (suv-lp)]

… but I confirm with Inkscape r22293+patch on OS X 10.5.8:
- no more crashes when node-editing the bend path
- no more warnings about missing helper paths.

thank you, Johan!

Note: 'Original Path is vertical': if I rotate the bent text 90° plus mirror it along its vertical axis I get the expected effect as shown in 'Bent_path-0.46.svg'

[su_v (suv-lp)]

~suv wrote:
> User error?
*blush* yes... got it (almost)

sorry, all fixed.