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_serve…

[изображение] 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.…

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) 18.Январь.2021 21:16:15 8

Отлично! Если вы сможете придумать способ сделать это более универсальным, то это, безусловно, будет приветствоваться в виде PR.

Тема		Ответов	Просм.
Error: PG::UndefinedColumn. Custom field not creating column Support	4	1633	11.08.2021
Best convention for custom plugin DB data for 2021? Development	4	663	17.03.2021
User custom field with plugin Development	2	2064	05.05.2017
Custom field casting affected by recent update? Development	5	1269	25.08.2020
Plugin to validate a text type custom field value Development	1	100	09.09.2024

Использование HasCustomFields в плагине

Связанные темы