Proffer doesn't create a value for scource_dir in the entity

[Fluit]

I'm using the Proffer plugin to upload several documents or pictures, linked to a machine.

In DocsTable I added:

        $this->addBehavior('Proffer.Proffer', [
            'photo' => [
                'root'=>WWW_ROOT.'files',
                'dir' => 'scource_dir',
                'thumbnailSizes' => [
                    'square' => ['w' => 100, 'h' => 100, 'crop'=>true],
                ],
                'thumbnailMethod' => 'Gd'
            ],
            // -> files/<table naam>/<key -> scource>/<value of scource_dir>/
            'scource' => [
                'root'=>WWW_ROOT.'files',
                'dir'=>'scource_dir'
            ]
        ]);

Since Machines has a hasMany association, I added in MachinesTable:

        $this->hasMany('MachineDocs',[
            'className'=>'Docs',
            'foreignKey'=>'machine_id'
        ]);

In the form is I use this code:

        $i = 0;
        <?= $this->Form->create($result,['type' => 'file']);?>
        <?= $this->Form->control('machine_docs.'.$i.'.id', ['type' => 'text','label' => __('id')])?>
        <?= $this->Form->control('machine_docs.'.$i.'.machine_id', ['type' => 'text','label' => __('machine_id')])?>
        <?= $this->Form->control('machine_docs.'.$i.'.idv_id', [
                'type' => 'text',
                'label' => __('idv_id'),
                'default' => 0
        ])?>
        <?= $this->Form->control('machine_docs.'.$i.'.omschrijving', [
                'type' => 'text',
                'default' => 'omschrijving'
        ])?>
        <?= $this->Form->control('machine_docs.'.$i.'.scource', ['type' => 'file','label' => __('Document')])?>
        <?= $this->Form->control('machine_docs.'.$i.'.scource_dir', ['type' => 'hidden',])?>
        <?= $this->Form->button(__('Upload file'),['type'=>'submit','confirm'=>'Confirm', 'class'=>'upload_button']);?>
      <?= $this->Form->end();?>

In the controller there is

            $result = $this->Machines->patchEntity($result, $data, ['associated' => ['MachineDocs']]);

            if ($this->Machines->save($result)) {
                $this->Flash->success(__('The {0} has been saved.',  strtolower($this->name)));
                if ($this->request->is('put')) $this->render; //de edit-view wordt opnieuw gerenderd
                /*
                 * @todo
                 * de index-view renderen als deze pagina beschikbaar is.
                 */
            }

Problem

Al seems to work fine. The result with al the data is send to my database correctly, except for "source_dir", which stays empty.

debug

After patching the $data in the controller I get as $result:

object(App\Model\Entity\Machine) {

    'id' => (int) 373,
    'reg_datum' => object(Cake\I18n\Time) {

        'time' => '2019-02-05T14:54:49+00:00',
        'timezone' => 'UTC',
        'fixedNowTime' => false

    },
    'naam' => 'MAKITA DTW450RTJ SLAGMOERSLEUTEL',
    'naamlijstnummer' => '',
    'asset_im_nr' => 'IM31561',
    'fabrikant' => 'MAKITA',
    'model_type' => 'DTW450RTJ',
    'leverancier' => 'ERIKS',
    'bouwjaar' => object(Cake\I18n\Date) {

        'time' => '2018-05-18T00:00:00+00:00',
        'timezone' => 'UTC',
        'fixedNowTime' => false

    },
    'gebruik' => '',
    'instructiebundel' => 'MAKITA DTW450_Accuslagmoersleutel_manual.pdf',
    'foto_id' => (int) 376,
    'vik_id' => (int) 0,
    'idvs_id' => (int) 0,
    'machine_parent_id' => (int) 0,
    'url_vifis' => '',
    'url_instructiebundel' => '',
    'vifis_id' => (int) 0,
    'manual_dir' => 'a9ecf18c-3284-495b-9bee-d2d206f734cc',
    'vifis_dir' => null,
    'vifis_file' => null,
    'asset_id' => null,
    'machine_docs' => [
        (int) 0 => object(App\Model\Entity\Doc) {

            'idv_id' => (int) 0,
            'omschrijving' => 'omschrijving test dsfq',
            'typedoc_id' => (int) 3,
            'scource' => '1GL_Algemeen.docx',
            'scource_dir' => null,
            '[new]' => true,
            '[accessible]' => [
                '*' => true,
                'id' => false,
                'photo' => true,
                'scource' => true,
                'scource_dir' => true
            ],
            '[dirty]' => [
                'idv_id' => true,
                'omschrijving' => true,
                'typedoc_id' => true,
                'scource' => true,
                'scource_dir' => true
            ],
            '[original]' => [],
            '[virtual]' => [],
            '[errors]' => [],
            '[invalid]' => [],
            '[repository]' => 'MachineDocs'

        }
    ],
...
...
    '[new]' => false,
    '[accessible]' => [
        '*' => true,
        'docs' => true,
        'id' => false
    ],
    '[dirty]' => [
        'machine_docs' => true
    ],
    '[original]' => [
        'machine_docs' => [
            (int) 0 => object(App\Model\Entity\Doc) {

                'id' => (int) 1,
                'reg_datum' => object(Cake\I18n\Time) {

                    'time' => '2019-02-08T11:05:02+00:00',
                    'timezone' => 'UTC',
                    'fixedNowTime' => false

                },
                'omschrijving' => 'omschrijving',
                'url' => '',
                'scource' => null,
                'scource_dir' => null,
                'typedoc_id' => (int) 2,
                'machine_id' => (int) 373,
                'idv_id' => (int) 0,
                'photo' => null,
                'typedoc' => object(Cake\ORM\Entity) {

                    'id' => (int) 2,
                    'naam' => 'handlediing',
                    'nom' => '',
                    'img' => 'tag_gray.png',
                    '[new]' => false,
                    '[accessible]' => [
                        '*' => true
                    ],
                    '[dirty]' => [],
                    '[original]' => [],
                    '[virtual]' => [],
                    '[errors]' => [],
                    '[invalid]' => [],
                    '[repository]' => 'Typedoc'

                },
                '[new]' => false,
                '[accessible]' => [
                    '*' => true,
                    'id' => false,
                    'photo' => true,
                    'scource' => true,
                    'scource_dir' => true
                ],
                '[dirty]' => [],
                '[original]' => [],
                '[virtual]' => [],
                '[errors]' => [],
                '[invalid]' => [],
                '[repository]' => 'MachineDocs'

            },
            (int) 1 => object(App\Model\Entity\Doc) {

                'id' => (int) 2,
                'reg_datum' => object(Cake\I18n\Time) {

                    'time' => '2019-02-08T11:16:59+00:00',
                    'timezone' => 'UTC',
                    'fixedNowTime' => false

                },
                'omschrijving' => 'omschrijving',
                'url' => '',
                'scource' => null,
                'scource_dir' => null,
                'typedoc_id' => (int) 2,
                'machine_id' => (int) 373,
                'idv_id' => (int) 0,
                'photo' => null,
                'typedoc' => object(Cake\ORM\Entity) {

                    'id' => (int) 2,
                    'naam' => 'handlediing',
                    'nom' => '',
                    'img' => 'tag_gray.png',
                    '[new]' => false,
                    '[accessible]' => [
                        '*' => true
                    ],
                    '[dirty]' => [],
                    '[original]' => [],
                    '[virtual]' => [],
                    '[errors]' => [],
                    '[invalid]' => [],
                    '[repository]' => 'Typedoc'

                },
                '[new]' => false,
                '[accessible]' => [
                    '*' => true,
                    'id' => false,
                    'photo' => true,
                    'scource' => true,
                    'scource_dir' => true
                ],
                '[dirty]' => [],
                '[original]' => [],
                '[virtual]' => [],
                '[errors]' => [],
                '[invalid]' => [],
                '[repository]' => 'MachineDocs'

            },
...
...
            (int) 9 => object(App\Model\Entity\Doc) {

                'id' => (int) 10,
                'reg_datum' => object(Cake\I18n\Time) {

                    'time' => '2019-02-08T15:58:24+00:00',
                    'timezone' => 'UTC',
                    'fixedNowTime' => false

                },
                'omschrijving' => 'omschrijving 16u59',
                'url' => '',
                'scource' => '1GL_Algemeen.docx',
                'scource_dir' => null,
                'typedoc_id' => (int) 5,
                'machine_id' => (int) 373,
                'idv_id' => (int) 0,
                'photo' => null,
                'typedoc' => object(Cake\ORM\Entity) {

                    'id' => (int) 5,
                    'naam' => '3de GL',
                    'nom' => '',
                    'img' => 'tag_orange.png',
                    '[new]' => false,
                    '[accessible]' => [
                        '*' => true
                    ],
                    '[dirty]' => [],
                    '[original]' => [],
                    '[virtual]' => [],
                    '[errors]' => [],
                    '[invalid]' => [],
                    '[repository]' => 'Typedoc'

                },
                '[new]' => false,
                '[accessible]' => [
                    '*' => true,
                    'id' => false,
                    'photo' => true,
                    'scource' => true,
                    'scource_dir' => true
                ],
                '[dirty]' => [],
                '[original]' => [],
                '[virtual]' => [],
                '[errors]' => [],
                '[invalid]' => [],
                '[repository]' => 'MachineDocs'

            },
            (int) 10 => object(App\Model\Entity\Doc) {

                'id' => (int) 11,
                'reg_datum' => object(Cake\I18n\Time) {

                    'time' => '2019-02-08T16:08:02+00:00',
                    'timezone' => 'UTC',
                    'fixedNowTime' => false

                },
                'omschrijving' => 'omschrijving test',
                'url' => '',
                'scource' => '1GL_Algemeen.docx',
                'scource_dir' => null,
                'typedoc_id' => (int) 4,
                'machine_id' => (int) 373,
                'idv_id' => (int) 0,
                'photo' => null,
                'typedoc' => object(Cake\ORM\Entity) {

                    'id' => (int) 4,
                    'naam' => '1ste GL',
                    'nom' => '',
                    'img' => 'tag_green.png',
                    '[new]' => false,
                    '[accessible]' => [
                        '*' => true
                    ],
                    '[dirty]' => [],
                    '[original]' => [],
                    '[virtual]' => [],
                    '[errors]' => [],
                    '[invalid]' => [],
                    '[repository]' => 'Typedoc'

                },
                '[new]' => false,
                '[accessible]' => [
                    '*' => true,
                    'id' => false,
                    'photo' => true,
                    'scource' => true,
                    'scource_dir' => true
                ],
                '[dirty]' => [],
                '[original]' => [],
                '[virtual]' => [],
                '[errors]' => [],
                '[invalid]' => [],
                '[repository]' => 'MachineDocs'

            }
        ]
    ],
    '[virtual]' => [
        (int) 0 => 'url_instructiebundel_cake',
        (int) 1 => 'url_keuring_cake'
    ],
    '[errors]' => [],
    '[invalid]' => [],
    '[repository]' => 'Machines'

When I look deeper into your code, I notice that there might be an issue with the method "beforeSave" in "ProfferBehavior.php". Although $entity does have 'scource' as a field it never creates an seed for 'scource_dir' and won't save the file to the server, I think caused by the fact that $entity->get('scource') isn't an array.

How can I solve this?

Thanks in advance.

[davidyell]

There is code to set the path back to the entity,

https://github.com/davidyell/CakePHP3-Proffer/blob/master/src/Model/Behavior/ProfferBehavior.php#L137

I think you are setting a value in your post data because you've got a hidden input. This empty value might be overwriting the entity property when you patchEntity with the request data.

[Fluit]

I've removed these lines. Even when de posted data doesn't contain the 'scource_dir' field, there is still no value in de saved record. The problem, I think (when looking a bit deeper) is that in line 87 $entity->get($field)['error'] === UPLOAD_ERR_OK returns 'false' and even '$entity->get('error') === UPLOAD_ERR_OK' in elseif, so it's why '$this->process()' is skipped. After passing line 87, directly true is returned (line 102)

[davidyell]

So you're having a problem uploading a file to the server. Perhaps this could be a file system permissions issue? move_uploaded_file() would be a good test to ensure that PHP can write to your file system.