Avoid assigning variables within a conditional -- at first glance it looks like you made a mistake and used one equals sign for comparison instead of two. Many Ruby style-checking tools will throw a warning on something like this. I would instead do:

```
@bookmark = Bookmark.find_by_shorturl(params[:shorturl])
if @bookmark.present?
  ...
```

This also makes the condition clearer.