Discussion on: Let's Use Rails Partials To Render Art from Magic: the Gathering!

View post

Replies for: Also another shortcut instead of this: session[:img_array] = session[:img_array] || [] Enter fullscreen mode Exit fullscreen m...

These are both excellent pieces of advice, thank you Andrew! I did incorporate the ||= bit (is there a name for that??) into the most recent refactor, and am on the lookout for other places to use it in my code. I gave you a shoutout at the bottom of my followup article where I used it, let me know if you want me to change it/remove it/link to something else of yours: dev.to/isalevine/using-rails-servi...

I'll definitely be referring back to your point about the simpler syntax for partials the next time I'm working with them too! I definitely like to be as overly-verbose and non-shortcut-y as possible at the start, but no question the shortened syntax looks better and more readable. :)

Andrew Brown 🇨🇦 • Nov 18 '19

I like to be as overly-verbose and non-shortcut-y as possible at the start

👍👍👍

Isa Levine • Nov 19 '19 • Edited

Also, this is interesting--I was refactoring to shorten-ify-icate the partials syntax, but apparently removing the partial: bit screws up how the collection: foo , as: :bar syntax.

So, this:

<%= render '/pages/cards/card', collection: session[:img_array], as: :img_hash %>

led to this:

Any idea why collection: isn't iterating through session[:img_array] and repeatedly passing each img_hash without the partial: explicitly there?

Andrew Brown 🇨🇦 • Nov 19 '19

With collection, I supposed short form doesn't work since the logic is too difficult.

So I guess short form only works for non-collection partials