This project is archived and is in readonly mode.
[PATCH] Parsing an xml file with multiple records and extra attributes (besides type) fails
Reported by Sidu Ponnappa | September 16th, 2009 @ 03:33 PM
This issue occurs when a collection is serialised to xml, and has attributes other than type='array' on the root node (for example is you choose to suppy page/per-page information for pagination as root attributes). This completely breaks ActiveResource when dealing with paginated collections.
I've taken the patch against rails/activesupport. I've changed Hash#typecast_xml_value to ignore extra attributes when 'type' is present. The sample data to reproduce the issue is included as part of the specs.
Comments and changes to this ticket
- Title changed from [Patch] Parsing an xml file with multiple records and extra attributes (besides type) fails to Parsing an xml file with multiple records and extra attributes (besides type) fails
- Tag changed from 2.3.6, activesupport, hash, xml to 2.3.6, activesupport, hash, patch, xml
Using the "patch" tag instead of prefixing the ticket title with "[PATCH]" to make sure patched tickets end up in the open patches bin. :)
- State changed from new to open
- Assigned user cleared.
- Tag changed from 2.3.6, activesupport, hash, patch, xml to activesupport, hash, patch, xml
- Importance changed from to Low
Sidu, the patch doesn't apply any more. Can you rebase/rewrite it and follow http://rails.lighthouseapp.com/projects/8994/sending-patches so that you get credit? :)
Also when the new patch comes, we need more then just a '+1'. If you are reviewing the patch please do comment on the code, whether it solves the problem and make sure the tests fail before the fix is applied.
- Title changed from Parsing an xml file with multiple records and extra attributes (besides type) fails to [PATCH] Parsing an xml file with multiple records and extra attributes (besides type) fails
This issue still persists in current rails head. And as the old patch is no more applicable, I am resubmitting the patch for latest code.
Good Puma shoes sale information for Puma shoes fans: Yesterday, German sportswear maker Cheap Running Shoes purchased effectively lot of money Brands. The transaction entails the Cobra brand, too as linked inventory, intellectual house and endorsement contracts. It also helps make puma speed cat Shoes . They provide you with the utmost game and support all through the feet.You can happen around several types of basketball sneakers such as high-top,low-top and mid-top.The Puma shoes online and boots give you best ankle support will support want you to obtain steadiness.
As we konw,Trail Puma Drift Cat Shoes black, Trail Puma future cat ShoesWhite Black? 50 many years backshore are actually released within a great accomplishment. Trail Running Shoes speed cat shoes?go in add-on to King Pele into planet tournament finals, bottom tennis discs matchrs within pasture inhabit an vital area in hold out in Wimbledon Dun; Its working firms have premier manufacturers and major marketplace positions in distilled spirits, residence and security, and golf products. The reply go all through the Cheap puma shoes on sale-here you are able to happen around several distcoun.puma shoes on sale .
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>