It’s just data

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

on Planet Intertwingly.

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

Sigh.  NNW displays the content of this entry wrong.

Instead of:

Associations aren't :dependent => true anymore

It displays:

Associations aren't :dependent => true anymore

I was very confused until I clicked through to the article.  :-/

Posted by Bob Aman at

Bob, it worked fine in NNW 2.1b28 here.

Posted by Simon Fell at

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 at

Pond 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

Add your comment