I have a model in a plugin. The model is Pfaffmananger::Server, the table is pfaffmanager_servers. For Reasons, I want to have custom fields for this model (I expect there to be ways that I want to extend the server data that are likely to be used only for a few instances and I don’t want a zillion …

[画像] pfaffman: If I use pfaffmanager_server_id instead of server_id the migration fails because it can’t find server_id Did you also update the add_index call to point to pfaffmanager_server_id?

Thanks so much, David! [画像] david: Did you also update the add_index call to point to pfaffmanager_server_id ? Yes. That could be my problem. I guess that’s the error message I should have included. And I can make the migration work with either field (though with pfaffmanager_server_id as t…

[画像] pfaffman: It doesn’t care what the index name is, does it? I don’t think so, no I think the second approach is probably the one to aim for (using pfaffman_server_id as the column name) I wonder if overriding this method may help: app/models/concerns/has_custom_fields.rb ma…

Or maybe it’s this bit :thinking: app/models/concerns/has_custom_fields.rb main value.to_i when :json

Thanks so much! So Server.new.custom_fields_fk is pfaffmanager_server_id if I use pfaffmanager_serfver_id. So maybe I should use server_id and then override as you suggested above. It looks like refresh_custom_fields_from_db is trying to use the wrong names. I don’t quite understand what _custom_f…

Ha! You did it! Thanks so much. I switched it to server_id and then did this in my server.rb model: def custom_fields_fk @custom_fields_fk ||= "server_id" end As long as that’s going to override that only for this model and not break user_custom_field and friends I think I should be …

[画像] pfaffman: I switched it to server_id and then did this in my server.rb model: Great! :tada: If you can think of a way to make it more generic then that would certainly be PR-welcome.

If something occurs to me I’ll let you know or submit a PR, but I have to think that a plugin that wants to use custom fields is pretty far out on the edge.

OM_fk_G. custom_fields_fk seemed so stupid that I assumed it was something that I meant to delete and had named it something stupid so I’d know it was safe to delete it later. It was later, so I deleted it. And then my specs failed. There was even a comment with a link to this topic in the code.

プラグインでHasCustomFieldsを使用する

開発

david (David Taylor) 2021 年 1 月 18 日午後 9:16 8

素晴らしいです！より汎用的にする方法をご提案いただけるようであれば、ぜひプルリクエストをお待ちしています。

トピック		返信	表示
Error: PG::UndefinedColumn. Custom field not creating column Support	4	1633	2021 年 8 月 11 日
Best convention for custom plugin DB data for 2021? Development	4	663	2021 年 3 月 17 日
User custom field with plugin Development	2	2064	2017 年 5 月 5 日
Custom field casting affected by recent update? Development	5	1269	2020 年 8 月 25 日
Plugin to validate a text type custom field value Development	1	100	2024 年 9 月 9 日

プラグインでHasCustomFieldsを使用する

関連トピック