From 8f38577b1359f2b834b43418df8bf5e47e29f0a7 Mon Sep 17 00:00:00 2001
From: David Heinemeier Hansson <david@loudthinking.com>
Date: Tue, 9 Sep 2008 23:14:14 -0500
Subject: [PATCH] Add back version check for REXML but account for different
 constants

---
 .../lib/active_support/core_ext/rexml.rb      | 45 ++++++++++---------
 1 file changed, 24 insertions(+), 21 deletions(-)

diff --git a/activesupport/lib/active_support/core_ext/rexml.rb b/activesupport/lib/active_support/core_ext/rexml.rb
index 058295b0571cb..82918f451560c 100644
--- a/activesupport/lib/active_support/core_ext/rexml.rb
+++ b/activesupport/lib/active_support/core_ext/rexml.rb
@@ -5,29 +5,32 @@
 # http://www.ruby-lang.org/en/news/2008/08/23/dos-vulnerability-in-rexml/
 # This fix is identical to rexml-expansion-fix version 1.0.1
 
-module REXML
-  class Entity < Child
-    undef_method :unnormalized
-    def unnormalized
-      document.record_entity_expansion! if document
-      v = value()
-      return nil if v.nil?
-      @unnormalized = Text::unnormalize(v, parent)
-      @unnormalized
-    end
-  end
-  class Document < Element
-    @@entity_expansion_limit = 10_000
-    def self.entity_expansion_limit= val
-      @@entity_expansion_limit = val
+# Earlier versions of rexml defined REXML::Version, newer ones REXML::VERSION
+unless (defined?(REXML::VERSION) ? REXML::VERSION : REXML::Version) > "3.1.7.2"
+  module REXML
+    class Entity < Child
+      undef_method :unnormalized
+      def unnormalized
+        document.record_entity_expansion! if document
+        v = value()
+        return nil if v.nil?
+        @unnormalized = Text::unnormalize(v, parent)
+        @unnormalized
+      end
     end
+    class Document < Element
+      @@entity_expansion_limit = 10_000
+      def self.entity_expansion_limit= val
+        @@entity_expansion_limit = val
+      end
 
-    def record_entity_expansion!
-      @number_of_expansions ||= 0
-      @number_of_expansions += 1
-      if @number_of_expansions > @@entity_expansion_limit
-        raise "Number of entity expansions exceeded, processing aborted."
+      def record_entity_expansion!
+        @number_of_expansions ||= 0
+        @number_of_expansions += 1
+        if @number_of_expansions > @@entity_expansion_limit
+          raise "Number of entity expansions exceeded, processing aborted."
+        end
       end
     end
   end
-end
+end
\ No newline at end of file