githubbob42 / mingle2github2

0 stars 1 forks source link

Quote Sync Pre-Check: Fails When Mobile User Does Not Have Read #5609

Open githubbob42 opened 5 years ago

githubbob42 commented 5 years ago

Mingle Card: 5972 Duplicate of https://liquidframeworks.mingle.thoughtworks.com/projects/back_office/cards/1499

The current work around is to turn off permission to the PreSyncController Apex class.

Steps to Reproduce

| | |
|-|-|
|**Version #**|

FieldFX™ 4.0.1.4128 on 9.6.3

Sync V4 on ~9.6.1

Managed Package: 1.232

   |
|**Hardware**|Laptop|
|**OS**|Windows 10|
|**Browser**|Windows Chrome 68.0.3440.106|
|**Username**|any|
|**Password**|any|
|ORG ID|00D0v0000008ngU|
|User ID|

**Nuverra Sandbox Logins in LastPass “Shared-Customer Production Logins\\Sandbox”**

**Sys Admin User:** smercer@nuverra.fieldfx.com.training

**Mobile Test User:** smercer.mobile@nuverra.fieldfx.com.training

Feel free to test with these users to reproduce, but please let Scott Mercer know before changing settings, as this is org is being actively configured.  This org is currently used for configuration only, despite having the “training” sandbox name.

   |
|RayGun Error ID| |
  1. Back Office: Configure Permissions for Mobile User so they do not have read access to Quote or Quote Item.
  2. Back Office: Ensure Mobile user has access to the Apex classe: FX5.PreSyncController
  3. Mobile: Log into Mobile as user that should not have access to Quote.

Expected Result

Mobile users should be able to sync without having read access to Quote.  Many implementations do not use Quoting in Back Office or Mobile.  For these implementations, Mobile users should not see the “Quote” icon in the left navigation bar.  Adding Quote to the left navigation bar can be confusing to some users, especially if the “Quote” icon doesn’t do anything.

Actual Result

Pre-Sync check returns an error when Mobile user does not have access to Quote.  This is ONLY when the Mobile user has access to the new FX5.PreSyncController.  If the Mobile user does not have access to this class, the Sync Pre-Check rolls over to the default pre-sync check.  The old pre-sync check does not return an error when the user does not have read to Quote.  If you give the user READ to FX5__Quote__c and FX5__Quote_Item__c, new new Pre-Sync check will let you in.

NOTE: ANY client that does not use quoting will experience this error in Mobile if they follow the package upgrade instructions and grant all FX5 classes to their “FX Apex and VisualForce” Permission Set.  This issue does not occur on the current push upgrade Managed Package, which is 1.228 from July 10.

Analysis

Related Cards

Impact Analysis

Developers: Fill in this area during code review.

Test Plan