Those two files were still using the constant instead of the method, …

…which determines if external submits are allowed or not. See review 326.
viviensuen · Jan 11, 2010 · 8fca691 · 8fca691
1 parent ba31d27
commit 8fca691
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/app/controllers/submissions_controller.rb b/app/controllers/submissions_controller.rb
@@ -224,13 +224,13 @@ def index
 
   # controller handles transactional submission of files
   def update_files
-    if REPOSITORY_EXTERNAL_SUBMITS_ONLY
-      raise "MarkUs is only accepting external submits"
-    end
     assignment_id = params[:id]
     assignment = Assignment.find(assignment_id)
     path = params[:path] || '/'
     grouping = current_user.accepted_grouping_for(assignment_id)
+    if grouping.group.repository_external_commits_only?
+      raise "MarkUs is only accepting external submits"
+    end
     if !grouping.is_valid?
       redirect_to :action => :file_manager, :id => assignment_id
       return

diff --git a/app/views/submissions/file_manager_populate.rjs b/app/views/submissions/file_manager_populate.rjs
@@ -1,4 +1,4 @@
-if REPOSITORY_EXTERNAL_SUBMITS_ONLY
+if @grouping.group.repository_external_commits_only?
   page.replace_html 'directories_tbody', :partial => 'submissions/table_row/exit_directory'
 end
 page.call "populate", @table_rows.to_json