[ARVADOS] updated: 32dd80c96d67add6779a9ed4c522276424e049d9

git at public.curoverse.com git at public.curoverse.com
Tue Dec 31 10:20:33 EST 2013


Summary of changes:
 .../app/controllers/application_controller.rb      |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

       via  32dd80c96d67add6779a9ed4c522276424e049d9 (commit)
      from  696e9cc71bb82411bb73148e5cdf1d68b11b709b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 32dd80c96d67add6779a9ed4c522276424e049d9
Author: Tom Clegg <tom at curoverse.com>
Date:   Tue Dec 31 07:20:13 2013 -0800

    Fix unexpected nil in @required_user_agreements. refs #1699

diff --git a/apps/workbench/app/controllers/application_controller.rb b/apps/workbench/app/controllers/application_controller.rb
index 3dc5407..f431be2 100644
--- a/apps/workbench/app/controllers/application_controller.rb
+++ b/apps/workbench/app/controllers/application_controller.rb
@@ -238,7 +238,7 @@ class ApplicationController < ActionController::Base
         if not @signed_ua_uuids.index ua.uuid
           Collection.find(ua.uuid)
         end
-      end
+      end.compact
       if @required_user_agreements.empty?
         # No agreements to sign. Perhaps we just need to ask?
         current_user.activate

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list