backlog 0.10.0 → 0.10.1

data/History.txt

@@ -1,3 +1,21 @@
+== 0.10.1 2007-09-23
+
+=== Features
+
+=== Fixes
+
+* Fixed bug in "remember me" login when running with JRuby
+* Fixed bug when moving a task to next sprint on mysql
+* Made user message more visible, especially register messages.
+* Added explanatory text in log for reading global config.
+* Excluded "pkg" from war file to avoid duplicates and save space.
+* Removed "bouncycastle" from WAR to get it to run on Tomcat.
+* Merged migration steps 001 and 002 to get them to run on SQLite.
+* Fixed reading of application version when running with JRuby.
+* Excluded "log" and "WEB-INF" directories from gem to save space.
+* Added "rails" to gem dependencies since Rails is not included anymore.
+* Bumped dependent version of rmagick from 1.15.9 to 1.15.10 to get the newest fixes.
+
 == 0.10.0 2007-09-21
 
 === Features

data/Rakefile

@@ -21,7 +21,7 @@ Hoe.new("backlog", APP::VERSION) do |p|
   p.remote_rdoc_dir = '' # Release to root
   p.changes = p.paragraphs_of('History.txt', 0..4).join("\n\n")
   p.rdoc_pattern = /^(app\/(controllers|helpers|models)|lib|bin)|txt$/
-  p.clean_globs = ['doc', 'log']
+  p.clean_globs = ['doc', 'log/*']
   p.spec_extras = {
     :files => Dir['**/*'].reject{|file_name| file_name =~ /^(doc|log|pkg|tmp|WEB-INF)/},
     :rdoc_options => ['--inline-source'],
@@ -30,16 +30,16 @@ Hoe.new("backlog", APP::VERSION) do |p|
   }
   p.need_zip = true
   p.url = 'http://rubyforge.org/projects/backlog/'
-  p.extra_deps = [['gruff', '>= 0.2.8'], ['rmagick', '>= 1.15.9'], ['postgres', '>= 0.7.1']]
+  p.extra_deps = [['rails', '>= 1.2.3'], ['gruff', '>= 0.2.8'], ['rmagick', '>= 1.15.10'], ['postgres', '>= 0.7.1']]
   p.rsync_args = "-av --delete --exclude=wiki*"
 end
 
 task :release_all do
   ENV['VERSION'] = APP::VERSION
-  #Rake::Task[:publish_docs].invoke
+  Rake::Task[:publish_docs].invoke
   Rake::Task[:release].invoke
   Rake::Task[:release_war].invoke
-  #Rake::Task[:post_news].invoke
+  Rake::Task[:post_news].invoke
 end
 
 task :release_war do

data/app/controllers/user_controller.rb

@@ -157,6 +157,12 @@ class UserController < ApplicationController
     case request.method
     when :get
       @user = User.new
+      if cookie = cookies['autologin']
+        cookie_value = cookie[0]
+        if remembered_user = User.find(:first, :conditions => ["id = ?", cookie_value])
+          @user.login = remembered_user.login
+        end
+      end
       render
       return true
     end

data/app/models/task.rb

@@ -165,6 +165,7 @@ class Task < ActiveRecord::Base
         new_task.period = new_period
         new_task.description = self.description
         new_task.initial_estimate = self.initial_estimate
+        new_task.save!
         new_task.insert_at 1
         new_task.position = 1
         new_task.save!

data/app/views/layouts/_left_top.rhtml

@@ -18,6 +18,21 @@
         <%=flash[:notice]%>
 	  </div>
 	  <% end %>
+      <% if flash['notice'] %>
+      <div id="notice">
+        <%=flash['notice']%>
+	  </div>
+	  <% end %>
+      <% if flash[:notice] %>
+      <div id="message">
+        <%=flash[:message]%>
+	  </div>
+	  <% end %>
+      <% if flash['message'] %>
+      <div id="message">
+        <%=flash['message']%>
+	  </div>
+	  <% end %>
       <%= error_messages_for :backlog %>
       <%= error_messages_for :estimate %>
       <%= error_messages_for :group %>

data/app/views/user/login.rhtml

@@ -1,14 +1,12 @@
 <% @page_title = l(:please_login) %>
 
 <div title="<%= title_helper %>" class="form">
-  <%= head_helper ' ' %>
-
   <div class="form-padding">
     <%= start_form_tag_helper  %>
       <table>
         <tr class="two_columns">
           <td class="prompt"><label>Login:</label></td>
-          <td class="value"><%= text_field 'user', 'login', :size => 30, :value => cookies['autologin'] && User.find(cookies['autologin'][0]).login %></td>
+          <td class="value"><%= text_field 'user', 'login', :size => 30 %></td>
         </tr>
         <tr class="two_columns">
           <td class="prompt"><label>Password:</label></td>

data/config/environment.rb

@@ -16,16 +16,24 @@ require File.join(File.dirname(__FILE__), 'boot')
 # The following section has been added to load database settings from system config.
 APP_CONFIG_FILE = Config::CONFIG["sysconfdir"] + '/backlog.conf'
 if File.exists? APP_CONFIG_FILE
+  puts "Found global config file (#{APP_CONFIG_FILE})"
   module Rails
     class Configuration
       def database_configuration
         db_conf = YAML::load(ERB.new(IO.read(database_configuration_file)).result)
         app_conf = YAML::load(ERB.new(IO.read(APP_CONFIG_FILE)).result)
-        db_conf['production'] = app_conf['database'] if app_conf['database']
+        if db_settings = app_conf['database']
+          puts "Found global database connection settings: #{db_settings.inspect}"
+          db_conf['production'] = db_settings
+        else
+          puts "Did not find global database connection settings: #{app_conf.inspect}"
+        end
         db_conf
       end
     end
   end
+else
+  puts "Global config file not found (#{APP_CONFIG_FILE})"
 end
 
 if RUBY_PLATFORM =~ /java/

data/db/migrate/001_create_tasks.rb

@@ -1,15 +1,22 @@
 class CreateTasks < ActiveRecord::Migration
   def self.up
+    create_table :backlogs do |table|
+      table.column :name,  :string, :limit => 32, :null => false
+    end
+  
     create_table :tasks do |table|
       table.column :position,  :integer, :null => true
       table.column :description, :string, :limit => 80, :null => false
       table.column :assigned_to, :integer
       table.column :estimate, :time
       table.column :finished_at, :datetime
+      table.column :backlog_id, :integer, :null => false
     end
   end
 
   def self.down
     drop_table :tasks
+    drop_table :backlogs
   end
+
 end

data/public/stylesheets/backlog.css

@@ -25,18 +25,21 @@ th.hours {text-align: right;}
 }
 
 #notice {
+  width: 320px;
   border: 2px solid green;
   padding: 7px;
   padding-bottom: 12px;
   margin: 1em;
   margin-left: 0px;
-  background: transparent;
+  color: #080;
+  background: white;
   float: right;
+  font-size: 15px;
 }
 
 #errorExplanation {
   border: 2px solid red;
-  padding: 7px;
+  padding: 7px; 
   padding-bottom: 12px;
   margin: 1em;
   margin-left: 0px;

data/test/functional/tasks_controller_test.rb

@@ -2,12 +2,7 @@ require File.dirname(__FILE__) + '/../test_helper'
 require 'tasks_controller'
 
 # Re-raise errors caught by the controller.
-class TasksController
-  def rescue_action(e)
-    puts e.backtrace.join("\n")
-    raise(e)
-  end
-end
+class TasksController; def rescue_action(e) raise e end; end
 
 class TasksControllerTest < Test::Unit::TestCase
   fixtures :parties, :users, :groups, :groups_users, :backlogs, :periods, :tasks, :task_files, :works, :estimates
@@ -19,6 +14,10 @@ class TasksControllerTest < Test::Unit::TestCase
     @request.session[:user_id] = 1000001
   end
   
+  def teardown
+    assert_sequences
+  end
+  
   def test_index
     get :index
     assert_redirected_to :controller => 'backlogs'
@@ -200,7 +199,24 @@ class TasksControllerTest < Test::Unit::TestCase
     new_task = Task.find_by_period_id_and_previous_task_id(periods(:future).id, before.id)
     assert_nil new_task.finished_at
     assert_nil new_task.resolution
-    tasks = Task.find_all_by_period_id(3)
+    assert_equal 1, new_task.position
+  end
+  
+  def test_move_to_next_period_from_past
+    task = tasks(:first)
+    before = Task.find(task.id)
+    
+    post :move_to_next_period, :id => task.id
+    
+    assert_response :redirect
+    assert_redirected_to :controller => 'periods', :action => :show, :id => periods(:past)
+    
+    after = Task.find(task.id)
+    assert_equal task.period_id, after.period_id
+    
+    new_task = Task.find_by_period_id_and_previous_task_id(periods(:active).id, task.id)
+    assert_nil new_task.finished_at
+    assert_nil new_task.resolution
     assert_equal 1, new_task.position
   end
   

metadata

@@ -3,8 +3,8 @@ rubygems_version: 0.9.4
 specification_version: 1
 name: backlog
 version: !ruby/object:Gem::Version 
-  version: 0.10.0
-date: 2007-09-22 00:00:00 +02:00
+  version: 0.10.1
+date: 2007-09-23 00:00:00 +02:00
 summary: Application to aid collecting, processing, organizing, reviewing and doing tasks.
 require_paths: 
 - lib
@@ -130,7 +130,6 @@ files:
 - db/migrate/003_add_estimation_points.rb
 - db/migrate/007_add_task_created_at.rb
 - db/migrate/004_add_period.rb
-- db/migrate/002_create_backlogs.rb
 - db/migrate/016_add_invoicable_flag.rb
 - db/migrate/018_create_groups.rb
 - db/migrate/005_add_field_work_started_at.rb
@@ -448,6 +447,15 @@ requirements:
 - ImageMagick or GraphicsMagick
 - PostgreSQL
 dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: rails
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Version::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: 1.2.3
+    version: 
 - !ruby/object:Gem::Dependency 
   name: gruff
   version_requirement: 
@@ -464,7 +472,7 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 1.15.9
+        version: 1.15.10
     version: 
 - !ruby/object:Gem::Dependency 
   name: postgres

data/db/migrate/002_create_backlogs.rb

@@ -1,31 +0,0 @@
-class CreateBacklogs < ActiveRecord::Migration
-  def self.up
-    create_table :backlogs do |table|
-      table.column :name,  :string, :limit => 32, :null => false
-    end
- 
-    if Task.count > 0
-      default_backlog = Backlog.create! :name => 'Default Backlog'
-      add_column :tasks, :backlog_id, :integer, :null => false, :default => default_backlog.id
-      
-      # TODO(uwe): This line is only needed because Rails 1.2 cannot set default to NULL
-      change_column_default :tasks, :backlog_id, 0
-      
-      change_column_default :tasks, :backlog_id, nil
-    else
-      add_column :tasks, :backlog_id, :integer, :null => false
-    end
-  end
-  
-  def self.down
-    remove_column :tasks, :backlog_id
-    drop_table :backlogs
-  end
-  
-  class Task < ActiveRecord::Base
-  end
-  
-  class Backlog < ActiveRecord::Base
-  end
-  
-end