Manual placement should be able to override 'position' about annotate_models HOT 4 CLOSED

--- annotate_models.rb.orig     2011-06-03 13:16:27.000000000 -0400
+++ annotate_models.rb  2011-06-03 13:17:34.000000000 -0400
@@ -126,11 +126,12 @@
         if old_header == new_header
           false
         else
-          # Remove old schema info
-          old_content.sub!(/^# #{COMPAT_PREFIX}.*?\n(#.*\n)*\n/, '')
-
-          # Write it back
-          new_content = options[:position] == 'before' ?  (info_block + old_content) : (old_content + "\n" + info_block)
+          # Replace the old schema info with the new schema info
+          new_content = old_content.sub(/^# #{COMPAT_PREFIX}.*?\n(#.*\n)*\n/, info_block)
+          # But, if there *was* no old schema info, we simply need to insert it
+          if new_content == old_content
+            new_content = options[:position] == 'before' ?  (info_block + old_content) : (old_content + "\n" + info_block)
+          end
 
           File.open(file_name, "wb") { |f| f.puts new_content }
           true

from annotate_models.

Can you send a pull request?

from annotate_models.

On 06/05/2011 11:00 AM, ctran wrote:

Can you send a pull request?

I've done this now. Thanks...

               - Chris

from annotate_models.

Patch applied.

from annotate_models.