Skip to content
This repository has been archived by the owner on May 19, 2021. It is now read-only.

Internal error (Redmine 2.3.0) #9

Open
R-i-c-k-y opened this issue Apr 22, 2013 · 2 comments
Open

Internal error (Redmine 2.3.0) #9

R-i-c-k-y opened this issue Apr 22, 2013 · 2 comments

Comments

@R-i-c-k-y
Copy link

Started GET "/hours" for 192.168.69.136 at 2013-04-22 17:56:39 +0200
Processing by HoursController#index as HTML
Current user: RRocca (id=6)
Completed 500 Internal Server Error in 3714ms

NoMethodError (undefined method project' for nil:NilClass): plugins/redmine_hours/app/controllers/hours_controller.rb:103:inblock in get_issues'
activerecord (3.2.13) lib/active_record/relation/delegation.rb:39:in each' activerecord (3.2.13) lib/active_record/relation/delegation.rb:39:insort_by'
activerecord (3.2.13) lib/active_record/relation/delegation.rb:39:in sort_by' plugins/redmine_hours/app/controllers/hours_controller.rb:103:inget_issues'
activesupport (3.2.13) lib/active_support/callbacks.rb:484:in _run__78658070__process_action__497811606__callbacks' activesupport (3.2.13) lib/active_support/callbacks.rb:405:in__run_callback'
activesupport (3.2.13) lib/active_support/callbacks.rb:385:in _run_process_action_callbacks' activesupport (3.2.13) lib/active_support/callbacks.rb:81:inrun_callbacks'
actionpack (3.2.13) lib/abstract_controller/callbacks.rb:17:in process_action' actionpack (3.2.13) lib/action_controller/metal/rescue.rb:29:inprocess_action'
actionpack (3.2.13) lib/action_controller/metal/instrumentation.rb:30:in block in process_action' activesupport (3.2.13) lib/active_support/notifications.rb:123:inblock in instrument'
activesupport (3.2.13) lib/active_support/notifications/instrumenter.rb:20:in instrument' activesupport (3.2.13) lib/active_support/notifications.rb:123:ininstrument'
actionpack (3.2.13) lib/action_controller/metal/instrumentation.rb:29:in process_action' actionpack (3.2.13) lib/action_controller/metal/params_wrapper.rb:207:inprocess_action'
activerecord (3.2.13) lib/active_record/railties/controller_runtime.rb:18:in process_action' actionpack (3.2.13) lib/abstract_controller/base.rb:121:inprocess'
actionpack (3.2.13) lib/abstract_controller/rendering.rb:45:in process' actionpack (3.2.13) lib/action_controller/metal.rb:203:indispatch'
actionpack (3.2.13) lib/action_controller/metal/rack_delegation.rb:14:in dispatch' actionpack (3.2.13) lib/action_controller/metal.rb:246:inblock in action'
actionpack (3.2.13) lib/action_dispatch/routing/route_set.rb:73:in call' actionpack (3.2.13) lib/action_dispatch/routing/route_set.rb:73:indispatch'
actionpack (3.2.13) lib/action_dispatch/routing/route_set.rb:36:in call' journey (1.0.4) lib/journey/router.rb:68:inblock in call'
journey (1.0.4) lib/journey/router.rb:56:in each' journey (1.0.4) lib/journey/router.rb:56:incall'
actionpack (3.2.13) lib/action_dispatch/routing/route_set.rb:612:in call' rack-openid (1.3.1) lib/rack/openid.rb:98:incall'
actionpack (3.2.13) lib/action_dispatch/middleware/best_standards_support.rb:17:in call' rack (1.4.5) lib/rack/etag.rb:23:incall'
rack (1.4.5) lib/rack/conditionalget.rb:25:in call' actionpack (3.2.13) lib/action_dispatch/middleware/head.rb:14:incall'
actionpack (3.2.13) lib/action_dispatch/middleware/params_parser.rb:21:in call' actionpack (3.2.13) lib/action_dispatch/middleware/flash.rb:242:incall'
rack (1.4.5) lib/rack/session/abstract/id.rb:210:in context' rack (1.4.5) lib/rack/session/abstract/id.rb:205:incall'
actionpack (3.2.13) lib/action_dispatch/middleware/cookies.rb:341:in call' activerecord (3.2.13) lib/active_record/query_cache.rb:64:incall'
activerecord (3.2.13) lib/active_record/connection_adapters/abstract/connection_pool.rb:479:in call' actionpack (3.2.13) lib/action_dispatch/middleware/callbacks.rb:28:inblock in call'
activesupport (3.2.13) lib/active_support/callbacks.rb:405:in _run__985808177__call__220721131__callbacks' activesupport (3.2.13) lib/active_support/callbacks.rb:405:in__run_callback'
activesupport (3.2.13) lib/active_support/callbacks.rb:385:in _run_call_callbacks' activesupport (3.2.13) lib/active_support/callbacks.rb:81:inrun_callbacks'
actionpack (3.2.13) lib/action_dispatch/middleware/callbacks.rb:27:in call' actionpack (3.2.13) lib/action_dispatch/middleware/remote_ip.rb:31:incall'
actionpack (3.2.13) lib/action_dispatch/middleware/debug_exceptions.rb:16:in call' actionpack (3.2.13) lib/action_dispatch/middleware/show_exceptions.rb:56:incall'
railties (3.2.13) lib/rails/rack/logger.rb:32:in call_app' railties (3.2.13) lib/rails/rack/logger.rb:16:inblock in call'
activesupport (3.2.13) lib/active_support/tagged_logging.rb:22:in tagged' railties (3.2.13) lib/rails/rack/logger.rb:16:incall'
actionpack (3.2.13) lib/action_dispatch/middleware/request_id.rb:22:in call' rack (1.4.5) lib/rack/methodoverride.rb:21:incall'
rack (1.4.5) lib/rack/runtime.rb:17:in call' activesupport (3.2.13) lib/active_support/cache/strategy/local_cache.rb:72:incall'
rack (1.4.5) lib/rack/lock.rb:15:in call' actionpack (3.2.13) lib/action_dispatch/middleware/static.rb:63:incall'
rack-cache (1.2) lib/rack/cache/context.rb:136:in forward' rack-cache (1.2) lib/rack/cache/context.rb:245:infetch'
rack-cache (1.2) lib/rack/cache/context.rb:185:in lookup' rack-cache (1.2) lib/rack/cache/context.rb:66:incall!'
rack-cache (1.2) lib/rack/cache/context.rb:51:in call' railties (3.2.13) lib/rails/engine.rb:479:incall'
railties (3.2.13) lib/rails/application.rb:223:in call' rack (1.4.5) lib/rack/content_length.rb:14:incall'
railties (3.2.13) lib/rails/rack/log_tailer.rb:17:in call' rack (1.4.5) lib/rack/handler/webrick.rb:59:inservice'
/usr/local/rvm/rubies/ruby-2.0.0-p0/lib/ruby/2.0.0/webrick/httpserver.rb:138:in service' /usr/local/rvm/rubies/ruby-2.0.0-p0/lib/ruby/2.0.0/webrick/httpserver.rb:94:inrun'
/usr/local/rvm/rubies/ruby-2.0.0-p0/lib/ruby/2.0.0/webrick/server.rb:295:in `block in start_thread'

@roelarents
Copy link

This continues on the problem in #1. There were more lines which assumed time_entry always has an issue associated. See the patch below.

# This patch file was generated by NetBeans IDE
# It uses platform neutral UTF-8 encoding and \n newlines.
--- HEAD
+++ Modified In Working Tree
@@ -100,17 +100,24 @@

     if @week_issue_matrix.empty?
       @week_issue_matrix = {}
-      last_week_time_entries = TimeEntry.for_user(@user).spent_between(@week_start-7, @week_end-7).sort_by{|te| te.issue.project.name}.sort_by{|te| te.issue.subject }
+           last_week_time_entries = TimeEntry.for_user(@user).spent_between(@week_start-7, @week_end-7).sort_by{|te| te.project.name}
+#          if te.issue
+#              last_week_time_entries.sort_by{|te| te.issue.subject }
+#          end
       last_week_time_entries.each do |te|
-        @week_issue_matrix["#{te.issue.project.name} - #{te.issue.subject} - #{te.activity.name}"] ||= {:issue_id => te.issue_id,
+               key = "#{te.project.name}" + (te.issue ? " - #{te.issue.subject}" : "") +  " - #{te.activity.name}"
+        @week_issue_matrix[key] ||= {:issue_id => te.issue_id,
                                                                                                       :activity_id => te.activity_id,
-                                                                                                      :project_id => te.issue.project.id,
-                                                                                                      :project_name => te.issue.project.name,
-                                                                                                      :issue_text => te.issue.to_s,
-                                                                                                      :activity_name => te.activity.name
+                   :project_id => te.project.id,
+                   :project_name => te.project.name,
+                   :issue_text => te.issue.try(:to_s),
+                   :activity_name => te.activity.name,
+                   :issue_class => ''
                                                                                                      }
-        @week_issue_matrix["#{te.issue.project.name} - #{te.issue.subject} - #{te.activity.name}"][:issue_class] ||= te.issue.closed? ? 'issue closed' : 'issue'
+               if te.issue
+                   @week_issue_matrix[key][:issue_class] ||= te.issue.closed? ? 'issue closed' : 'issue'
       end
+      end
       @week_issue_matrix = @week_issue_matrix.sort
     end

@zosiu
Copy link
Contributor

zosiu commented Jul 15, 2013

Now that the problem is clear, I'll fix it as soon as I can.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants