smindel / silverstripe-gridfieldeditablemanymanyextracolumns

adds form fields to the Silverstripe GridField to edit DataObject::$many_many_extraFields
7 stars 2 forks source link

Not working in SS 3.1 #3

Open anselmdk opened 9 years ago

anselmdk commented 9 years ago

This seems to be a very useful module, and I don't understand why this hasn't been picked up more. I was hesitant trying it because it didn't have a composer.json file, and it actually doesn't work in 3.1. See screenshot.

I'd love to help getting it going, but am having issues wrapping my head around some of the code - maybe we could make this a shared (or sponsored) effort?

screenshot 2015-05-16 09 22 00

anselmdk commented 9 years ago

Actually I got it to display something adding if (is_array($extrafields[$remotename])) { on line 47, this looks great! @smindel what are your plans with this? it's a shame this module is unmaintained.

anselmdk commented 9 years ago

What I'm referring to are these - I had been looking for a way to display extra fields i a Gridfield, and had no idea how to. See screenshot, and model here:

private static $many_many_extraFields = [
    'Brands' => [
        "Active" => "Boolean",
        "Activated" => "SS_Datetime",
        "Deactivated" => "SS_Datetime",
    ]
];

They don't really conicide, as "Activated" and "Deactivated" should be datetime fields, but it's definitely a start!

screenshot 2015-05-16 09 36 26

anselmdk commented 9 years ago

Looking more int this, I see some effort has been done to allow similar behavior:

None of it is completely obvious though, I'll try to collect some more information.