This Trigger that I have written isn't working correctly.  What i am trying to do is get a list of the users that includes ID and Salesman Number.  Then I trying to update the account with the new user ID.  Currently nothing is changing, once I click edit and I don't know why.

thanks.

kevin

trigger AccountOwnerUpdateTrigger on Account (before insert, before update) {

    // this map will contain a list of Accounts with the same salesman #
    Map<String, List<Account>> AccountOwnerMap = new Map<String, List<Account>> ();
    for( Account Acc : trigger.new ) {
        // create the list if it doesn't already exist
        List<Account> AcctList = AccountOwnerMap.get( Acc.Ownerid );
        if( AcctList == null ) {
            AcctList = new List<Account>();
        }
        AcctList.add( Acc );
        AccountOwnerMap.put( Acc.OwnerID, AcctList );
    }

    // get list of CURRENTLY ACTIVE Users using the field Salesman_Number__c
   
    List<User> UserList = [
                        SELECT ID, Salesman_Number__c
                        FROM User
                        WHERE ID IN :AccountOwnerMap.keySet() AND IsActive = True ];
              
                                          
    // go through the User List and get a list of Accounts that have the same salesman
    for( User User : UserList ) {
        List<Account> AcctList = AccountOwnerMap.get( User.ID );

        // assign the Account Owner for each Account in the list
        for( Account Acc : AcctList )
        Acc.Ownerid = User.ID;
        }
}