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 …

[immagine] 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! [immagine] 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_i…

[immagine] 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 …

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 …

[immagine] 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.

Utilizzo di HasCustomFields in un plugin

Sviluppatore

david (David Taylor) 18 Gennaio 2021, 9:16pm 8

Ottimo! Se riesci a pensare a un modo per renderlo più generico, sarebbe certamente ben accetto come PR.

Argomento		Risposte	Visualizzazioni
Error: PG::UndefinedColumn. Custom field not creating column Support	4	1633	Agosto 11, 2021
Best convention for custom plugin DB data for 2021? Development	4	663	Marzo 17, 2021
User custom field with plugin Development	2	2064	Maggio 5, 2017
Custom field casting affected by recent update? Development	5	1269	Agosto 25, 2020
Plugin to validate a text type custom field value Development	1	100	Settembre 9, 2024

Utilizzo di HasCustomFields in un plugin

Argomenti correlati