My first impression: it works for me
I don't test it in depth but in the context of my original post it
works like a champ
Thanks Christophe and the others!!
On Jan 20, 12:38 am, Garito <[EMAIL PROTECTED]> wrote:
> Ok, thanks! I'll try it and ask if I have problems
>
> See ya!
>
> On Jan 19, 12:01 am, Christophe Porteneuve <[EMAIL PROTECTED]> wrote:
>
> > If you *do* need optional editorId behavior, use the attached patch.
>
> > If you never used a diff file before, here's how to apply it on *nix or
> > Windows with Cygwin:
>
> > * Open a shell
> > * Move to the directory with controls.js
> > * Run the following line
>
> > $ patch -p0 < path/to/your_download_of/optional_editorid_param.diff
>
> > --
> > Christophe Porteneuve aka TDD
> > [EMAIL PROTECTED]
>
> > [optional_editorid_param.diff]Index: controls.js
> > ===================================================================
> > --- controls.js (revision 8662)
> > +++ controls.js (working copy)
> > @@ -624,7 +624,8 @@
> > var params = this.options.callback(form, value) || '';
> > if (Object.isString(params))
> > params = params.toQueryParams();
> > - params.editorId = this.element.id;
> > + if (this.element.id)
> > + params.editorId = this.element.id;
> > if (this.options.htmlResponse) {
> > var options = Object.extend({ evalScripts: true },
> > this.options.ajaxOptions);
> > Object.extend(options, {
> > @@ -668,7 +669,6 @@
> > this._controls.editor.disabled = true;
> > var options = Object.extend({ method: 'get' },
> > this.options.ajaxOptions);
> > Object.extend(options, {
> > - parameters: 'editorId=' + encodeURIComponent(this.element.id),
> > onComplete: Prototype.emptyFunction,
> > onSuccess: function(transport) {
> > this._form.removeClassName(this.options.loadingClassName);
> > @@ -681,6 +681,8 @@
> > }.bind(this),
> > onFailure: this._boundFailureHandler
> > });
> > + if (this.element.id)
> > + options.parameters = 'editorId=' + encodeURIComponent(this.element.id)
> > : ,
> > new Ajax.Request(this.options.loadTextURL, options);
> > },
> > postProcessEditField: function() {
> > @@ -775,7 +777,6 @@
> > this.showLoadingText(this.options.loadingCollectionText);
> > var options = Object.extend({ method: 'get' },
> > this.options.ajaxOptions);
> > Object.extend(options, {
> > - parameters: 'editorId=' + encodeURIComponent(this.element.id),
> > onComplete: Prototype.emptyFunction,
> > onSuccess: function(transport) {
> > var js = transport.responseText.strip();
> > @@ -786,6 +787,8 @@
> > }.bind(this),
> > onFailure: this.onFailure
> > });
> > + if (this.element.id)
> > + options.parameters = 'editorId=' +
> > encodeURIComponent(this.element.id),
> > new Ajax.Request(this.options.loadCollectionURL, options);
> > },
>
> > @@ -813,7 +816,6 @@
> > this.showLoadingText(this.options.loadingText);
> > var options = Object.extend({ method: 'get' },
> > this.options.ajaxOptions);
> > Object.extend(options, {
> > - parameters: 'editorId=' + encodeURIComponent(this.element.id),
> > onComplete: Prototype.emptyFunction,
> > onSuccess: function(transport) {
> > this._text = transport.responseText.strip();
> > @@ -821,6 +823,8 @@
> > }.bind(this),
> > onFailure: this.onFailure
> > });
> > + if (this.element.id)
> > + options.parameters = 'editorId=' +
> > encodeURIComponent(this.element.id),
> > new Ajax.Request(this.options.loadTextURL, options);
> > },
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Ruby
on Rails: Spinoffs" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/rubyonrails-spinoffs?hl=en
-~----------~----~----~----~------~----~------~--~---
