Add support to table_name_prefix and table_name_suffix

cavi21 commented 2 years ago

Address the issue #207

What is the purpose of this pull request?

As described in #207, there is no support for when the config.active_record.table_name_prefix or config.active_record.table_name_suffix are defined. This PR fixes what's relevant to logidze.

What changes did you make? (overview)

I add the method full_table_name as suggested by @palkan in the issue #207.
Quoted all the references to table or triggers names, in order to safely handle prefixes or suffixes that include for instance a - (hyphen). I explore the idea of using the quote_table_name() method on the connection for it but was not sure what approach do you prefer? what I thought that maybe we can have something like:
```
def quote_db_object_name(object_name)
ActiveRecord::Base.connection.quote_table_name(object_name)
end
```
The idea of having a method like this, is so that we can pass for instance logidze_on_#{full_table_name} and not only table names. And also because this gem supports only PG not sure if make sense to have this in there.
On the Logidze::Model I tweak the method reload_log_data in order to prevent the raise of the ActiveRecord::Sanitization::ClassMethods#disallow_raw_sql!
Changes on the specs to properly match the " (quotes) on the object names

Is there anything you'd like reviewers to focus on?

It's important to have in mind that this changes did not addresses the issue with the naming of the fx functions or triggers. I open an issue over there in order to know if it's something that's in the scope of the gem
Also because there are multiline and onliners SQL sometimes we need a escaped quote and sometimes we don't not sure if you want to move everything to use the <<~SQL and in such the quotation can be more consistent.

Checklist

[x] I've added tests for this change
[x] I've added a Changelog entry
[ ] ~I've updated a documentation (Readme)~

cavi21 commented 2 years ago

Hi @palkan I just pushed the requested changes, let me know what do you think

palkan commented 2 years ago

Merged into master (only relevant commits, w/o merge ones).

cavi21 commented 2 years ago

Thanks @palkan for all the effort put into this! have a great week!

palkan / logidze