reordering bug #1

Merged
merged 1 commit into from Apr 3, 2012

Conversation

Projects
None yet
4 participants
@ghost

ghost commented Mar 14, 2012

may have fixed something guys... no intension to cause disruption.. this seems to run better. any thoughts otherwise heres mah patch. thx for the help!!

@cyrill62

This comment has been minimized.

Show comment Hide comment
@cyrill62

cyrill62 Mar 19, 2012

Collaborator

can you make a quick test with test::unit which reveal the bug and his fix ?

Collaborator

cyrill62 commented Mar 19, 2012

can you make a quick test with test::unit which reveal the bug and his fix ?

@ghost

This comment has been minimized.

Show comment Hide comment
@ghost

ghost Mar 27, 2012

dont have time to make a test right now. we are using this code in production with the patch. i tested this by hand during patching with a jquery sortable. the list was reordering improperly obviously when refreshed. i patched the behavior out and then committed. we are using the code live with no problems. thanks

ghost commented Mar 27, 2012

dont have time to make a test right now. we are using this code in production with the patch. i tested this by hand during patching with a jquery sortable. the list was reordering improperly obviously when refreshed. i patched the behavior out and then committed. we are using the code live with no problems. thanks

webpulser added a commit that referenced this pull request Apr 3, 2012

@webpulser webpulser merged commit 5292c4b into webpulser:master Apr 3, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment