From: acspike <acspike@users.sourceforge.net>
Date: Sun, 10 Feb 2008 20:33:01 +0000 (+0000)
Subject: Partial fix
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=96e731b63dfe3eb1cd9b3c2d396be2825b34ed47;p=inkscape.git

Partial fix

This extension is currently broken and as it stands will be removed from 0.46
I'm committing this partial fix which updates the syntax for lxml anyway
Additional fixes will probably be required because as it stands this script
crashes inkscape for me
---

diff --git a/share/extensions/svg_and_media_zip_output.py b/share/extensions/svg_and_media_zip_output.py
index 93bf2ebad..b404caf4d 100644
--- a/share/extensions/svg_and_media_zip_output.py
+++ b/share/extensions/svg_and_media_zip_output.py
@@ -44,79 +44,64 @@ class MyEffect(inkex.Effect):
     def __init__(self):
         inkex.Effect.__init__(self)
 
-        self.documentDst=None
-
-    def parseTmp(self,file=None):
-        """Parse document in specified file or on stdin"""
-        reader = inkex.xml.dom.ext.reader.Sax2.Reader()
-        try:
-            try:
-                stream = open(file,'r')
-            except:
-                stream = open(self.args[-1],'r')
-        except:
-            stream = sys.stdin
-        self.documentDst = reader.fromStream(stream)
-        stream.close()
-
     def output(self):
         pass
 
     def effect(self):
+        ttmp_orig = self.document.getroot()
         
-        #get needed info from orig document
-        ctx_orig = inkex.xml.xpath.Context.Context(self.document,processorNss=inkex.NSS)
+        docbase = ttmp_orig.get(inkex.addNS('docbase',u'sodipodi'),'')
+        docname = ttmp_orig.get(inkex.addNS('docname',u'sodipodi'))
 
-        ttmp_orig = inkex.xml.xpath.Evaluate('/svg',self.document, context=ctx_orig)
-        
-        docbase = ttmp_orig[0].attributes.getNamedItemNS(inkex.NSS[u'sodipodi'],'docbase')
-        docname = ttmp_orig[0].attributes.getNamedItemNS(inkex.NSS[u'sodipodi'],'docname')
+        inkex.debug((docbase,docname))
 
         orig_tmpfile = sys.argv[1]
 
-        # create destination zip in same directory as the document
-        z = zipfile.ZipFile(docbase.value + '/'+ docname.value + '.zip', 'w')    
-
         #create os temp dir
         tmp_dir = tempfile.mkdtemp()
+        
+        # create destination zip in same directory as the document
+        z = zipfile.ZipFile(tmp_dir + '/'+ docname + '.zip', 'w')    
 
         #fixme replace whatever extention
-        docstripped = docname.value.replace('.zip', '')
+        docstripped = docname.replace('.zip', '')
     
         #read tmpdoc and copy all images to temp dir
-        for node in inkex.xml.xpath.Evaluate('//image',self.document, context=ctx_orig):
+        for node in self.document.xpath('//image',inkex.NSS):
             self.collectAndZipImages(node, tmp_dir, docname, z)
 
         ##copy tmpdoc to tempdir
         dst_file = os.path.join(tmp_dir, docstripped)
         stream = open(dst_file,'w')
     
-        inkex.xml.dom.ext.Print(self.document,stream)
+        self.document.write(stream)
         
         stream.close()
         
         z.write(dst_file.encode("latin-1"),docstripped.encode("latin-1")+'.svg') 
         z.close()
 
-        shutil.move(docbase.value + '/'+ docname.value + '.zip',docbase.value + '/'+ docname.value)
+        out = open(tmp_dir + '/'+ docname + '.zip','r')
+        sys.stdout.write(out.read())
+        out.close() 
         
         shutil.rmtree(tmp_dir)
 
     def collectAndZipImages(self, node, tmp_dir, docname, z):
-        xlink = node.attributes.getNamedItemNS(inkex.NSS[u'xlink'],'href')
+        xlink = node.get(inkex.addNS('href',u'xlink'))
         if (xlink.value[:4]!='data'):
-            absref=node.attributes.getNamedItemNS(inkex.NSS[u'sodipodi'],'absref')
+            absref=node.get(inkex.addNS('absref',u'sodipodi'))
 
-            if (os.path.isfile(absref.value)):
-                shutil.copy(absref.value,tmp_dir)
-                z.write(absref.value.encode("latin-1"),os.path.basename(absref.value).encode("latin-1"))
+            if (os.path.isfile(absref)):
+                shutil.copy(absref,tmp_dir)
+                z.write(absref.encode("latin-1"),os.path.basename(absref).encode("latin-1"))
 
-            elif (os.path.isfile(tmp_dir + '/' + absref.value)):
-                shutil.copy(tmp_dir + '/' + absref.value,tmp_dir)
-                z.write(tmp_dir + '/' + absref.value.encode("latin-1"),os.path.basename(absref.value).encode("latin-1"))
+            elif (os.path.isfile(tmp_dir + '/' + absref)):
+                shutil.copy(tmp_dir + '/' + absref,tmp_dir)
+                z.write(tmp_dir + '/' + absref.encode("latin-1"),os.path.basename(absref).encode("latin-1"))
 
-            xlink.value = os.path.basename(absref.value)
-            absref.value = os.path.basename(absref.value)
+            node.set(inkex.addNS('href',u'xlink'),os.path.basename(absref))
+            node.set(inkex.addNS('absref',u'sodipodi'),os.path.basename(absref))
 
             
 e = MyEffect()