pkolarov / folders2flickr

Automatically exported from code.google.com/p/folders2flickr
7 stars 2 forks source link

Set of Enhancements #29

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Hi,
I have added the following enhancements to the base code:
- Support for updating Flickr photo (delete and upload again) if the file was 
modified locally in file system since it was uploaded to Flickr. It takes into 
account the last modified date from the file system and the date posted 
(uploaded) to Flickr.
- Added a LOT of log sentences to describe what the code is doing
- Support for running the process by folders, instead of first uploading all 
the files and then create the sets, now it does this two steps by folder. In 
this case if you have plenty of folders and files to upload you would see that 
the files are uploaded by folder and also included in sets as the folder is 
processed
- Support for stopping the application if a number of files uploaded has been 
reached
- Added logic to manage case when a file has been moved to another folder, for 
instance, if you have these files : C:\Pictures\A\1.jgg, C:\Pictures\A\2.jpg 
and C:\Pictures\B\3.jpg and run uploadr it will upload the files, create the 
sets and tag the files within also the file name (also added support to have a 
tag with the whole name plus extension). Now, if I move the file 1.jpg to folde 
B, next time that uploadr runs it will realize about this change and it will 
delete picture 1.jpg from Flickr and upload 1.jpg with the correct tags and it 
will be included in Set B instead of A.
- Support for grouping pictures in Flickr into a set called "LocallyDeleted" 
when a file has been deleted from file system and the corresponding picture is 
present in Flickr so users could do a manual delete from Flickr if they like it.

Attached is the .patch for SVN.

Regards and Thank you!

Original issue reported on code.google.com by gsfernan...@gmail.com on 8 Jul 2013 at 3:00

Attachments:

GoogleCodeExporter commented 9 years ago
Hi,
I like your enhancements. Unfortunately, I was not able to apply the patches. I 
used the following command in terminal and received the attached error messages:

patch -p0 -i ./folders2flickr.patch

Cheers

Original comment by tue...@gmail.com on 18 Jul 2013 at 2:50

Attachments:

GoogleCodeExporter commented 9 years ago
Hi, 

Thank you!

I have generated the svn patch by using Eclipse Subclipse plugin. I'll generate 
it again and post it here. I actually did more changes to this, some minor 
changes but still in the same philosophy (more logs, error handler when EXIF is 
not present in files, etc). I don't have the code here so I'd post it later 
Today or Tomorrow.

Regards

Original comment by gsfernan...@gmail.com on 18 Jul 2013 at 2:54

GoogleCodeExporter commented 9 years ago
Great to have these enhancements, and good that you are at least able to 
execute the code.
There are a lot of people struggling to use this software because of issue 24. 
Can you help with that one as well?

Original comment by pet...@gmail.com on 19 Jul 2013 at 10:49

GoogleCodeExporter commented 9 years ago
Thank you, I'm still fixing some issues that I found in the enhancements added. 
As soon as this code changes are stable I would like to upgrade the code to 
Python 3.x

Just watch for my comment on this ticket to get news about the upgrade, 
hopefully I would change it in the next weeks.

Regards

Original comment by gsfernan...@gmail.com on 19 Jul 2013 at 12:24

GoogleCodeExporter commented 9 years ago
Hi,
I have fixed a few issues that I found in these features.
- While appending photo ids to the "locallyDeleted" set it was using the photo 
object to build the list of ids instead of id values
- Improved logic to manage number of uploads per execution, now it will check 
upload by upload if the limit has been reached instead of a folder basis. This 
will avoid to process the whole directory and then check for number of uploaded 
files.
- Improved error messages to show the whole array of messages instead of first 
element only, this will allow users to understand the errors

With respect to the patch file, let me know if you can apply it to your working 
copy.

Regards,
Guillermo

Original comment by gsfernan...@gmail.com on 22 Jul 2013 at 5:09

Attachments:

GoogleCodeExporter commented 9 years ago
Hi,

thanks for the update. I never worked with eclipse before, therefore I failed 
to patch the files. I used Eclipse 3.8, which comes with the latest Linux mint 
distribution 15.

I am not sure how to go forward.

Regards

Original comment by tue...@gmail.com on 22 Jul 2013 at 9:39

GoogleCodeExporter commented 9 years ago
Hi,
It doesn't matter which tool I used to generate the patch, it should work in 
your environment too.
Try this, go to your folders2flickr checkout directory, copy the patch file 
that I attached to the same folder and then run the command:
patch -p0 -i folders2flickr.issue29.v2.patch

Let me know if it works, if not I will attach the .zip file with the sources to 
the ticket so you could download it.

Regards

Original comment by gsfernan...@gmail.com on 22 Jul 2013 at 9:59

GoogleCodeExporter commented 9 years ago
You could try with svn patch instead:
http://svnbook.red-bean.com/en/1.7/svn.ref.svn.c.patch.html

svn patch PATCHFILE [WCPATH]

Regards,
Guillermo

Original comment by gsfernan...@gmail.com on 22 Jul 2013 at 10:00

GoogleCodeExporter commented 9 years ago
Hi,

I downloaded a new copy of folders2flickr and applied the patch command above. 
But it did not work out, a lot errors appeared (only tagging.py worked).  Also 
svn did not do the trick. The sources would be very helpful.

Cheers

Original comment by tue...@gmail.com on 23 Jul 2013 at 6:23

GoogleCodeExporter commented 9 years ago
Hi,

That's strange, I'll take a look to it later. Meantime, here is the source code 
with the changes applied so you could use it.

Regards

Original comment by gsfernan...@gmail.com on 23 Jul 2013 at 11:02

Attachments:

GoogleCodeExporter commented 9 years ago
Hi,

I reverted my working copy to the latest version in trunk and applied the patch 
without any problems by using svn patch command.

Regards,
Guillermo

Original comment by gsfernan...@gmail.com on 23 Jul 2013 at 12:57

GoogleCodeExporter commented 9 years ago
Hi  pet...@gmail.com

I have tried to migrate the code to python 3 but it is too complex for me, I 
don't have experience in python and seems that many changes are needed in order 
to make this code work on python 3.

Regards,
Guillermo

Original comment by gsfernan...@gmail.com on 24 Jul 2013 at 8:39