cafe-grader-web Commit - r758:d23edd5adc06

cafe-grader-web

Commit `r758:d23edd5adc06` public

parent | child

Description:

revert simple_form_for to form_tag

Commit status:

[Not Reviewed]

References:

algo

Diff options:

| | | |

Comments:

0 Commit comments 0 Inline Comments

Unresolved TODOs:

There are no unresolved TODOs

Add another comment

r758:d23edd5adc06 - Fri, 12 Jul 2019 12:43:27 - 5 files changed: 11 inserted, 30 deleted

Gemfile.lock ¶ -19

                   coffee-script (2.4.1)
                     coffee-script-source
                     execjs
                   coffee-script-source (1.12.2)
                   concurrent-ruby (1.1.5)
                   crass (1.0.4)
-            -     diff-lcs (1.3)
                   dynamic_form (1.1.4)
                   erubi (1.8.0)
                   erubis (2.7.0)
                   execjs (2.7.0)
                   ffi (1.11.1)
                   fuzzy-string-match (1.0.1)
                   rb-fsevent (0.10.3)
                   rb-inotify (0.10.0)
                     ffi (~> 1.0)
                   rdiscount (2.2.0.1)
                   regexp_parser (1.5.1)
                   rouge (3.3.0)
-            -     rspec-core (3.8.2)
-            -       rspec-support (~> 3.8.0)
-            -     rspec-expectations (3.8.4)
-            -       diff-lcs (>= 1.2.0, < 2.0)
-            -       rspec-support (~> 3.8.0)
-            -     rspec-mocks (3.8.1)
-            -       diff-lcs (>= 1.2.0, < 2.0)
-            -       rspec-support (~> 3.8.0)
-            -     rspec-rails (3.8.2)
-            -       actionpack (>= 3.0)
-            -       activesupport (>= 3.0)
-            -       railties (>= 3.0)
-            -       rspec-core (~> 3.8.0)
-            -       rspec-expectations (~> 3.8.0)
-            -       rspec-mocks (~> 3.8.0)
-            -       rspec-support (~> 3.8.0)
-            -     rspec-support (3.8.2)
                   ruby-progressbar (1.10.0)
                   ruby_dep (1.5.0)
                   ruby_parser (3.13.1)
                     sexp_processor (~> 4.9)
                   rubyzip (1.2.3)
                   sass (3.7.4)
                 puma
                 rails (~> 5.2)
                 rails-controller-testing
                 rails_bootstrap_sortable
                 rdiscount
                 rouge
-            -   rspec-rails
                 sassc-rails
                 select2-rails
                 selenium-webdriver
                 simple_form
                 spring
                 spring-watcher-listen (~> 2.0.0)

app/models/problem.rb ¶ +3 -3

                   available.order(date_added: :desc).order(:name)
                   #Problem.available.all(:order => "date_added DESC, name ASC")
                 end
                 def self.create_from_import_form_params(params, old_problem=nil)
                   org_problem = old_problem || Problem.new
-            -     import_params, problem = Problem.extract_params_and_check(params,
+            +     import_params, problem = Problem.extract_params_and_check(params,
                                                                             org_problem)
                   if !problem.errors.empty?
                     return problem, 'Error importing'
                   end
                   end
                   import_to_db = params.has_key? :import_to_db
                   importer = TestdataImporter.new(problem)
-            -     if not importer.import_from_file(import_params[:file],
-            -                                      import_params[:time_limit],
+            +     if not importer.import_from_file(import_params[:file],
+            +                                      import_params[:time_limit],
                                                    import_params[:memory_limit],
                                                    import_params[:checker_name],
                                                    import_to_db)
                     problem.errors.add(:base,'Import error.')
                   end

app/views/problems/import.html.haml ¶ +1 -1

               %p= link_to '[Back to problem list]', problems_path
               - if @problem and @problem.errors
                 =error_messages_for 'problem'
-            - = simple_form_for :problem, url: do_import_problems_path, :multipart => true  do |f|
+            + = form_tag do_import_problems_path, :multipart => true  do |f|
                 .submitbox
                   %table
                     %tr
                       %td Name:
                       %td= text_field_tag 'name'

db/seeds.rb ¶ +6 -6

                 seed_roles
                 seed_root
               end
               def seed_more_languages
                 Language.delete_all
-            -   Language.create( name: 'c', pretty_name: 'C', ext: 'c', common_ext: 'c' )
-            -   Language.create( name: 'cpp', pretty_name: 'C++', ext: 'cpp', common_ext: 'cpp,cc' )
-            -   Language.create( name: 'pas', pretty_name: 'Pascal', ext: 'pas', common_ext: 'pas' )
-            -   Language.create( name: 'ruby', pretty_name: 'Ruby', ext: 'rb', common_ext: 'rb' )
-            -   Language.create( name: 'python', pretty_name: 'Python', ext: 'py', common_ext: 'py' )
-            -   Language.create( name: 'java', pretty_name: 'Java', ext: 'java', common_ext: 'java' )
+            +   Language.find_or_create_by( name: 'c', pretty_name: 'C', ext: 'c', common_ext: 'c' )
+            +   Language.find_or_create_by( name: 'cpp', pretty_name: 'C++', ext: 'cpp', common_ext: 'cpp,cc' )
+            +   Language.find_or_create_by( name: 'pas', pretty_name: 'Pascal', ext: 'pas', common_ext: 'pas' )
+            +   Language.find_or_create_by( name: 'ruby', pretty_name: 'Ruby', ext: 'rb', common_ext: 'rb' )
+            +   Language.find_or_create_by( name: 'python', pretty_name: 'Python', ext: 'py', common_ext: 'py' )
+            +   Language.find_or_create_by( name: 'java', pretty_name: 'Java', ext: 'java', common_ext: 'java' )
               end
               seed_config
               seed_users_and_roles
               seed_more_languages

lib/testdata_importer.rb ¶ +1 -1

                 def save_testdata_file(tempfile)
                   ext = TestdataImporter.long_ext(tempfile.original_filename)
                   testdata_filename = File.join(Dir.tmpdir,"#{@problem.name}#{ext}")
                   return nil if tempfile==""
+            -
+            +
                   if tempfile.instance_of?(Tempfile)
                     tempfile.close
                     FileUtils.move(tempfile.path,testdata_filename)
                   else
                     File.open(testdata_filename, "wb") do |f|
                       f.write(tempfile.read)

Write
Preview

You need to be logged in to leave comments. Login now

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository permissions settings

Sign in to your account

Author

r758:d23edd5adc06 - Fri, 12 Jul 2019 12:43:27 - 5 files changed: 11 inserted, 30 deleted

Sign in to your account

Commit r758:d23edd5adc06 public

Author

r758:d23edd5adc06 - Fri, 12 Jul 2019 12:43:27 - 5 files changed: 11 inserted, 30 deleted

Commit `r758:d23edd5adc06` public