Support id_column and index_column in table_for #6461
Conversation
| @@ -27,7 +27,7 @@ Feature: Belongs To | |||
| """ | |||
| ActiveAdmin.register User | |||
| ActiveAdmin.register Post do | |||
| belongs_to :user | |||
| belongs_to :author, class_name: "User", param: "user_id", route_name: "user" | |||
There was a problem hiding this comment.
Are these and the other changes in this file related to this PR?
There was a problem hiding this comment.
Yes, they are needed. I am not too familiar with belongs_to, but AFAICT the arguments are necessary for this to work at all (because the class name, User, does not match the attribute name, author). But I don't understand why the tests did not fail before. I'll try to figure out what is going on.
There was a problem hiding this comment.
Did you figure it out @c960657?
bb23b36 to
8666eb2
Compare
docs/12-arbre-components.md
Outdated
| The `column` method can take a title as its first argument and data | ||
| (`:your_method`) as its second (or first if no title provided). Column also | ||
| takes a block. | ||
| `column`, `index_column`, and `id_column` works like for [index tables](3-index-pages/index-as-table.md). |
There was a problem hiding this comment.
| `column`, `index_column`, and `id_column` works like for [index tables](3-index-pages/index-as-table.md). | |
| `column`, `index_column`, and `id_column` work like for [index tables](3-index-pages/index-as-table.md). |
I think "work" is grammatically correct?
8666eb2 to
4cb4bf8
Compare
Likewise :-)
The difference is caused by the change from |
|
We also need |
id_columnandindex_columnare not only useful in index tables but also intable_for. Move these two methods to the parent case, so they are available for any table.I renamed the
i18noption fortable_fortoresource_class(with fallback toi18nfor BC), because with this change, the specified model is used not only for i18n purposes but also for determining the primary key.