Project

General

Profile

Actions

Bug #85772

open

EXT:form SaveToDatabaseFinisher cannot handle multi value fields

Added by Vasyl Mosiychuk over 6 years ago. Updated over 3 years ago.

Status:
New
Priority:
Could have
Assignee:
-
Category:
Form Framework
Target version:
-
Start date:
2018-08-07
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
11
PHP Version:
7.2
Tags:
Documentation
Complexity:
Is Regression:
Sprint Focus:

Description

[..]
        properties:
          options:
            1: 'Label 1'
            2: 'Label 2'
        type: MultiSelect
        identifier: categories
        label: 'Multi select'
        defaultValue:
          - '2'
[...]
          uid_foreign:
            value: '{SaveToDatabase.insertedUids.0}'
[...]

If I save only one value and not empty, work it. If a value more one it not work

I mean this is about `type: MultiSelect` and `type: ImageUpload`.
How to work with `SaveToDatabase` if I to pass for saving in the DB not one value and two value or three and more


Related issues 2 (1 open1 closed)

Related to TYPO3 Core - Task #85216: EXT:form SaveToDatabase Finisher saves files as sys_file, not as sys_file_referenceClosed2018-06-11

Actions
Related to TYPO3 Core - Feature #85378: EXT:form option for saving uploaded file as sys_file_referenceNew2018-06-25

Actions
Actions #1

Updated by Vasyl Mosiychuk over 6 years ago

  • Related to Task #85216: EXT:form SaveToDatabase Finisher saves files as sys_file, not as sys_file_reference added
Actions #2

Updated by Vasyl Mosiychuk over 6 years ago

  • Related to Feature #85378: EXT:form option for saving uploaded file as sys_file_reference added
Actions #3

Updated by Susanne Moog about 6 years ago

  • Subject changed from I can not saving in the DB values if I pass for saving in the DB not one value and two value or three and more? to EXT:form SaveToDatabaseFinisher cannot handle multi value fields
Actions #4

Updated by Susanne Moog about 6 years ago

  • Target version changed from 9.4 to Candidate for patchlevel
Actions #5

Updated by Björn Jacob over 5 years ago

  • Sprint Focus set to Remote Sprint
Actions #6

Updated by Susanne Moog over 5 years ago

The SaveToDatabase finisher was originally meant to provide a starting point for custom implementations and is therefor currently not capable of rendering all possible fields and combinations. In stable releases it should be used for very simple forms or as code tutorial on how to develop own finishers.

For future versions we need to define a concept of how and what we want to ship with the core.

Actions #7

Updated by Michael Waack over 5 years ago

  • Assignee set to Michael Waack
Actions #8

Updated by Michael Waack over 5 years ago

  • Assignee deleted (Michael Waack)
Actions #9

Updated by Björn Jacob over 5 years ago

  • Sprint Focus deleted (Remote Sprint)
Actions #10

Updated by Vasyl Mosiychuk over 5 years ago

Susanne Moog wrote:

The SaveToDatabase finisher was originally meant to provide a starting point for custom implementations and is therefor currently not capable of rendering all possible fields and combinations. In stable releases it should be used for very simple forms or as code tutorial on how to develop own finishers.

For future versions we need to define a concept of how and what we want to ship with the core.

In to manual not said nothing about that the `ext:from` ... should be used for very simple forms or as code tutorial on how to develop own finishers, as for a non-technical editor, so and as an experienced integrator.

I think the `ext:form` is should provide possibility to save all data in the database which sent from the form.

In this case TYPO3 and the `ext:from` will be available for all users, and really will be simple and easy...

Actions #11

Updated by Björn Jacob over 5 years ago

  • Target version deleted (Candidate for patchlevel)

If you want to store the form data in your database, try this extension: https://github.com/pagemachine/typo3-formlog.

Actions #12

Updated by Vasyl Mosiychuk over 5 years ago

Bjoern Jacob wrote:

If you want to store the form data in your database, try this extension: https://github.com/pagemachine/typo3-formlog.

No, this is not that what need...

I need save data in the table of database for example extension news or other extension where is in the record has more one category and more one images.

Actions #13

Updated by Susanne Moog over 5 years ago

The problem is, we will not be able to support arbitrary use cases. One use case could be to have categories as csv - kind of a log entry -, the next wants the categories created and saved as 1:n relation, next is m:m relation and so on. We have no secure way of figuring out what kind of form people are building and guessing will make the situation worse. Trying to support any kind of datastructures will be extremely complicated, quite impossible to get right and complex - whereas creating an own finisher for your specific case should be a lot more straightforward, as you - and only you - know exactly what you are building and what the logic should be for the relations.

The only reliable configuration source we have for tables with relations at the moment would be TCA, which would essentially mean, rebuilding form engine (backend forms) for the frontend - that however would not be a task for the form framework, but rather a separate package.

Actions #14

Updated by Vasyl Mosiychuk over 5 years ago

Susanne Moog wrote:

The problem is, we will not be able to support arbitrary use cases. One use case could be to have categories as csv - kind of a log entry -, the next wants the categories created and saved as 1:n relation, next is m:m relation and so on. We have no secure way of figuring out what kind of form people are building and guessing will make the situation worse. Trying to support any kind of datastructures will be extremely complicated, quite impossible to get right and complex - whereas creating an own finisher for your specific case should be a lot more straightforward, as you - and only you - know exactly what you are building and what the logic should be for the relations.

The only reliable configuration source we have for tables with relations at the moment would be TCA, which would essentially mean, rebuilding form engine (backend forms) for the frontend - that however would not be a task for the form framework, but rather a separate package.

...and there are not so many cases, mainly it is "categories" and "tags" and "sys_file_reference".

at least core "sys_category" and "sys_file_reference" would be well

I think, that need describing in the documentation about those cases. Can you describe in documentation create own finisher for this case as an example?

Actions #15

Updated by Susanne Moog about 5 years ago

  • Sprint Focus set to Remote Sprint
Actions #16

Updated by Mathias Brodala almost 5 years ago

  • Tags set to Documentation
Actions #17

Updated by Julian Stock over 4 years ago

Vasyl Mosiychuk wrote:

[...]
I think, that need describing in the documentation about those cases. Can you describe in documentation create own finisher for this case as an example?

Do you still need help? I managed to get some sys_category in the ext:form frontend via a custom formElementsDefinition (in yaml & php, mainly following this: https://daniel-siepmann.de/posts/2017/how-to-create-typo3-form-select-element-with-options-selected-from-database.html), set it to MultiSelect and used the default (but extended) SaveToDatabaseFinisher plus a following SaveToDatabaseFinisher which gets the '{SaveToDatabase.insertedUids.0}' value.

This is the logic of my second SaveToDatabaseFinisher:

        // ...

        protected function saveToDatabase(array $databaseData, string $table, int $iterationCount) {
            // ...

            $categories = explode(",", $databaseData["categories"]);
            $sorting = 0;
            foreach ($categories as $category) {
                $categoryData = [
                    "uid_local" => $databaseData["guide_id"],
                    "uid_foreign" => $category,
                    "sorting" => $sorting
                ];

                parent::saveToDatabase($categoryData, $table, $iterationCount);

                $sorting++;
            }
        }

        // ...

Have a great day!

Actions #18

Updated by Björn Jacob over 3 years ago

  • TYPO3 Version changed from 9 to 11
  • Sprint Focus deleted (Remote Sprint)
Actions

Also available in: Atom PDF