Add support for xml processing instructions in atom_feed_helper

I think it should be use hash format :

:instruct => { :target => {attribute => value} }

In that case, the array should be at the target level, not the instruct level. target => [{attribute => value}, {attribute => value}]. Both forms should be taken, though. Since the normal way is not to have two targets.

Looks nice. But I'm getting one test failure :

  1) Failure:
test_feed_xml_processing_instructions_duplicate_targets(AtomFeedTest)
    [template/atom_feed_helper_test.rb:244:in `test_feed_xml_processing_instructions_duplicate_targets'
     template/atom_feed_helper_test.rb:254:in `with_restful_routing'
     ./../lib/action_controller/test_process.rb:508:in `with_routing'
     template/atom_feed_helper_test.rb:250:in `with_restful_routing'
     template/atom_feed_helper_test.rb:241:in `test_feed_xml_processing_instructions_duplicate_targets'
     ./../lib/../../activesupport/lib/active_support/testing/setup_and_teardown.rb:94:in `__send__'
     ./../lib/../../activesupport/lib/active_support/testing/setup_and_teardown.rb:94:in `run']:
<"<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<?target1 a=\"1\" b=\"2\"?>\n<?target1 d=\"4\" c=\"3\"?>\n<feed xml:lang=\"en-US\" xmlns=\"http://www.w3.org/2005/Atom\">\n  <id>tag:www.nextangle.com,2008:/scrolls?id=feed_with_xml_processing_instructions_duplicate_targets</id>\n  <link type=\"text/html\" rel=\"alternate\" href=\"http://www.nextangle.com\"/>\n  <link type=\"application/atom+xml\" rel=\"self\" href=\"http://www.nextangle.com/scrolls?id=feed_with_xml_processing_instructions_duplicate_targets\"/>\n  <title>My great blog!</title>\n  <updated>2007-12-12T15:00:00Z</updated>\n  <entry>\n    <id>tag:www.nextangle.com,2008:Scroll/1</id>\n    <published>2007-12-12T15:00:00Z</published>\n    <updated>2007-12-12T15:00:00Z</updated>\n    <link type=\"text/html\" rel=\"alternate\" href=\"http://www.nextangle.com/scrolls/1\"/>\n    <title>Hello One</title>\n    <content type=\"html\">Something &lt;i&gt;COOL!&lt;/i&gt;</content>\n    <author>\n      <name>DHH</name>\n    </author>\n  </entry>\n  <entry>\n    <id>tag:www.nextangle.com,2008:Scroll/2</id>\n    <updated>2007-12-12T15:00:00Z</updated>\n    <link type=\"text/html\" rel=\"alternate\" href=\"http://www.nextangle.com/scrolls/2\"/>\n    <title>Hello Two</title>\n    <content type=\"html\">Something Boring</content>\n    <author>\n      <name>DHH</name>\n    </author>\n  </entry>\n</feed>\n"> expected to be =~
</<\?target1 c="3" d="4"\?>/>.

(from [42cbd710bb116998f92029adacb45698905a2e3b]) Add support for xml processing instructions in atom_feed_helper [#926 Add support for xml processing instructions in atom_feed_helper state:resolved]

Signed-off-by: Pratik Naik pratiknaik@gmail.com http://github.com/rails/rails/co...