Provide better feedback when using CsvBulkLoader

[sabina-talipova]

Description

We pass message as empty string in BulkLoader_Result::addUpdated from CsvBulkLoader::processRecord (See). We must pass a more clear message to the addUpdated method, or delete this variable from processRecord , since the method addUpdated uses null by default.

See:

• silverstripe-framework/src/Dev/CsvBulkLoader.php

  Line 163 in cf78a9d

  * @todo Better messages for relation checks and duplicate detection

• silverstripe-framework/src/Dev/CsvBulkLoader.php

  Line 264 in cf78a9d

  // @todo better message support

Note that this message is (or is supposed to be) displayed in the form modal which is used to trigger the bulk action.

[GuySartorelli]

@sabina-talipova The description doesn't seem related to the issue title or the links - can you please check this one? I think it got mixed up with #10995

[sabina-talipova]

Based on your post here.
If we are not going to add support of $columnMap in CsvBulkLoader::findExistingObject, than I think we can close this ticket as well and open another one to remove parameter from both "methods signature".

[GuySartorelli]

Sounds like a plan

[sabina-talipova]

Sorry, I copied description from another ticket by accident. I've update description again. I reopen ticket.