FCKeditor not saving drafts

I have not used newer versions of the fckeditor. Does the same occur with the previous version of the fckeditor module?

I will try and get a download and determine what is occurring but currently working on some other pieces so probably be monday before i get around to looking into this.

Thanks, will wait on your reply to this.

This was the old code

// Replace line 38 through 41 with this code please
if (typeof FCKeditor == 'object' && fckLaunchedTextareaId.length) {
for ( var i = 0; i < fckLaunchedTextareaId.length; i++) {
var text = FCKeditorAPI.GetInstance( fckLaunchedJsId[i] ).GetXHTML();
document.getElementById( fckLaunchedTextareaId[i] ).value = text;
};
};

//Replace 109 through 115 with this code please
// for FCKEditor
if (typeof FCKeditor == 'object' && fckLaunchedTextareaId.length) {
for ( var i = 0; i < fckLaunchedTextareaId.length; i++) {
var text = document.getElementById(fckLaunchedTextareaId[i]).value;
if (text) {
FCKeditorAPI.GetInstance(fckLaunchedJsId[i]).InsertHtml(text);
}
}
}

This should work for you with the versions.

Please let me know if it works, if it does i will re-place it again.

Honestly not sure, it has not been tested with the alpha version.

I will add that code back into the FCK Editor portion. Thanks for getting back to me on it.

I had the same issue using draft 6.x-1.1 and fckeditor 6.x-1.3-rc6 (FCKEditor 2.6.3)
I managed to fix it by changing Line 109 through 115 to:

  // for FCKEditor
  if (typeof FCKeditorAPI == 'object') {
    for( var i = 0 ; i < fckLaunchedJsId.length ; i++ ) {
      var fck = FCKeditorAPI.GetInstance( fckLaunchedJsId[i] );
      fck.SetHTML($('#edit-body')[i].value);
    };
  };

edit: in draft.js of course.

Thank you Miruoss. I made the change you suggested and everything works now. I'm on Draft 6.x-1.1 and FCKEditor 6.x-1.3-rc7.

But god did it suck to try to restore a loooooooong draft and have nothing come up, and then check the DB to see it was blank. Thank god I backed it all up in notepad++. :/

Thanks again!

Ok, spoke too soon... this KIND of works for me.

If I follow these steps (after adding the updated code above):
1) Create a node (story in this example) and save a draft.
2) Pull the draft up. All the fields populate correctly.
3) Select "Preview Content".

At this point I get the following error:
warning: Invalid argument supplied for foreach() in C:\Program Files\Apache Software Foundation\Apache2.2\htdocs\modules\taxonomy\taxonomy.module on line 70.

I also get an "Access is denied" error, and then the draft is deleted from the drafts table.

If I create a node that has 0 taxonomy references, I only receive the "Access is denied" error, but the outcome is the same (draft is deleted).

Has anyone else gotten this? I'm backing everything up so I haven't lost anything, luckily, but in case anyone else is using my configuration I just wanted to give the forewarning.

Thanks -

Ok so in case anyone's keeping score, a summary:

Draft: Draft 6.x-1.1
FCKEditor 6.x-1.3-rc7

Scenario 1: Default install of both allows me to save to the drafts table correctly. Everything looks fine. When I recall a draft node, the title populates, but no body content is displayed (though I see it in the table). If I preview the recalled draft node, I get an "Access Denied" error and the record in the draft table is deleted.

Scenario 2: Tweak the draft.js file exactly per Darren's recommendations above. A record in this case is saved to the drafts table, however the BODY tag in the data field is always empty. When I recall the draft node, the title is populated but obviously nothing is put in the body. Attempting to preview the draft causes the Access Denied error again, followed by the record being deleted.

Scenario 3: Change the "object" type in Darren's code to "function" per bsimon's suggestion. This works perfectly insofaras the node draft is stored properly in the database and recalled properly in the draft node. I'm able to save the draft multiple times in a single node and the changes are reflected correctly. So this is definitely the way to go (thanks bsimon).

But I still get the Access Denied error if I attempt to view a preview of a draft node (and the record is deleted from the DB)... is anyone else having this happen? I wouldn't even know where to begin with this. Permissions are set across the board for the admin account i'm testing on, and I can view the preview as long as I don't save it as a draft first... I'll bounce this into a different thread if noone here is sure, I don't want to hijack the OP.

Hmmm, that's definitely not what's going on from my side. I haven't modified the view and I'm just trying to preview the drafts immediately after they're created from the same author.

But also, something must be getting coughed up in the write process when I hit the preview button which is what's causing the drafts to get deleted.. I wonder if Drafts does a write before it attempts to preview? Just odd, but I guess good to know so I can be cautious of it...

Again, I had the same error. I tried to preview a draft and got "access denied" because the draft was deleted in the meantime. I fixed this by changing draft.module as follows:

Line 189: Add a ($form_state['values']['op'] != 'Preview') to the if-statement. The function then should look like this:

/**
 * Submission handler for when the node is submitted
 */
function draft_node_form_submit($form, &$form_state) {
  // check if we have a draft id associated with this submission if so we can remove it
  if (isset($form_state['values']['draft_id']) && is_numeric($form_state['values']['draft_id']) && $form_state['values']['op'] != 'Preview') {
    db_query("DELETE FROM {drafts} WHERE draft_id = %d", $form_state['values']['draft_id']);
  }
}

But then I had the problem that the form got filled twice, once by drupal (preview) and again by the draft module. I therefore packed lines 110 through 142 into another if-statement:

        if($form_state['post']['op'] != 'Preview') {
          if ($_GET['draft_id'] && is_numeric($_GET['draft_id'])) {
            $result = db_fetch_object(db_query("SELECT * FROM {drafts} WHERE draft_id = %d", $_GET['draft_id']));
            if ($result->uid != $user->uid) {
              drupal_access_denied();
              die();
            }
            drupal_add_js('$(document).ready(function() { setTimeout("Drupal.draft.populateForm();", 3000); });', 'inline');
            $settings['draft']['form_data'] = unserialize($result->data);
          }
          // adding the pertinent javascript files to the page
          drupal_add_js(DRAFT_PATH .'/jquery.field.js');
          drupal_add_css(DRAFT_PATH .'/draft.css');
          drupal_add_js(DRAFT_PATH .'/draft.js');
          // setting the javascript settings we will require in order to utilize saving the form
          $settings['draft']['form_id'] = 'node-form';
          $settings['draft']['autosave'] = 0;
          $settings['draft']['img_url'] = url('draft/images');
          $settings['draft']['interval'] = variable_get('draft_autosave_interval', 30000);
          $settings['draft']['url'] = url('draft/save');
          $settings['draft']['button_id'] = 'edit-save-as-draft';
          $settings['draft']['form_elements_ignore'] = variable_get('draft_form_elements_ignore', 'op|form_token|form_build_id');
          $settings['draft']['node_type'] = $node->type;
          // checking to see if autosave is enabled or not for this account
          if (variable_get('draft_autosave_'. $user->uid, 0)) {
            drupal_add_js('Drupal.draft.saveTimeout();', 'inline');
            // setting the autosave interval if the user has it set in their account since it can
            // over ride the administrator autosave timeout incase the user wishes it to be longer or shorter
            if (($interval = variable_get('draft_autosave_interval_'. $user->uid, 0))) {
              $settings['draft']['interval'] = $interval;
            }
            $settings['draft']['autosave'] = 1;
          }
          drupal_add_js($settings, 'setting');
        }

I hope this helps.

Michael

Im having the same problem. The code that miruoss posted at #18 did not change anything for me. The drafted nodes turns out to be empty.
Im using FCKeditor too.

#18 is an attempt to fix the preview functionality. Did you try the code in #11?

I have changed the code in draft.js and in draft.module like the suggestions in #11 and #18. Still getting the same results.
The draft can be found empty in ?q=draft/list and if I try to preview it from the draft I get a Access deny to that page...

Im using
FCKeditor 6.x-2.0-alpha5
Draft 6.x-1.1 (modified, see http://drupal.org/node/348165)
Core 6.10
Jolt-Cola 1 every hour

Have not tried the new FCKEditor but will be pushing to make updates to the Draft module in the coming weeks. Will be certain to make sure we have functionality for TinyMCE and FCKEditor and hopefully the latest versions will not break anything.

Hi Apollo,

Did you ever discover what created the taxonomy module error you are receiving above? I am using Draft module, however I'm getting the EXACT same error, but from a node type that is not using the Draft features...

Any pointers on how you got rid of it would be great!

Thanks,

Adam

Hi Adam, are you able to narrow down exactly what's causing the Taxonomy error? If you disable Draft, does the error go away?

I can't recall exactly what fixed the taxonomy error from my side, though I know it's no longer an issue. I'll try to help you troubleshoot where I can, talking through the problem may cause me to remember what the problem was and how it was resolved. ;)

Hi Apollo,

Actually... the error appeared to disappear for me also. I'm not quite sure how, but in these circumstances, and given that the code referenced in the taxonomy module appears to have no relevance to what I'm working on I'm going to pretend it never happened unless I see it again :) I'm actually working with the Workflow and Rules modules so... not quite sure how taxonomy was brought into it!

Anyway, thanks so much for getting back to me and apologies to the maintainer for slightly hijacking this thread! :)

All the best,

Adam

FCKEditor support now seems to be fully functional in the Draft module in CVS.

Closing out this ticket.