#microformats 2017-04-28
2017-04-28 UTC
# 
ben_thatmustbeme I think there are a lot more of those value errors with the tests
# 
KartikPrabhu possibly
[kevinmarks], tantek and GWG joined the channel
# 
gRegorLove ben_thatmustbeme: Can you add them to https://github.com/microformats/tests/issues/58? I can rename the issue to be more general
# gRegorLove Fixing it in php-mf2 now, could use more test cases
# 
@lamestdomo @usasmitty I write some mean semantic markup with microformats if you want something to whack off about (twitter.com/_/status/857776150512373761)
[chrisaldrich] and nitot joined the channel
# ben_thatmustbeme gRegorLove: sure, I plan to go through all the tests in failing one by one, and I'm pretty sure I saw some others already. I have other issues I'll likely be adding too like the -v2 folder had done mf1 in it, should just move those
# ben_thatmustbeme Also I know there are parsing rules that don't seem to be tested, so i may try to add some more for completeness
KartikPrabhu, [tamaracks], [chrisaldrich] and nitot joined the channel
# gRegorLove PR for the child "value" issue in php-mf2: https://github.com/indieweb/php-mf2/pull/113
[chrisaldrich], [tamaracks], barpthewire, tantek, dede1, [colinwalker], adactio, gRegorLove, [kevinmarks], nitot, rodolfojcj and sknebel joined the channel
# ben_thatmustbeme vcard is mf1 is it not?
# ben_thatmustbeme basically that one should be in the -v1 or -mixed folder? https://github.com/microformats/tests/blob/master/tests/microformats-v2/rel/duplicate-rels.html
nitot joined the channel
# ben_thatmustbeme well you can make a rel folder in the -v1 folder
# ben_thatmustbeme unless the rel stuff is only v2
# ben_thatmustbeme in which case it should be in -mixed/rel
[kevinmarks] joined the channel
# 
[kevinmarks] We didn't explicitly define rel parsing until 2 I think.
# [kevinmarks] Rels are the oldest microformats (xfn) but having a specific format to parse them too was more recent
# ben_thatmustbeme another question is why one of the tests has an alternates:
# ben_thatmustbeme oh neverment, that was already done, aparently the npm version i pulled wasn't up to date
# ben_thatmustbeme neverment? good one fingers, good one
nitot, [cleverdevil], [kevinmarks] and gRegorLove joined the channel
# ben_thatmustbeme oh god
rodolfojcj joined the channel
# ben_thatmustbeme this seems a bit overcomplex
# ben_thatmustbeme also not entirely clear on much of it
rodolfojcj, [colinwalker], [chrisaldrich] and tantek joined the channel
# gRegorLove datetimes are hard, heh
# 
Loqi tantek: ben_thatmustbeme left you a message 22 hours, 38 minutes ago: hitting what is either an error in the mf2 tests and a bug in php-mf2 or something missing in the spec and a bug in mf2py. children elements seem to be getting a value: set, but not sure why. https://github.com/microformats/tests/blob/master/tests/microformats-v2/h-card/nested.html
# ben_thatmustbeme i've been going through finding minor issues in the tests
# ben_thatmustbeme SHOULD be no value there https://github.com/microformats/tests/blob/master/tests/microformats-v2/h-card/nested.json
# ben_thatmustbeme php-mf2 puts a value there as well
# gRegorLove Yes, my PR last night will fix that.
# gRegorLove mf2py has it correct
rodolfojcj joined the channel
# gRegorLove All this mf2 parsing progress is exciting
# gRegorLove One of the nerdier things I've said recently
rodolfojcj and barpthewire joined the channel
# ben_thatmustbeme if the microformats property expects a simple string, enumerated value, or telephone number, then the values extracted from the value elements should be concatenated without inserting additional characters or white-space.
# ben_thatmustbeme http://microformats.org/wiki/value-class-pattern says "Where there are multiple descendants of a property with class name of value (multiple value elements)
# ben_thatmustbeme oh wait, it does that
# ben_thatmustbeme gah, why is the version in NPM so out of date
# ben_thatmustbeme https://github.com/microformats/tests/blob/master/tests/microformats-v2/h-card/impliedurl.json this one seems to get http://exmple.com out of nowhere
[chrisaldrich] joined the channel
# KartikPrabhu ben_thatmustbeme: I think it is implicitly using example.com to make URLs absolute
# ben_thatmustbeme hmm, seems kind of odd since there is a base tag for that in a few of them
# ben_thatmustbeme but yes, it looks like its doing that
[cleverdevil] joined the channel
# KartikPrabhu ben_thatmustbeme: yeah the base tag should be explicit in the tests
# ben_thatmustbeme or at least the test sweet should indicate that it uses that
# KartikPrabhu yup
# ben_thatmustbeme also another value for a child h-* in that example as well
# KartikPrabhu <sigh>
# gRegorLove Captured that one in the gh issue
# ben_thatmustbeme i'm making not of all of these
# ben_thatmustbeme i've moved over to using a copy of the GH repo so that i am not dealing with the out of date npm package
# ben_thatmustbeme and so any changes i make i can see via git diff
# KartikPrabhu ben_thatmustbeme: cool, maybe you can update the tests too then
# ben_thatmustbeme eep, h-* is case sensitive
# ben_thatmustbeme will do
# ben_thatmustbeme as soon as i believe i have found them all
# KartikPrabhu I don't think there should be any uppercase mf2 objects/properties
# ben_thatmustbeme i just hit that one
# KartikPrabhu yeah so it is correct I think.
# KartikPrabhu Adr is not an actual microformat since all microformats have lowercase letters
# ben_thatmustbeme so only match h-[a-z-]+ not [hH]-[a-zA-Z-]+
# KartikPrabhu interesting, mf2py rejects the H-* but not the h-Adr hmmm
# ben_thatmustbeme is that true for properties as well
# KartikPrabhu looking at the wiki
# gRegorLove Yes
# KartikPrabhu gRegorLove: so mf2 parsing should ignore uppercase objects/properties?
# ben_thatmustbeme i feel like i have seen that used in the wild though
# gRegorLove And numbers
# ben_thatmustbeme The "*" for root (and property) class names consists only of lowercase a-z and '-' characters.
# KartikPrabhu aah
# KartikPrabhu so mf2py fails to do this
# ben_thatmustbeme no not numbers
# gRegorLove Related discussion / resolution: http://microformats.org/wiki/microformats2-parsing-issues#ignore_u-camelCase_properties
# KartikPrabhu and so does php-mf2
# gRegorLove Sorry, was replying to Kartik on that. "ignored uppsercase... and numbers"
# gRegorLove *ignores
# KartikPrabhu yeah
# gRegorLove Hm? Should be fixed in php-mf2....
# KartikPrabhu gRegorLove: was trying on pin13 which might not have latest
# gRegorLove Ah, yep.
# ben_thatmustbeme can things end in a - ?
# ben_thatmustbeme h-something-
# ben_thatmustbeme it seems to be allowed
# 
@softwarereview7 Wordpress Implementation of Google Microformats by dansavoie #Freelance #Job https://www.freelancer.com/projects/gerri_13070488.html … https://twitter.com/i/web/status/858053332509773824 (twitter.com/_/status/858053332509773824)
# 
@CreativeJobFeed Wordpress Implementation of Google Microformats by dansavoie https://www.freelancer.com/projects/Matthewall_13070488.html #creativejobs #jobs (twitter.com/_/status/858053246002151424)
# 
@FreelanceDirect Wordpress Implementation of Google Microformats by dansavoie #Freelance #Job https://www.freelancer.com/projects/gerri_13070488.html (twitter.com/_/status/858052877708865537)
# 
@latestprojects Wordpress Implementation of Google Microformats by dansavoie https://www.freelancer.com/projects/altfuns_13070488.html (twitter.com/_/status/858052689418215425)
# KartikPrabhu yeah, nothing seems to forbid those
# ben_thatmustbeme so whats more confusing is that test result
# ben_thatmustbeme its ignoring anything under a H-adr
# ben_thatmustbeme seems like its recognizing it as a h-* and then checking the case and throwing the entire thing away
# ben_thatmustbeme rather than just ignoring the class and parsing down
# KartikPrabhu yeah that is correct
# KartikPrabhu oh! hmmm
# gRegorLove H-adr isn't a valid root, why would it parse down?
# KartikPrabhu so if there were a h-something inside the H-adr
# ben_thatmustbeme oh, heh, yeah
# ben_thatmustbeme i'm thinking this is inside a root
# KartikPrabhu that makes sense
# ben_thatmustbeme found a few others
# gRegorLove ben_thatmustbeme++
[chrisaldrich], aspis1, [dgold], tantek, [kevinmarks], [cleverdevil], nitot and gRegorLove joined the channel