Update SugarFolders.php Re:277

[pstevens71]

These changes are necessary to fix multiple problems with group folders display for users. Fix isgroup vs is_group error. Test if user has access to the group folder, ensure admin always sees the group folders in the arrays, and to remove child folders from the array as well as check for and manage duplicates so folders don't appear twice.

This change is for 7.13+

Description

Motivation and Context

How To Test This

This change is required in conjunction with a few others. See #277 I'll add a complete list of changes. User will be able to add group folders if they are given access via security group. Admin will see all folders.

NOTE: in conjunction with other changes, it will be necessary to re-save group inbound accounts. This is necessary so that the folders get properly marked as "is_group=1"

Types of changes

• [ x] Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Final checklist

• [x ] My code follows the code style of this project found here.
• My change requires a change to the documentation.
• [ x] I have read the How to Contribute guidelines.

[pstevens71]

Not sure how to resolve the conflict? This part is correct:

<<<<<<< patch-2
          $isGroup = $item['is_group'] ?? '';
            if ($isGroup == 1) {
                //added check if user has access to the inbound account, OR is admin and also check for duplicates
                if ((isset($userAccessibleInboundIds[$item['id']]) || is_admin($user)) && !in_array($item['id'], $groupFolderIdsAdded)) {
=======

It has to check if the user either created the account, is admin, or if the user has access to the security group.