Actions
Patch #24382
closedMore readable regex for parse_redmine_links
Status:
Closed
Priority:
Normal
Assignee:
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:
0%
Estimated time:
Description
In method parse_redmine_links used very long regex that impossible to understand with first look.
Added names for capture groups.
Files
Updated by Jean-Philippe Lang almost 8 years ago
- Status changed from New to Needs feedback
Breaks a test:
ruby test\unit\helpers\application_helper_test.rb DL is deprecated, please use Fiddle Run options: --seed 7252 # Running: .............................................................E.................. .................... Finished in 2.519144s, 39.6960 runs/s, 154.0206 assertions/s. 1) Error: ApplicationHelperTest#test_parse_redmine_links_should_handle_a_tag_without_attri butes: NoMethodError: undefined method `+' for nil:NilClass app/helpers/application_helper.rb:943:in `block in parse_redmine_links' app/helpers/application_helper.rb:800:in `gsub!' app/helpers/application_helper.rb:800:in `parse_redmine_links' test/unit/helpers/application_helper_test.rb:1279:in `test_parse_redmine_lin ks_should_handle_a_tag_without_attributes' 100 runs, 388 assertions, 0 failures, 1 errors, 0 skips
Updated by Dmitry Lisichkin almost 8 years ago
Sorry, wrong place for "tag_content" capture group.
Fixed.
Updated by Go MAEDA almost 8 years ago
- Category changed from Text formatting to Code cleanup/refactoring
Updated by Jean-Philippe Lang almost 8 years ago
- Subject changed from simplified regex for parse_redmine_links to More readable regex for parse_redmine_links
- Status changed from Needs feedback to Closed
- Assignee set to Jean-Philippe Lang
Committed, thanks for fixing the patch.
Updated by Mischa The Evil almost 8 years ago
Dmitry, that's a nice refactoring making the method a lot more readable and thus easier to understand.
Actions