mcchan1/adminFee-4.sol

If we're doing the iteration over the tokens ourselves, shares are not needed anymore.
Also, we're currently making a few changes in and around the loop, so, for simplicity, here's a version of this function as I would write it.
I removed the comments. Of course, you can add them again, as you see fit.
Most importantly: The code is untested!

This version iterates over all whitelisted tokens. It is not impossible to write a more sophisticated version of this function that receives as input a list of tokens, so that some tokens can be excluded. However, there would be some subtle gas-related points to consider, so I suggest to stick to this version.

The final code is quite short and should be pretty self-explanatory, but if there are any questions, please feel free to ask.

contract Moloch is ReentrancyGuard, Ownable {
    using SafeMath for uint256;

    //....REST IS SAME AS MOLOCH DAO CODE. 

    uint256 constant paymentPeriod = 90 days;
    uint256 public lastPaymentTime = summoningTime;

    function withdrawAdminFee(uint256 adminFeeDenominator, address laoFundAddress) public nonReentrant onlyOwner {
        require(adminFeeDenominator >= 200);

        require (now >= lastPaymentTime.add(paymentPeriod), "90 days have not passed since last withdrawal");
        lastPaymentTime = now;

        for (uint256 i = 0; i < approvedTokens.length; i++) {
            address token = approvedTokens[i];
            uint256 amount = userTokenBalances[GUILD][token] / adminFeeDenominator;
            if (amount > 0) { // otherwise skip for efficiency
                unsafeInternalTransfer(GUILD, laoFundAddress, token, amount);
            }
        }
    }
}

Without the laoFundAddress parameter... and having it transfer back to owner/msg.sender...

    using SafeMath for uint256;

    //....REST IS SAME AS MOLOCH DAO CODE. 

    uint256 constant paymentPeriod = 90 days;
    uint256 public lastPaymentTime = summoningTime;

// @param adminFeeDenoimnator must be >= 200. Greater than 200, will equal 0.5% or less of assets.  
//This results in Token Assets Under LAO/adminFeeDenominator

    function withdrawAdminFee(uint256 adminFeeDenominator) public nonReentrant onlyOwner {
        require(adminFeeDenominator >= 200);
        require (now >= lastPaymentTime.add(paymentPeriod), "90 days have not passed since last withdrawal");
        lastPaymentTime = now;

        for (uint256 i = 0; i < approvedTokens.length; i++) {
            address token = approvedTokens[i];
            uint256 amount = userTokenBalances[GUILD][token] / adminFeeDenominator;
            if (amount > 0) { // otherwise skip for efficiency
                unsafeInternalTransfer(GUILD, msg.sender, token, amount); //transfer tokens to msg.sender/Owner
            }
        }
    }
}```

The main change looks good to me.
But note that in the line uint256 amount = userTokenBalances[GUILD][token] / adminFeeDenominator; you changed the single / to //. The original version was correct, because this is intended to be a division.

thank you... also if we want .5% which is more accurate: >=200 or >=199

In this version, it's >= 200, because you're not minting new shares/loot.