This project is archived and is in readonly mode.
Reflection do not reset quoted_table name when set_table_name is invoked
-
Neeraj Singh
def quoted_table_name @quoted_table_name ||= connection.quote_table_name(table_name) endAbove code can easily be modified to not to do memoization given that to get the quoted_name no database call is made. However quoted_table_name is called so many times that it is better to have faster access through an instance variable.
I will let people with more domain knowledge in this field decide if this should be fixed.
-
Ryan Bigg
- Tag set to bugmash
- State changed from new to incomplete
Please submit a patch (with tests) for this issue.
-
Santiago Pastorino
- Milestone cleared.
- Tag changed from bugmash to activerecord, bugmash
- State changed from incomplete to open
Please we have to be nice don't close the ticket we have some information.
I'm digging on this. -
Repository
- State changed from verified to resolved
(from [47c9a355062888feb2c7ea7c794e914a9b78f50c]) Reset quoted_table_name after set_table_name [#4568 Reflection do not reset quoted_table name when set_table_name is invoked state:resolved]
Signed-off-by: Pratik Naik pratiknaik@gmail.com
http://github.com/rails/rails/commit/47c9a355062888feb2c7ea7c794e91...
