Support DC_File/DC_Folder

[m-vo]

see #30

@zoglo Could you test if this works in your setup?

[zoglo]

Hello @m-vo ,

thank you for your effort, i've installed and tested your new branch: "dev-feature/support-file-dcs".

Unfortunately, it still doesn't work as intended. The group widget still doesn't show within DC_File.
I've tested my DCA field within a DC_Table and it does show and save as intended.

Here is an example of my DCA field:

'groupWidgetTest' => array
(
	'exclude' => true,
	'inputType' => 'group',
	'eval' => ['tl_class' => 'clr'],
	'palette' => [
		'image',
		'textColor',
		'textSize',
		'textOffsetX',
		'textOffsetY',
		'step'
	],
	'fields' => [
		'image' => [
			'inputType' => 'fileTree',
			'eval' => ['mandatory' => true, 'fieldType' => 'radio', 'filesOnly' => true, 'extensions' => Config::get('validImageTypes')],
		],
		'textColor' => [
			'inputType' => 'text',
			'eval' => ['mandatory' => true, 'valign' => 'bottom', 'maxlength' => 6, 'size' => 1, 'colorpicker' => true, 'isHexColor' => true, 'decodeEntities' => true, 'tl_class' => 'wizard', 'style' => 'width:100px'],
		],
		'textSize' => [
			'inputType' => 'text',
			'default' => 14,
			'eval' => ['mandatory' => true, 'valign' => 'bottom', 'maxlength' => 6, 'size' => 1, 'colorpicker' => true, 'isHexColor' => true, 'decodeEntities' => true, 'tl_class' => 'wizard', 'style' => 'width:100px'],
		],
		'textOffsetX' => [
			'inputType' => 'text',
			'default' => 0,
			'eval' => ['mandatory' => true, 'valign' => 'bottom', 'style' => 'width:100px', 'rgxp' => 'digit'],
		],
		'textOffsetY' => [
			'inputType' => 'text',
			'default' => 0,
			'eval' => ['mandatory' => true, 'valign' => 'bottom', 'style' => 'width:100px', 'rgxp' => 'digit'],
		],
		'step' => [
			'inputType' => 'text',
			'default' => 0,
			'eval' => ['mandatory' => true, 'valign' => 'bottom', 'style' => 'width:80px', 'rgxp' => 'natural'],
		]
	],
	'min' => 1,
	'max' => 5,
	'order' => false,
	'sql' => [
		'type' => 'blob',
		'length' => \Doctrine\DBAL\Platforms\MySqlPlatform::LENGTH_LIMIT_BLOB,
		'notnull' => false,
	],
),

[m-vo]

Did you adjust the default tl_files palette?

[zoglo]

Yes, I am using the default palette for it.
We are using this DCA to save it into localconfig.php

// Config
'config' => array
(
    'dataContainer'          => 'File',
    'closed'                 => true
),


// Palettes
'palettes' => array
(
    'default' => '{default_legend},groupWidgetTest,...'
),

// Fields
'fields' => array
(
   ...
    'groupWidgetTest' => array
    (
       ...
    )
)

[m-vo]

Ah sorry, I thought you wanted to use this e.g. for additional fields in tl_files (this would be DC_Folder then). It should show in DC_Files as well, though. Could you debug why it's not?

Nevertheless, you would need another storage adapter as well if you want to store into a file: https://github.com/m-vo/contao-group-widget/blob/main/docs/storage/extending.md

[m-vo]

This should probably return 0 (or any other static value) for DC_File.

[zoglo]

Hey @m-vo ,

I've found out a few things whist debugging, I hope that following information helps:

I do not reach getRowId due to following line:

public function initializeGroups(string $table): void
{
...
    if (!\in_array($action = $request->get('act', ''), ['edit', 'delete'], true)) {
        return;
    }
    

There is no action in the request (contao?do=config), thus it never gets registered.

---

After removing that part:

$dc instanceof DC_File returns true

$dc->id returns null

---

I'll test your suggestion with the storage adapter and will get back to you.

[m-vo]

@zoglo I updated the PR. The difference with the file driver is that it doesn't have a the concept of a "row" - there's only one. This PR works around this now. But you'll have to manage records yourself with your own storage.

Have a look at the SerializedStorage - you should be able to adapt your - say - FileStorage from this but change the database parts to file access.