Settings icon for LDAP and encryption

[MorrisJobke]

• follow up to add icons to navigation of personal & admin settings #3151

[mention-bot]

@MorrisJobke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @blizzz and @nickvergessen to be potential reviewers.

[ChristophWurst]

fancy!

[nickvergessen]

There was 1 failure:
1) OCA\User_LDAP\Tests\Settings\SectionTest::testGetIcon
Expectation failed for method name is equal to <string:imagePath> when invoked 1 time(s)
Parameter 1 for invocation OCP\IURLGenerator::imagePath('user_ldap', 'app-dark.svg') does not match expected value.
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'app.svg'
+'app-dark.svg'
/drone/src/github.com/nextcloud/server/apps/user_ldap/lib/Settings/Section.php:80
/drone/src/github.com/nextcloud/server/apps/user_ldap/tests/Settings/SectionTest.php:74

[jancborchardt]

Can you use the icon for multiple people instead of a single person? :) That makes more sense.

[nickvergessen]

Well that is the current app icon:

So if we change this, we should change both.

Right, probably good after all since otherwise it would be confused with Contacts.