fix(NcAppSettingsDialog): unregisterSection should remove the section instead of remove all other

[susnux]

☑️ Resolves

There was a logic error where unregister removed all but the unregistered section.
Also fix some minor issues and only warn on duplicated name instead of error.

🏁 Checklist

• ⛑️ Tests are included or are not applicable
• 📘 Component documentation has been extended, updated or is not applicable

[nickvergessen]

I guess this fixes nextcloud/spreed#10858 ?

[nickvergessen]

and only warn on duplicated name instead of error.

Not sure about this. Duplicate IDs are a HTML validation error and no one reads warnings :(

[ShGKme]

selectedSection and sorting don't work on dynamic sections rendering

[ShGKme]

I guess this fixes nextcloud/spreed#10858 ?

This error is caused by a typo in a provide. "Register" is provided as unregister.

	provide() {
		return {
			registerSection: this.registerSection,
			unregisterSection: this.registerSection,
		}
	},

[susnux]

I guess this fixes nextcloud/spreed#10858 ?

This error is caused by a typo in a provide. "Register" is provided as unregister.

Yes that is part of this fixes :)

[susnux]

Not sure about this. Duplicate IDs are a HTML validation error and no one reads warnings :(

@nickvergessen duplicated IDs still throw an error, only the duplicated name is a warning now.

Fixed all of the comments

[ShGKme]

Fine by issue solving.

But I'm not a fun of testing implementation in unit tests, because it doesn't help to check the unit and prevent further refactoring without rewriting the unit test. I can add an alternative test if it is ok for you.