Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

JQPost appends arguments onto url #651

Closed
GoogleCodeExporter opened this issue Mar 25, 2015 · 2 comments
Closed

JQPost appends arguments onto url #651

GoogleCodeExporter opened this issue Mar 25, 2015 · 2 comments
Labels
auto-migrated JQuery Priority-Medium Type-Defect

Comments

@GoogleCodeExporter
Copy link 

I came across a problem with the JQPost this morning whereby it still appends 
arguments onto the url. My solution was the following although I don't claim to 
be an expert at this!

JQPost>>#arguments
   | arguments |
   arguments := OrderedCollection with: self url.
   self data collection isEmpty ifFalse: [ arguments add: self data ].
   self options
       at: 'complete'
       ifPresent: [ :value | arguments add: value ].
   self options
       at: 'dataType'
       ifPresent: [ :value | arguments add: value ].
   ^ arguments

Original issue reported on code.google.com by renggli on 25 Apr 2011 at 9:15
@GoogleCodeExporter
Copy link
Author 

Name: JQuery-Core-lr.116
Author: lr
Time: 25 April 2011, 2:34:15 pm
UUID: b5489524-87bc-469c-b303-ffd1df8b8b5b
Ancestors: JQuery-Core-lr.115

- fix get, get json, get script, and post ajax requests; they work mostly broken
- use #onSuccess: on these request types, not #onComplete: (which has changed 
in the jQuery code)

Name: JQuery-Tests-Core-lr.128
Author: lr
Time: 25 April 2011, 2:31:03 pm
UUID: 0516e42f-f445-4226-97ca-3a34557d3df2
Ancestors: JQuery-Tests-Core-pmm.127

- add tests for get, getscript and post ajax requests (they were broken)

Original comment by renggli on 25 Apr 2011 at 12:48

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Original comment by renggli on 25 Apr 2011 at 12:48

  • Changed state: Fixed
  • Added labels: ****
  • Removed labels: ****

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
auto-migrated JQuery Priority-Medium Type-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@GoogleCodeExporter