Skip to content
Snippets Groups Projects

Render submodule bugfix

Merged gitlab-qa-bot requested to merge github/fork/AlexDenisov/submodule_rendering_bug into master

Created by: AlexDenisov

Have this error when submodule url is nil.

ActionView::Template::Error (can't convert nil into String):
    31:           %td
    32:           %td
    33:
    34:       = render_tree(tree.contents)
    35:
    36:     - if tree.readme
    37:       = render "tree/readme", readme: tree.readme
  app/helpers/tree_helper.rb:18:in `+'
  app/helpers/tree_helper.rb:18:in `block in render_tree'
  app/helpers/tree_helper.rb:15:in `each'
  app/helpers/tree_helper.rb:15:in `render_tree'
  app/views/tree/_tree.html.haml:34:in `_app_views_tree__tree_html_haml__1071874968_86634150'
  app/views/tree/show.html.haml:3:in `_app_views_tree_show_html_haml___680544721_79735610'
  app/controllers/tree_controller.rb:17:in `show'

Render empty string instead of nil fix this error.

P.S. right now I have no time to add tests on this case.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Created by: AlexDenisov

    Seems that fail is unrelated...

    By Administrator on 2013-01-21T10:12:56 (imported from GitLab project)

    By Administrator on 2013-01-21T10:12:56 (imported from GitLab)

  • Created by: dzaporozhets

    Thank you

    By Administrator on 2013-01-21T10:23:52 (imported from GitLab project)

    By Administrator on 2013-01-21T10:23:52 (imported from GitLab)

Please register or sign in to reply
Loading