Closed (works as designed)
Project:
Feeds
Version:
7.x-2.0-alpha4
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
20 Dec 2011 at 00:12 UTC
Updated:
20 Dec 2011 at 18:25 UTC
I started using the feeds module to import from csv and create nodes..
Let's say, I have imported 3 nodes from the CSV file in drupal.
Now with "Replace existing nodes" option checked if I reimport the CSV file but missing 1 entry , the node that was previously has been created is not deleted...Instead I get the message "There are no new nodes"..
I check in the database in feeds_item table and I see that there are still 3 items
Shouldn't the node that corresponds to the missing entry in the CSV file be deleted?
I just started working with feeds and don't know which exactly is the expected behaviour..But it seems to me like a bug..
Thank's Loukas
Comments
Comment #1
colanNo, because that node wasn't replaced. The only nodes that get replaced are those listed in the CSV. The intention is not to delete nodes not listed in the CSV.
Comment #2
lsdoc77 commentedLooking at the issues list I found this thread http://drupal.org/node/661314 which looks like the functionality I had in mind.. So I'll have a better look (it is a quite big thread 8-) ) at this node , since it is something that I will need in a new project of mine..
Right now I can achieve this function by comparing the two csv (the original and the altered) files manually (eg with diff command in bash linux), then importing the newer csv with the option "update existing nodes" and after remove the nodes that are missing (based on the output of the diff command) by hand..
It seems to me that a fourth option of "Sync Nodes" in the settings page of the node processor should be very useful.
I will also have a look at the code and try to implement my own* pluggin Node processor with this option but my php skills are not professional, so cannot promise anything..
Thank's for the reply
Loukas
*I already managed to implement my own parser to convert a csv with greek charachters to latin charachters :-)
Comment #3
colanYes, makes sense as a new feature request. Thanks for checking the queue first. It's getting huge. ;)