allow . in dbname on web install

[butonic]

Uses JS to update the dbname input pattern when switching databases in the installer. Removes the autoconfig workaround that is still necessary after merging #10694

@DeepDiver1975 @ser72 @VicDeo @cdamken @gig13 @craigpg

[scrutinizer-notifier]

A new inspection was created.

[ghost]

🚀 Test Passed. 🚀
Refer to this link for build results: https://ci.owncloud.org/job/pull-request-analyser/7134/

[karlitschek]

makes sense 👍

[LukasReschke]

What's the reason to have this in the JS instead in the template? (e.g.

core/core/templates/installation.php

Line 125 in 317e6d0

pattern="[0-9a-zA-Z$_-]+" />

)

[butonic]

The pattern is different for oracle:
[0-9a-zA-Z$_-.]+ for oracle, because the dbname can contain dots
[0-9a-zA-Z$_-]+ for sane dbs

[butonic]

Qh ond there still is a server side check:
https://github.com/owncloud/core/blob/master/lib/private/setup/oci.php#L23
https://github.com/owncloud/core/blob/master/lib/private/setup/abstractdatabase.php#L23

[LukasReschke]

Thanks for the explanation. Then this makes indeed sense 👍

[LukasReschke]

@karlitschek The referenced PR #10694 was backported to stable7. Shall we backport this as well?

[karlitschek]

Yes please. We have bug reports here

[LukasReschke]

Stable7: 5e5d9e0