From 9ade298cd66056237866025b2b658e29d86e8fb9 Mon Sep 17 00:00:00 2001
From: Jeff Schiller <schiller@helium>
Date: Sat, 1 Aug 2009 10:48:55 -0500
Subject: [PATCH] Also cover quoted forms of url(#grad) for removing duplicate
 gradients

---
 scour.py                                 | 3 ++-
 unittests/remove-duplicate-gradients.svg | 6 +++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/scour.py b/scour.py
index a227eed..6b21416 100755
--- a/scour.py
+++ b/scour.py
@@ -759,7 +759,8 @@ def removeDuplicateGradients(doc):
 				# find out which attribute referenced the duplicate gradient
 				for attr in ['fill', 'stroke']:
 					# TODO: also need to check for url("#id")
-					if elem.getAttribute(attr) == 'url(#'+dup_id+')':
+					v = elem.getAttribute(attr)
+					if v == 'url(#'+dup_id+')' or v == 'url("#'+dup_id+'")' or v == "url('#"+dup_id+"')":
 						elem.setAttribute(attr, 'url(#'+master_id+')')
 				if elem.getAttributeNS(NS['XLINK'], 'href') == '#'+dup_id:
 					elem.setAttributeNS(NS['XLINK'], 'href', '#'+master_id)
diff --git a/unittests/remove-duplicate-gradients.svg b/unittests/remove-duplicate-gradients.svg
index c529d59..536344a 100644
--- a/unittests/remove-duplicate-gradients.svg
+++ b/unittests/remove-duplicate-gradients.svg
@@ -12,7 +12,7 @@
 	<radialGradient xlink:href="#g2" id="g4" gradientUnits="userSpaceOnUse" gradientTransform="matrix(1, 0, 0, 0.481529, -718.417, 98.7046)" cx="323.75433" cy="209.73672" fx="323.75433" fy="209.73672" r="6.2794499"/>
 	
 	<rect id="r1" fill="url(#g1)" width="100" height="100"/>
-	<rect id="r2" stroke="url(#g2)" width="100" height="100"/>
+	<rect id="r2" stroke="url('#g2')" width="100" height="100"/>
 	<rect id="r3" stroke="url(#g3)" width="100" height="100"/>
-	<rect id="r4" fill="url(#g4)" width="100" height="100"/>
-</svg>
\ No newline at end of file
+	<rect id="r4" fill='url("#g4")' width="100" height="100"/>
+</svg>