I am concerned that you are just checking for a group membership for the T: drive not anything related to @primaryGroup and then attempting to use @primarygroup for S:.

Code:
; Primary Group Mappings
Select
  Case @PrimaryGrpoup = "Roberts Estimating"
    Use t: "\\hyperion\estimating$$" 
  Case @PrimaryGrpoup = "Roberts AC Full Permissions"
    Use t: "\\hyperion\airconditioning$$"
  Case @PrimaryGrpoup = "Roberts Mechanical"
    Use t: "\\hyperion\mechanical$$"
  Case @PrimaryGrpoup = "Roberts Electrical"
    Use t: "\\hyperion\electrical$$"
  Case @PrimaryGrpoup = "Roberts IT"
    Use t: "\\hyperion\apps$$"
  Case "Roberts Reception"
    Use t: "\\hyperion\reception$$"
EndSelect

If InGroup ( "Roberts Accounting" )
  If @PRIMARYGROUP <> "Roberts Accounting"
    Use s: "\\hyperion\accounting$" 
  EndIf
EndIf
If InGroup ( "Roberts Estimating" )
  If @PRIMARYGROUP <> "Roberts Estimating"
    Use s: "\\hyperion\estimating$" 
  EndIf
EndIf


But even this could have the effect of making multiple attempts to map S: if the users is in more than one group.
_________________________
Home page: http://www.kixhelp.com/hb/