Typo-Atom patch
Seven months ago, I got Jim Weirich to make a change to builder.
Monday, I got David Heinemeier Hansson to incorporate that change into Rails.
Today I noticed that the title for this entry showed up as
Associations aren't :dependent => true anymore
Time to kick it up a notch. This time I’m armed with a patch. A patch that includes test cases.
Along the way, I fixed a number of other things. Summaries placed in content. And enclosures without a length.
I also noticed that the function tests can make use of a local copy of the Feed Validator. Sweet. It should surprise no-one that I have a local copy of the Feed Validator on my machine.
Apparently, these tests haven’t been run for a while, as they don’t pass. Invalid itunes elements, invalid authors, and other errors abound.
Open offer: if this patch is accepted, I’ll fix those issues next.
And I’d like to talk to somebody about xml:base...
Ticket #840
Okay, colour me an embarrassed shade of, um, embarrassment. I could have sworn I’d got feedvalidator installed and being run every time I ran the tests.
Except, it turns out that I’d done everything except make the feedvalidator binary executable.
I’ll be in the corner, wearing a dunce’s cap and generally feeling like an idiot.
Apart from that, thanks for an exemplary patch. As soon as rails gets a 1.1.3 release we shall be applying the patch with all speed.
Posted by Piers Cawley atPond Envy
My first recommendation aligns with Aristotle’s: diversify. I didn’t do any Perl in the past week, but I did do Python, PHP, Ruby, and JavaScript. In pond size terms, PHP is huge and growing. Ruby is nascent but exploding. My second recommend... [more]Trackback from Sam Ruby at
Typo - Bug #746: [PATCH] Atom 1.0 issues
Those are not expected failures. Those errors exist with or without this patch. If you upgrade to the latest Feed Validator, you will find more failures. I’ve made an open offer ... if this patch (potentially modulo the one line that I’ve agreed to...Excerpt from Typo bug tracker at
Sigh. NNW displays the content of this entry wrong.
Instead of:
It displays:
I was very confused until I clicked through to the article. :-/
Posted by Bob Aman at