hdgarrood / redmine_release_notes

A plugin to add release notes to Redmine
GNU General Public License v3.0
73 stars 47 forks source link

Error deleting Release Note #104

Open mbuchner opened 10 years ago

mbuchner commented 10 years ago

We installed your plugin in Redmine:

Environment: Redmine version 2.2.3.stable Ruby version 1.8.7 (x86_64-linux) Rails version 3.2.12 Environment production Database adapter MySQL Redmine plugins: due_date_reminder 0.3.2 projects_table 0.0.4 redmine_inline_attach_screenshot 0.4.2 redmine_monitoring_controlling 0.1.1 redmine_mylyn_connector 2.8.2.stable redmine_release_notes 1.3.1

If we try to delete a release note we get asked if we really want to delete it and then get forwarded to an error page! The following error is written to the log file:

NoMethodError (undefined method release_notes_custom_value' for nil:NilClass): plugins/redmine_release_notes/app/controllers/release_notes_controller.rb:115:inupdate_custom_field' plugins/redmine_release_notes/app/controllers/release_notes_controller.rb:70:in destroy' actionpack (3.2.12) lib/action_controller/metal/implicit_render.rb:4:insend_action' actionpack (3.2.12) lib/action_controller/metal/implicit_render.rb:4:in send_action' actionpack (3.2.12) lib/abstract_controller/base.rb:167:inprocess_action' actionpack (3.2.12) lib/action_controller/metal/rendering.rb:10:in process_action' actionpack (3.2.12) lib/abstract_controller/callbacks.rb:18:inprocess_action' activesupport (3.2.12) lib/active_support/callbacks.rb:458:in _run__722325261__process_action__556031166__callbacks' activesupport (3.2.12) lib/active_support/callbacks.rb:405:insend' activesupport (3.2.12) lib/active_support/callbacks.rb:405:in __run_callback' activesupport (3.2.12) lib/active_support/callbacks.rb:385:in_run_process_action_callbacks' activesupport (3.2.12) lib/active_support/callbacks.rb:81:in send' activesupport (3.2.12) lib/active_support/callbacks.rb:81:inrun_callbacks' actionpack (3.2.12) lib/abstract_controller/callbacks.rb:17:in process_action' actionpack (3.2.12) lib/action_controller/metal/rescue.rb:29:inprocess_action' actionpack (3.2.12) lib/action_controller/metal/instrumentation.rb:30:in process_action' activesupport (3.2.12) lib/active_support/notifications.rb:123:ininstrument' activesupport (3.2.12) lib/active_support/notifications/instrumenter.rb:20:in instrument' activesupport (3.2.12) lib/active_support/notifications.rb:123:ininstrument' actionpack (3.2.12) lib/action_controller/metal/instrumentation.rb:29:in process_action' actionpack (3.2.12) lib/action_controller/metal/params_wrapper.rb:207:inprocess_action' activerecord (3.2.12) lib/active_record/railties/controller_runtime.rb:18:in process_action' actionpack (3.2.12) lib/abstract_controller/base.rb:121:inprocess' actionpack (3.2.12) lib/abstract_controller/rendering.rb:45:in process' actionpack (3.2.12) lib/action_controller/metal.rb:203:indispatch' actionpack (3.2.12) lib/action_controller/metal/rack_delegation.rb:14:in dispatch' actionpack (3.2.12) lib/action_controller/metal.rb:246:inaction' actionpack (3.2.12) lib/action_dispatch/routing/route_set.rb:73:in call' actionpack (3.2.12) lib/action_dispatch/routing/route_set.rb:73:indispatch' actionpack (3.2.12) lib/action_dispatch/routing/route_set.rb:36:in call' journey (1.0.4) lib/journey/router.rb:68:incall' journey (1.0.4) lib/journey/router.rb:56:in each' journey (1.0.4) lib/journey/router.rb:56:incall' actionpack (3.2.12) lib/action_dispatch/routing/route_set.rb:601:in call' rack-openid (1.3.1) lib/rack/openid.rb:98:incall' actionpack (3.2.12) 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:35:in call' actionpack (3.2.12) lib/action_dispatch/middleware/head.rb:14:incall' actionpack (3.2.12) lib/action_dispatch/middleware/params_parser.rb:21:in call' actionpack (3.2.12) 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.12) lib/action_dispatch/middleware/cookies.rb:341:in call' activerecord (3.2.12) lib/active_record/query_cache.rb:64:incall' activerecord (3.2.12) lib/active_record/connection_adapters/abstract/connection_pool.rb:479:in call' actionpack (3.2.12) lib/action_dispatch/middleware/callbacks.rb:28:incall' activesupport (3.2.12) lib/active_support/callbacks.rb:405:in _run__581174207__call__4__callbacks' activesupport (3.2.12) lib/active_support/callbacks.rb:405:insend' activesupport (3.2.12) lib/active_support/callbacks.rb:405:in __run_callback' activesupport (3.2.12) lib/active_support/callbacks.rb:385:in_run_call_callbacks' activesupport (3.2.12) lib/active_support/callbacks.rb:81:in send' activesupport (3.2.12) lib/active_support/callbacks.rb:81:inrun_callbacks' actionpack (3.2.12) lib/action_dispatch/middleware/callbacks.rb:27:in call' actionpack (3.2.12) lib/action_dispatch/middleware/remote_ip.rb:31:incall' actionpack (3.2.12) lib/action_dispatch/middleware/debug_exceptions.rb:16:in call' actionpack (3.2.12) lib/action_dispatch/middleware/show_exceptions.rb:56:incall' railties (3.2.12) lib/rails/rack/logger.rb:32:in call_app' railties (3.2.12) lib/rails/rack/logger.rb:16:incall' activesupport (3.2.12) lib/active_support/tagged_logging.rb:22:in tagged' railties (3.2.12) lib/rails/rack/logger.rb:16:incall' actionpack (3.2.12) 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.12) lib/active_support/cache/strategy/local_cache.rb:72:incall' rack (1.4.5) lib/rack/lock.rb:15:in call' actionpack (3.2.12) lib/action_dispatch/middleware/static.rb:62:incall' rack-cache (1.2) lib/rack/cache/context.rb:136:in forward' rack-cache (1.2) lib/rack/cache/context.rb:143:inpass' rack-cache (1.2) lib/rack/cache/context.rb:155:in invalidate' rack-cache (1.2) lib/rack/cache/context.rb:71:incall!' rack-cache (1.2) lib/rack/cache/context.rb:51:in call' railties (3.2.12) lib/rails/engine.rb:479:incall' railties (3.2.12) lib/rails/application.rb:223:in call' railties (3.2.12) lib/rails/railtie/configurable.rb:30:insend' railties (3.2.12) lib/rails/railtie/configurable.rb:30:in method_missing' /usr/lib/ruby/1.8/phusion_passenger/rack/request_handler.rb:92:inprocess_request' /usr/lib/ruby/1.8/phusion_passenger/abstract_request_handler.rb:207:in main_loop' /usr/lib/ruby/1.8/phusion_passenger/rack/application_spawner.rb:118:inrun' /usr/lib/ruby/1.8/phusion_passenger/rack/application_spawner.rb:65:in spawn_application' /usr/lib/ruby/1.8/phusion_passenger/utils.rb:184:insafe_fork' /usr/lib/ruby/1.8/phusion_passenger/rack/application_spawner.rb:58:in spawn_application' /usr/lib/ruby/1.8/phusion_passenger/rack/application_spawner.rb:41:inspawn_application' /usr/lib/ruby/1.8/phusion_passenger/spawn_manager.rb:159:in spawn_application' /usr/lib/ruby/1.8/phusion_passenger/spawn_manager.rb:287:inhandle_spawn_application' /usr/lib/ruby/1.8/phusion_passenger/abstract_server.rb:352:in __send__' /usr/lib/ruby/1.8/phusion_passenger/abstract_server.rb:352:inmain_loop' /usr/lib/ruby/1.8/phusion_passenger/abstract_server.rb:196:in`start_synchronously' /usr/lib/phusion_passenger/passenger-spawn-server:61

We created a custom list file -> see screen shot

image

mbuchner commented 10 years ago

Any update on this one?

hdgarrood commented 10 years ago

I'm sorry, I have no time to look at this at the moment.

Americas commented 9 years ago

Hi, I ran into this same problem, and I have fixed it.

Go to _{redmine_folder}/plugins/redmine_release_notes/app/controllers/release_notescontroller.rb and change the function destroy with this one:

  def destroy
    release_note = ReleaseNote.find(params[:id])
    @issue = Issue.find(release_note.issue_id)

    update_custom_field(false)

    release_note.destroy

    flash[:notice] = l(:notice_successful_delete)
    redirect_to @issue
  end