sebsync 1.0.3__tar.gz → 1.0.4__tar.gz

{sebsync-1.0.3 → sebsync-1.0.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: sebsync
-Version: 1.0.3
+Version: 1.0.4
 Summary: Synchronize Standard Ebooks catalog with local EPUB collection.
 Home-page: https://github.com/pbryan/sebsync/
 License: MIT

{sebsync-1.0.3 → sebsync-1.0.4}/pyproject.toml

@@ -8,7 +8,7 @@ target-version = ['py312']
 
 [tool.poetry]
 name = "sebsync"
-version = "1.0.3"
+version = "1.0.4"
 description = "Synchronize Standard Ebooks catalog with local EPUB collection."
 readme = "README.md"
 authors = ["sebsync authors"]

{sebsync-1.0.3 → sebsync-1.0.4}/sebsync.py

@@ -178,7 +178,7 @@ def sortable_author(author: str) -> str:
 
 
 def books_are_different(local_ebook: LocalEbook, remote_ebook: RemoteEbook) -> bool:
-    """Return if differences between local and remote ebooks are detected."""
+    """Return if differences are detected between local and remote ebooks."""
 
     # if metadata has exact modification times, then local is considered current
     if remote_ebook.updated == local_ebook.modified:
@@ -186,11 +186,16 @@ def books_are_different(local_ebook: LocalEbook, remote_ebook: RemoteEbook) -> b
 
     stat = local_ebook.path.stat()
 
-    # if timestamp of remote is less than that of local file, assume local is current
-    if remote_ebook.updated < datetime.fromtimestamp(stat.st_mtime, timezone.utc):
-        return False
+    file_modified = datetime.fromtimestamp(stat.st_mtime, timezone.utc)
+    if remote_ebook.updated > file_modified:
+        return True
+
+    response = requests.head(remote_ebook.href)
+    content_length = int(response.headers["Content-Length"])
+    if content_length != stat.st_size:
+        return True
 
-    return True
+    return False
 
 
 def ebook_filename(ebook: RemoteEbook) -> str:
@@ -219,11 +224,6 @@ def ebook_filename(ebook: RemoteEbook) -> str:
     type=click.Path(exists=True, file_okay=False, dir_okay=True, writable=True, path_type=Path),
     default=if_exists(Path.home() / "Downloads"),
 )
-@click.option(
-    "--force-update",
-    help="Force update of all local ebooks (implies --update).",
-    is_flag=True,
-)
 @click.option(
     "--dry-run",
     help="Perform a trial run with no changes made.",
@@ -234,6 +234,11 @@ def ebook_filename(ebook: RemoteEbook) -> str:
     help="Email address to authenticate with Standard Ebooks.",
     required=True,
 )
+@click.option(
+    "--force-update",
+    help="Force update of all local ebooks (implies --update).",
+    is_flag=True,
+)
 @click.help_option()
 @click.option(
     "--naming",