Hi Gordon,
The recommended place to add code for setting the file name, or owner, is
in the WebHandler procedure, ProcessLink method, before Parent Call.
Where you have it is probably ok, unless of course, in embed code, you call a procedure that uses Access:File.Open rather than p_web.Openfile. Given that that's a probably use case, I'd recommend moving the code to the "convention" position.
Yes the extra code is there by design. The whole point of the _OpenFile (et al) methods is to provide a consistent interface between ABC and Legacy apps. The Parent call is generated by the standard class code - the ABC bit is slipped in before it in the case where you have an ABC app.
Cheers
Bruce