Sorry, I haven't had a chance to test the new release of the module yet, as we've moved on to a different project. The SQL errors I reported previously did not affect what we were doing, since we only needed to update regular nodes (not CCK).
I did want to report one other, slightly more significant, error that we experienced, though:
If a find&replace is made in a node, the node teaser somehow gets set to be the full node body. It completely ignores the
that we were using to manually force a teaser break & prevent parts of HTML tables from showing up without their closing tags. This means that I have to go through the site after the node find&replace is finished to find which nodes have lost their
.
Simply saving the node again fixes the problem. Therefore, I think adding node_save to the module's code might do the trick, but it might be too expensive for queries that return matches in a lot of nodes. Maybe there's a better way.
Comments
Comment #1
regx commentedI just commited revision 1.3.2.8 which should fix the issue with the teasers.
Thanks for this issue, I wouldn't have caught that.
Comment #2
EvanDonovan commentedThanks. You're quick!
If you consider these issues to be fixed, you can feel free to close them. As I said I probably won't have time to test this module for a while, but it's good to know that these changes have been committed for the next time that we do a round of module upgrades.
Comment #3
regx commentedThe issue will automatically close in 2 weeks if there is no activity.
Without any confirmation I prefer to just leave them open.
Comment #4
Rhino commentedI just downloaded and used 5.x-1.3 - and presto all of my full nodes are spat out straight FULL, no teaser and
<!--break-->and then the rest inside. Good thing there's an "undo".(is it -dev that I should be using?)
Comment #5
Rhino commentedDev works fine, sorry for misunderstaning which one to use - read the .module to see what was changed and got a laugh out of "Body count" as well.
Comment #6
EvanDonovan commentedFixed in 5.x-1.5 release.