Description:
rename_problem fix
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r234:45474cc69e4c - - 1 file changed: 2 inserted, 2 deleted
@@ -5,13 +5,13 | |||||
|
5 | def config |
|
5 | def config |
|
6 | Grader::Configuration.get_instance |
|
6 | Grader::Configuration.get_instance |
|
7 | end |
|
7 | end |
|
8 |
|
8 | ||
|
9 | def rename_problem(old_problem_name, new_problem_name) |
|
9 | def rename_problem(old_problem_name, new_problem_name) |
|
10 |
|
10 | ||
|
11 |
- |
|
11 | + unless valid_problem_name(new_problem_name) |
|
12 | puts "Bad new problem name: #{new_problem_name}" |
|
12 | puts "Bad new problem name: #{new_problem_name}" |
|
13 | return |
|
13 | return |
|
14 | end |
|
14 | end |
|
15 |
|
15 | ||
|
16 | problem = Problem.find_by_name(old_problem_name) |
|
16 | problem = Problem.find_by_name(old_problem_name) |
|
17 | if problem==nil |
|
17 | if problem==nil |
@@ -57,13 +57,13 | |||||
|
57 | end |
|
57 | end |
|
58 |
|
58 | ||
|
59 | def valid_problem_name(name) |
|
59 | def valid_problem_name(name) |
|
60 | if name.length==0: |
|
60 | if name.length==0: |
|
61 | return false |
|
61 | return false |
|
62 | else |
|
62 | else |
|
63 |
- return |
|
63 | + return (/^[a-zA-Z0-9_\-]+$/ === name) |
|
64 | end |
|
64 | end |
|
65 | end |
|
65 | end |
|
66 |
|
66 | ||
|
67 | if (ARGV.length!=2) |
|
67 | if (ARGV.length!=2) |
|
68 | usage |
|
68 | usage |
|
69 | exit(0) |
|
69 | exit(0) |
You need to be logged in to leave comments.
Login now