I am Getting the error undefined method `empty?' for nil:NilClass

Question

I am working on a super simple newsletter application, and I'm confused about this error. Why is there a nil class? I am only asking it to render, so why cant I put a redirect_to where the render call is?.

 <% if admin_signed_in? %>

   <p id="notice"><%= notice %></p>

   <h1>Subscribedusers</h1>

   <table>
 <thead>
  <tr>
    <th>Email</th>
    <th colspan="3"></th>
  </tr>
</thead>

<tbody>
<% @subscribedusers.each do |subscribeduser| %>
  <tr>
    <td><%= subscribeduser.email %></td>
    <td><%= link_to 'Show', subscribeduser %></td>
    <td><%= link_to 'Edit', edit_subscribeduser_path(subscribeduser) %></td>
    <td><%= link_to 'Destroy', subscribeduser, method: :delete, data: { confirm: 'Are you sure?' } %></td>
  </tr>
  <% end %>
</tbody>
</table>

<br>

<%= link_to 'New Subscribeduser', new_subscribeduser_path %>

<% else %>

<%= render '/' %>

<% end %>

Why does this part of the code <%= render '/' %> trigger an undefined method 'empty?' for nil:NilClass error?

What are you trying to achieve with render '/'?

Nic Nilov
– Nic Nilov

2016-07-28 22:30:10 +00:00
Commented Jul 28, 2016 at 22:30 — Nic Nilov
– Nic Nilov, Commented Jul 28, 2016 at 22:30
i want it to return to the homepage

Dotun Longe
– Dotun Longe

2016-07-29 14:00:32 +00:00
Commented Jul 29, 2016 at 14:00 — Dotun Longe
– Dotun Longe, Commented Jul 29, 2016 at 14:00

Nic Nilov · Accepted Answer · 2016-07-29 20:20:50Z

2

Since your desire is to return the user to the home page, instead of

render '/'

you should use

redirect_to root_path

The difference is render prepares the output to be displayed as the result of the current request and redirect_to commands user's browser to make a new request at specified url.

While it can be possible to render the contents of your home page in an arbitrary action, this is rarely desirable. One downside is the page url would not automatically update to your site's root in the browser's address line.

As a side note, render '/' is not a correct syntax. render generally accepts a hash of options and not a string.

answered Jul 29, 2016 at 20:20

Nic Nilov

5,1652 gold badges24 silver badges38 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

PS. · Accepted Answer · 2016-07-28 23:00:55Z

0

Why do you have an <% else %> clause without any conditional loops? And what are you trying to call with <% render '/' %>? Are you trying to call the index page? If so, you can specify by saying <% render "index" %> .

answered Jul 28, 2016 at 23:00

PS.

3431 gold badge3 silver badges8 bronze badges

1 Comment

Dotun Longe Over a year ago

Sorry, there is an if condition, i did not include it in the code properly, still get the same error though

Rajnik · Accepted Answer · 2016-08-01 05:05:44Z

0

So you don't need if and else block in view side for page redirect to root path. From your controller's whatever action you need following code.

if admin_signed_in? redirect_to root_path end

also remove if and else block from view.

answered Aug 1, 2016 at 5:05

Rajnik

839 bronze badges

Collectives™ on Stack Overflow

I am Getting the error undefined method `empty?' for nil:NilClass

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related