Hi Tamar

>>It's confusing but it works. I could dispense with passing and receiving the array, which would make it clearer, or re-engineer it to make it properly encapsulated, which means changing the UDFs.
>>
>>Or just document the existing situation.
>>
>>What would you do?
>
>Fix it. Think about how long it took you to understand what's going on. Even if you document, it'll still be a mess.
>
>You don't have to fix all of it at once, but get rid of the most egregious problems.

The other issue is informing the client and asking for approval.