SQL Error in Redirect Module

[SicorDev]

SQL Error in Redirect Module

[thommyhh]

@SicorDev This one is unfortunately not that easy. The problem is that TYPO3 saves the record on type change. In that cases all validations in the form are skipped, probably because the fields might change with the type change.

I tried to improve the behaviour by removing the empty default option in the domain select, so the redirect would in any case show in in the list and would not disappear. Unfortunately this could also lead to that duplicate key sql error because the url to the homepage has an empty path.

I also made the path required, so this might help the user to understand, that this field must be filled. But as I said, I currently can't find a way to avoid TYPO3 from saving the record in type change.

I update all 3.x branches. Please try the change and let me know what you think.

[SicorDev]

@thommyhh
I've tried the current master branch, but I couldn't find any difference in the behaviour.

I've got another idea, how to prevent the error in all cases:
You could add a "afterSaveHook", that sets a default path value, if no value is set.
(Untested example)

ext_tables.php

$GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['t3lib/class.t3lib_tcemain.php']['processDatamapClass'][] = 'EXT:' . $_EXTKEY . '/Classes/Hooks/Hook.php:Hook';

Hook.php

function processDatamap_afterDatabaseOperations($status, $table, $id, &$fieldArray, &$pObj) {
  if ($table == 'tx_naworkuri_uri' && $status === 'new') {
    if(!$fieldArray['path']) {
       $fieldArray['path'] = 'default-value' . time();
    }
  }
}

[thommyhh]

@SicorDev The problem is, that the SQL errors occurs when saving, meaning a default value must be set before. On the other hand, it could theoretically be possible to have an empty path. If I remember correctly, the url for the homepage has an empty path, which doesn't really makes sense.

Maybe the fix is, to not allow empty paths at all, and handle an empty path as the homepage. Adding the default value, if the record was saved by the type change with an empty path.