Hacks for comparing sql attribute in equals() to ignore unimportant differences

[dbsrgits/SQL-Translator.git] / lib / SQL / Translator / Schema / View.pm

diff --git a/lib/SQL/Translator/Schema/View.pm b/lib/SQL/Translator/Schema/View.pm
index 2ad4bf9..401cbc5 100644 (file)
--- a/lib/SQL/Translator/Schema/View.pm
+++ b/lib/SQL/Translator/Schema/View.pm
@@ -1,7 +1,7 @@
 package SQL::Translator::Schema::View;
 
 # ----------------------------------------------------------------------
-# $Id: View.pm,v 1.12 2005-08-10 16:46:55 duality72 Exp $
+# $Id: View.pm,v 1.13 2007-03-14 20:22:58 duality72 Exp $
 # ----------------------------------------------------------------------
 # Copyright (C) 2002-4 SQLFairy Authors
 #
@@ -50,7 +50,7 @@ use base 'SQL::Translator::Schema::Object';
 
 use vars qw($VERSION $TABLE_COUNT $VIEW_COUNT);
 
-$VERSION = sprintf "%d.%02d", q$Revision: 1.12 $ =~ /(\d+)\.(\d+)/;
+$VERSION = sprintf "%d.%02d", q$Revision: 1.13 $ =~ /(\d+)\.(\d+)/;
 
 # ----------------------------------------------------------------------
 
@@ -229,7 +229,17 @@ Determines if this view is the same as another
     return 0 unless $self->SUPER::equals($other);
     return 0 unless $case_insensitive ? uc($self->name) eq uc($other->name) : $self->name eq $other->name;
     #return 0 unless $self->is_valid eq $other->is_valid;
-    return 0 unless $self->sql eq $other->sql;
+    
+    my $selfSql = $self->sql;
+    my $otherSql = $other->sql;
+    # Remove comments
+    $selfSql =~ s/--.*$//mg;
+    $otherSql =~ s/--.*$//mg;
+    # Collapse whitespace to space to avoid whitespace comparison issues
+    $selfSql =~ s/\s+/ /sg;
+    $otherSql =~ s/\s+/ /sg;
+    return 0 unless $selfSql eq $otherSql;
+    
     my $selfFields = join(":", $self->fields);
     my $otherFields = join(":", $other->fields);
     return 0 unless $case_insensitive ? uc($selfFields) eq uc($otherFields) : $selfFields eq $otherFields;