2018-05-26 UTC
[cleverdevil], [jgmac1106], [tantek], [Natris1979], tantek_, toomim, [dougbeal], tantek, barpthewire, KartikPrabhu, [chrisaldrich], tae and [quinnvinlove] joined the channel
[kevinmarks], [kim_landwehr], Kyle-K, barpthewire, jalcine, [miklb], [cleverdevil] and [grantcodes] joined the channel; Kyle-K left the channel
# 20:48 KartikPrabhu !tell [kevinmarks] Is it possible that I get permissions to close and label issues on the mf2py main repo?
# 20:48 Loqi Ok, I'll tell them that when I see them next
# 21:15 Loqi [kartikprabhu] #104 backcompat parsing changes original document
# 21:19 Zegnat I think most people are passing in an HTML string and getting parsed mf2 out, so they can’t run into this.
# 21:20 Zegnat But if you pass in a DOMDocument to the PHP parser, and then try to use the DOMDocument object again after parsing these same issues might occur.
# 21:20 KartikPrabhu yeah that seems reasonable. I haven't run into this in practice either
# 21:20 Zegnat Would need to test. Filing bug. Not something I want to start testing past 11pm :P
# 21:21 Zegnat Neh. Bugs don’t cost anything. Either someone beats me to testing, or I test it tomorrow.
[chrisaldrich] joined the channel
# 21:26 Loqi [Zegnat] #174 Do we change the DOMDocument instance that get passed in, and is this an issue?
# 22:37 KartikPrabhu gRegorLove: This is not for the parsed content.html Does php-mf2 change the original "DOMDocument" that the users gives it?
# 22:37 KartikPrabhu I have fixed the content.html thing for mf2py very recently (yesterday)
# 22:38 KartikPrabhu so in mf2py running parse a second time on the same "DOM" object gives wrong results
# 22:46 gRegorLove Confirmed if you pass in a DOMDocument to php-mf2 it's changed after parsing.
# 22:47 KartikPrabhu ok. don't know if that is good or bad yet. In php-mf2 it might not be an issue since the whole DOM is changed to mf2
# 22:47 gRegorLove Including the children, though, so a second pass shouldn't (I think) cause problems
# 22:48 gRegorLove Sounds like php-mf2 should definitely copy the DOMDocument and work on that
# 22:48 KartikPrabhu right if the backcompat rule that "ignore mf1 if mf2 is present" is used then it won't be an issue in second pass
# 22:48 KartikPrabhu I am also in favour of not changing things the user passes in explicitly
# 22:51 KartikPrabhu in mf2py that might need some funny code since BeautifulSoup does not support "deepcopy" i.e. making a separate duplicate
# 22:53 gRegorLove Yeah, I'm wondering if PHP is similar. I wouldn't have expected it to change the input DOMDocument since it's not by reference, but maybe it's a special case
# 23:02 gRegorLove Yeah, looks like object references don't contain the object, just the identifer so accessors can find it.
[cleverdevil] joined the channel
# 23:04 KartikPrabhu oh hey! I can make deepcopies of the whole doc too! just not of the elements :P
[grantcodes] joined the channel
KartikPrabhu joined the channel