Description
mig5 is broken:
revstr is declared only in the tslists loop, in line 234revstr = '%08d' % rev so if tslist is empty, revstr is not defined later in line 256: f.write(revstr)
after defining it in line 229 revstr = '%08d' % rev, and running the script, pages with a text fiile and no backups are NOT copied.
If I define revstr = None in line 229, I get a call to {f.write(None)}. That shows that tslist is empty sometimes, and then revstr is not 000000001 as it should be, and of course the file is not copied in the loop.
Discussion
If tslist (time stamp list) is empty, then the data gathering steps couldn't get any revisions. Maybe there was no "text" file and also no "backup/*" files in the src page.
In patch-233, I added a condition not to create "revisions/" and "current" for such pages, attachments will be copied, if found. I hope this fixes it, but I didn't test it. If it still fails, provide directory listings of src page where it fails.
Plan
- Priority: Critical - loss of user data on migration
- Assigned to:
- Status: Closed