Ticket #159 (new enhancement)

Opened 3 years ago

Last modified 2 years ago

Default 'type' for ajax set to 'GET'

Reported by: admorphit Owned by: jf.hovinne
Priority: trivial Milestone: 1.x
Component: editor Version: trunk
Keywords: ajax type get Cc:

Description

I've been trying to implement WYMeditor in a project which already implements JQuery. I use the $.ajaxSetup method to setup my ajax requests with type attribute set to POST as default. Initially, I had trouble getting WYMeditor to work because ajax requests by WYMeditor were all POSTing to IIS which were then rejected. I fixed the problem by setting type:'GET' on all ajax requests made by WYMeditor.

I'm just suggesting maybe the following lines should be changed in the next release to explicitly use the GET request?

Line 1100: eval(jQuery.ajax({type:'GET', url:this._options.langPath

Line 1307: CssParser.parse(jQuery.ajax({type:'GET', url: this._options.stylesheet,async:false}).responseText);

Line 1399: eval(jQuery.ajax({type:'GET', url:this._options.skinPath

Change History

comment:1 Changed 2 years ago by mr_lundis

  • Priority changed from major to trivial
  • Milestone set to 0.5

This is somewhat of an edge case - chances are that we'll step on someone else's toes by changing this.

Leaving the ticket open until a decision is made.

comment:2 Changed 2 years ago by mr_lundis

  • Milestone changed from 0.5 to 0.5.x
Note: See TracTickets for help on using tickets.