LPS-32615 Replace getScopeGroupId() with getSiteGroupId()

What is really needed in this class is always the groupId of the site where the asset publisher instance is placed. This change is safe because in this context the scopeGroupId and the siteGroupId are always going to be the same (which is the same reason why it has worked so far). But semantically speaking what makes sense is the siteGroupid.
LPS-32615 The parameter always represent the groupId of the site where the asset publisher instance is placed
LPS-32615 Removing condition that will never happen for parent sites
LPS-32615 Allow site admin to always select children sites
LPS-32745 do not show button for children if there are no children neither button for parents if there are not parents
LPS-32745 autogenerated
LPS-32745 javadocs
LPS-32745 Source formatting
LPS-32745 We are assigning the var siteBrowserURLString but never using it
LPS-32745 But at least we were consistent in the bug :) so easy to fix in multiple places
LPS-32745 The pattern "String taglib*" is used vs "String ...TaglibURL", see existing cases, just search for "String taglib".
LPS-32745 Lang keys
LPS-32548
LPS-32548 Test Formatting
LPS-32548 Test Formatting
LPS-32815 Ability to search BookmarksFolder in the search portlet
LPS-32815 Ability to search JournalFolder in the search portlet
LPS-32343 Ability to search DLFolder in the search portlet
LPS-32815 Make JournalFolder and JournalFolder API consistent
LPS-32815 simplify logic for folder icons