Gabriele Renzi

.decorate on an ActiveRecord::Associations::CollectionProxy ignores current value

1

This is probably related to https://github.com/drapergem/draper/issues/646, but my specific problem is with unpersisted objects, i.e. if there is an unsaved object in an association in the same example code (`user...

I have some html that is rendered through the rails rendering pipeline, and ends up with properly escaped special characters. Sadly, when this goes inside premailer, it comes out unescaped....

False positive of protect_from_forgery

1

this code ``` class ApplicationController < ActionController::Base protect_from_forgery with: :exception end ``` results in a warning `Owasp Ror CheatSheet: Cross Site Request Forgery check failed` but unless I'm missing something,...

crash parser when defining method inside block

2

Reproductible with ``` $ cat c.rb class C send :each do def foo end end end $ laser -d c.rb {:fix=>false, :display=>true, :"report-fixed"=>false, :"line-length"=>nil, :only=>nil, :stdin=>false, :"list-modules"=>false, :profile=>false, :debug=>true, :inline_comment_space=>2,...

Allow repl launched from code to be handled from within notebook?

3

The use case is the following: I have a long data massaging loop like ``` entries.each do begin ... a lot of stuff that can go wrong rescue => e...

unary splat operator is not handled properly

5

I have seen this with the rails/strong_parameters snippet, but I think the problem is more general. Basically, given a source like ``` attr_accessible *args ``` doing ``` eval arguments.first.to_source #...

improvement: warn user if split processing omits some styles

1

i.e. I had styles: {round: .., large: ..., small: ...} process_only: [:large] process_in_background :picture, only_process: %i[small], and forgot one style, which would never be processed. It would be nice to...

coding style: single line method definitions

1

I wonder if there is consensus on how/when to use single line method definitions. E.g. ``` def foo() bar end vs def foo; bar; end ``` It's a small thing,...

I realized these are missing, both as editor conventions (e.g., if base indent is 2, indent "private" keyword by 1 is what I use) and coding style. Off the top...