Is this module only compatible with Drupal 6 core version of jQuery (v1.2.6)? I installed jQuery update and added (jQuery v1.3.2) to get some additional functionality, but the jEditable module ceases to update the field when I click the save button.

If this module is not compatible with the "newer" version of jQuery, is there a way to update it?

CommentFileSizeAuthor
#14 1043800-14.patch1.17 KBsuresh prabhu parkala
#11 jeditable-1043800-jeditable-appears-but-does-not-save-11.patch1.54 KBmrdalesmith

Comments

giorgosk’s picture

giorgosk
Location Chios, Greece 🇬🇷
commented
Title: jQuery Compatability » jeditable appears but does not save after hitting save (throbber does not appear)
Status: Active » Postponed (maintainer needs more info)

I have it working with JQuery UI 1.7.3 and jQuery Update 1.3.2
so its not a jquery compatibility issue

we need to find out what it is that it bugs it

here is my initial thought but closed the ticket #1040686: does not seem to work with VBO on integer field

I have also tried disabling all optimizations from admin/settings/performance
(css/js compression/minifying) and cleared cache but no luck

please report any findings

Shinzon’s picture

Shinzon commented

Can someone please, please fix this. Module is great but not working in my case. Can't save new data, after clicking on save nothing happened, just showing old input.

Dave.Ingram’s picture

Dave.Ingram commented

Here's some details I need:

1. Is firebug giving any error messages?

2. What type of field(s) have you tried this on?

3. Are you sure you have edit access to the node type and the node fields?

I am running this on a site that uses jQuery Update and jQuery UI and all works fine, so there must be something else going on.

kevin.mcnamee@mailbox.org’s picture

kevin.mcnamee@mailbox.org commented
Status: Postponed (maintainer needs more info) » Closed (won't fix)

Closing this as it has been a year since it was active. Reopen if the issue is still a problem.

svnindia’s picture

svnindia commented
Status: Closed (won't fix) » Active

me too facing the same issue..
After clicking save it show only old data..
If I fix it will give you the patch

svnindia’s picture

svnindia commented

I have tried with node_save instead of drupal_execute and it works

tinflute’s picture

tinflute commented

In reply to #3:
1. firebug give this response:

<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
<html><head>
<title>404 Not Found</title>
</head><body>
<h1>Not Found</h1>
<p>The requested URL /jeditable/ajax/save was not found on this server.</p>
</body></html>

2. I have tried this on a field types boolean, text, numeric, date with formatter jEditable textfield in my view.

3. Verified that I have permission to the node, view, and jEditable widgets.

I have the same bug: jEditable widget appears, but save has no effect, field reverts to original value. No throbber.
Note that I am running this on 7.x-1.x-dev version.

tinflute’s picture

tinflute commented
Version: 6.x-1.3 » 6.x-1.4

svnindia could you explain a little more what you did...
where in the code did you replace drupal_execute with node_save?

m.weinhold@module-7.com’s picture

m.weinhold@module-7.com commented
Issue summary: View changes

maybe the problem is that you use a url like www.my-domain.com/your-drupal-installation? If i use this it do not work, but if i use only my-domain than it saves all values.

leturan’s picture

leturan commented

m.weinhold theory is true : adding Drupal.settings.basePath + in front of all urls in just worked.
See below the changed lines :
drupal_jeditable.js:4 $('.jeditable-textfield', context).editable(Drupal.settings.basePath + '/jeditable/ajax/save', {
drupal_jeditable.js:11 $('.jeditable-textarea', context).editable(Drupal.settings.basePath + '/jeditable/ajax/save', {
drupal_jeditable.js:18 $('.jeditable-select', context).editable(Drupal.settings.basePath + '/jeditable/ajax/save', {
drupal_jeditable.js:19 loadurl : Drupal.settings.basePath + '/jeditable/ajax/load',

mrdalesmith’s picture

mrdalesmith commented
Version: 6.x-1.4 » 7.x-1.x-dev
Category: Support request » Bug report
Status: Active » Needs review
StatusFileSize
new jeditable-1043800-jeditable-appears-but-does-not-save-11.patch1.54 KB

Here's a patch to the js file for the 7x version as mentioned above.

katrien_w’s picture

katrien_w commented

Sorry, no success with this patch. Trying to save now, I get an ajax error.

nehajyoti’s picture

nehajyoti commented
Status: Needs review » Needs work

The patch doesn't get apply cleanly, hence marking it as needs work.

suresh prabhu parkala’s picture

suresh prabhu parkala commented
Status: Needs work » Needs review
StatusFileSize
new 1043800-14.patch1.17 KB

A re-rolled patch against the latest 7.x-1.x. Please review.