OpenClipart import crashes with certain clipart(s?)

Bug #1435074 reported by SPNK
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Invalid
High
Unassigned

Bug Description

I go to
1. File > Import Clipart
2. Type "phone pictogram/symbol"
3. Click on "Phone", a clipart by temsq which has exactly the description we just searched for
4. Click "Import"
The result is the error message I just have added.

My operating system is Windows 8.1
and my Inkscape Version is 0.91 r13725.
I downloaded the installer for Windows 64bit.

Revision history for this message
SPNK (lychnus) wrote :
Revision history for this message
su_v (suv-lp) wrote :

Reproduced on OS X 10.7.5 with Inkscape 0.91 r13725 and 0.91+devel r14027.

tags: added: crash openclipart
Changed in inkscape:
importance: Undecided → High
status: New → Confirmed
Revision history for this message
su_v (suv-lp) wrote :
Revision history for this message
su_v (suv-lp) wrote :

The file as downloaded by Inkscape (stored in $TEMPDIR/openclipart/images)

Revision history for this message
jazzynico (jazzynico) wrote :

Also reproduced on Crunchbang Waldorf, Inkscape trunk rev.14029.
Not reproduced with 0.48.3.1.

Changed in inkscape:
milestone: none → 0.92
status: Confirmed → Triaged
tags: added: regression
Revision history for this message
jazzynico (jazzynico) wrote :

Unrelated to OpenClipArt itself, but to how the file is opened.
Opening the tmp file (the one attached comment #4) works when using File>Open, but crashes with File>Import.

Revision history for this message
jazzynico (jazzynico) wrote :

Workaround: removing the content of the defs element (or the defs element completely) fixes the issue and the file can be correctly imported.

tags: added: importing
removed: openclipart
Revision history for this message
jazzynico (jazzynico) wrote :

The CSS embedded in the OpenClipArt file is not correctly formatted.
Attached is a fixed file that follows the w3c recommendations (http://www.w3.org/TR/SVG/styling.html#StylingWithCSS).

Of course, Inkscape should handle it without crashing, but the file needs to be fixed in OCAL.

Revision history for this message
jazzynico (jazzynico) wrote :

It seems there's something to be fixed in SPDocument::importDefs when the defs has values and no child element.

jazzynico (jazzynico)
Changed in inkscape:
milestone: 0.92 → 0.93
Revision history for this message
Jonathan Hofinger (jhofinger) wrote :

Openclipart integration has been removed from Inkscape (https://gitlab.com/inkscape/inkscape/issues/440), since OpenClipart is permanently out of service. This makes this bug report obsolete.

Please feel free to file new bugs about the issues you're seeing at http://inkscape.org/report.

Closed by: https://gitlab.com/jhofinger

Changed in inkscape:
status: Triaged → Invalid
Revision history for this message
Jonathan Hofinger (jhofinger) wrote :

PS: The attached file imports fine in 0.92.4 and Inkscape 1.1-dev (c990a67, 2020-04-09) when using Open -> Import.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.