Whenever I go to the Calais tab on a node and hit "save" my URL alias is stripped and the node is only identified as node/

This happens when both "automatic alias" is check and unchecked. Using version 6.x-1.4 of pathauto.

This makes it incredibly inconvenient to edit the Calais tags since you have to go back and redo the URL alias after you save them.

CommentFileSizeAuthor
#3 opencalais-897630-3.patch1.5 KBfebbraro
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

To clarify, the "node/" would be, for example, www.example.com/node/12345 or www.example.com/node/12

ie. as a normal node with no alias at all.

lavamind’s picture

lavamind CreditAttribution: lavamind commented

I have the same problem. Error messages appear when this happens :

    * warning: Invalid argument supplied for foreach() in /var/alternc/html/u/uclweb/www.causecommune.net/drupal/sites/www.causecommune.net/modules/boost/boost.module on line 686.
    * warning: Invalid argument supplied for foreach() in /var/alternc/html/u/uclweb/www.causecommune.net/drupal/sites/www.causecommune.net/modules/filefield_paths/modules/filefield.inc on line 50.
febbraro’s picture

febbraro CreditAttribution: febbraro commented
FileSize
opencalais-897630-3.patch1.5 KB

Thanks for the bug report. I have a fix in place, but it is intermingled with some other bigger changes I'm making. I will try my best to get a release pushed out today or tomorrow. Thanks for your patience.

In the meantime, here is a patch that should work (it contains some other fixes too)

febbraro’s picture

febbraro CreditAttribution: febbraro commented
Status: Needs work » Fixed

This was committed yesterday and should be in a dev release now. Looking for an official release this week.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

Thanks. Looking forward to it.

lavamind’s picture

lavamind CreditAttribution: lavamind commented

I upgraded to the current -dev release, there's no more error message but now it seems tags aren't saved at all...

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

kking’s picture

kking CreditAttribution: kking commented

The dev release has been updated as of late yesterday. This should hold onto the URL alias and save the tags.

Please give it a try: http://drupal.org/node/415802

kking’s picture

kking CreditAttribution: kking commented
Status: Closed (fixed) » Fixed
lavamind’s picture

lavamind CreditAttribution: lavamind commented

Latest dev release fixed the problem for me. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

sstose’s picture

sstose CreditAttribution: sstose commented

I found this thread, and report the same issue still occurring, with both the latest dev version and 6.x-3.5. Saving Calais tab strips URL requiring one to return and re-save on Edit tab (this time by manually having to check "create automatic alias" checkbox--which otherwise occurs by default).

I realize this is closed, but didn't want to create another thread.

Many thanks.

SeanJA’s picture

SeanJA CreditAttribution: SeanJA commented

Same problem here, we only just started using open calais and this happened for both of the tests that we did on two different nodes.

rhavill’s picture

rhavill CreditAttribution: rhavill commented

SeanJA and sstose, have you tried the latest stable version (6.x-3.6)? According to the release notes, this bug was fixed in that version.

jrstmartin’s picture

jrstmartin CreditAttribution: jrstmartin commented

I can confirm that I had this issue in 6.x-3.5. After upgrading to 6.x-3.6, it is fixed. Thank you and Merry Christmas!

nickgunman’s picture

nickgunman CreditAttribution: nickgunman commented

The patch didn't work for me, It didn't remove the path alias, but it didn't save the new terms. What I did was the following:

  • at line 149: + $complete_node = node_load($nid);
  • at line 152: + $node->path = $complete_node->path;
  • at line 196: +$complete_node = node_load($node->nid);
  • at line 197: +$node->path = $complete_node->path;

That is working for me with version 6.3.5

sstose’s picture

sstose CreditAttribution: sstose commented

I had the dev version 6.x-3.6 loaded when I wrote the earlier comment. Today I downloaded the same version, now the stable recommended 6.x-3.6, and all is solved. Thank you!