defaultForAllTables doesn't work as described in the docs

[j4k3]

The configuration value defaultForAllTables does not seem to work as described in the documentation. It is, in fact not even mentioned in the hook method, getDefaultUploadFolder() at all.
I've modified the file by duplicating the following lines and appending them at line 45, modifying them like so,

    if ($subFolder['value'] === null) {
        $subFolder = $backendUserAuthentication->getTSConfig(
            'default_upload_folders.' . $table,
            $pageTs
        );
    }
    if ($subFolder['value'] === null) {
        $subFolder = $backendUserAuthentication->getTSConfig(
            'default_upload_folders.defaultForAllTables',
            $pageTs
        );
    }

It now seems to work as intended. Was this just missing, or am I doing something wrong?

[joey-bolts]

@j4k3 From what i have been able to test it seems to be working. Is this issue still relevant, if so. How can we help?