Fix #39 - ajout de la fonction imposeSortOrderOnPublishedCollection pour gérer l'ordre de tri des collections publiées

Author: Gotcha26

piwigoPublish.lrplugin/PublishServiceProvider.lua

@@ -56,19 +56,21 @@ return {
 		{ key = "mdDescription",         default = "{{caption}}" },
 		{ key = "syncAlbumDescriptions", default = false },
 		{ key = "syncCommentsPublish",   default = true },
-		{ key = "syncCommentsPubOnly",   default = false }
-
+		{ key = "syncCommentsPubOnly",   default = false },
 	},
-	--[[
-	metadataThatTriggersRepublish = {
-		default = false,
-		title = true,
-		caption = true,
-		keywords = true,
-		gps = true,
-		dateCreated = true,
-    },
-]]
+
+	metadataThatTriggersRepublish = function(publishSettings, photoId, fieldName)
+		-- This function is called by Lightroom to determine whether a metadata
+		-- change should trigger republishing.
+		local triggerFields = {
+			title = true,
+			caption = true,
+			keywords = true,
+			gps = true,
+			dateCreated = true,
+		}
+		return triggerFields[fieldName] or false
+	end,
 	-- canExportToTemporaryLocation = true
 	-- canExportToTemporaryLocation = true
 	-- showSections = { 'fileNaming', 'fileSettings', etc... }
@@ -107,6 +109,7 @@ return {
 	renamePublishedCollection                        = PublishTask.renamePublishedCollection,
 	reparentPublishedCollection                      = PublishTask.reparentPublishedCollection,
 	deletePublishedCollection                        = PublishTask.deletePublishedCollection,
+	imposeSortOrderOnPublishedCollection			 = PublishTask.imposeSortOrderOnPublishedCollection,
 	validatePublishedCollectionName                  = PublishTask.validatePublishedCollectionName,
 
 	-- view on Piwigo Options

piwigoPublish.lrplugin/PublishTask.lua

@@ -632,6 +632,65 @@ function PublishTask.shouldDeletePhotosFromServiceOnDeleteFromCatalog(publishSet
     return nil -- Show builtin Lightroom dialog.
 end
 
+-- ************************************************
+function PublishTask.imposeSortOrderOnPublishedCollection(publishSettings, info, remoteIdSequence)
+    -- This callback is called by Lightroom for smart collections.
+    -- It allows you to detect published photos that no longer meet the criteria and mark them for
+    -- deletion.
+    log:info("PublishTask.imposeSortOrderOnPublishedCollection")
+    
+    local validSequence = {}
+    local publishedCollection = info.publishedCollection
+    
+    if not publishedCollection then
+        return nil
+    end
+    
+    -- Check if it is a smart collection
+    if not publishedCollection:isSmartCollection() then
+        return nil
+    end
+    
+    -- Retrieve photos currently in the smart collection (according to criteria)
+    local currentPhotos = publishedCollection:getPhotos()
+    local currentPhotoIds = {}
+    for _, photo in ipairs(currentPhotos) do
+        currentPhotoIds[photo.localIdentifier] = true
+    end
+    
+    -- Browse the remoteIds of published photos
+    -- remoteIdSequence contains the remoteIds in the current order
+    local publishedPhotos = publishedCollection:getPublishedPhotos()
+    local remoteIdToPhoto = {}
+    for _, pubPhoto in ipairs(publishedPhotos) do
+        local remoteId = pubPhoto:getRemoteId()
+        if remoteId then
+            remoteIdToPhoto[remoteId] = pubPhoto
+        end
+    end
+    
+    -- Build the valid sequence: only photos that still meet the criteria
+    for _, remoteId in ipairs(remoteIdSequence) do
+        local pubPhoto = remoteIdToPhoto[remoteId]
+        if pubPhoto then
+            local lrPhoto = pubPhoto:getPhoto()
+            if lrPhoto and currentPhotoIds[lrPhoto.localIdentifier] then
+                -- The photo still meets the criteria, keep it.
+                table.insert(validSequence, remoteId)
+            end
+            -- If the photo is no longer in currentPhotoIds, it will be marked for deletion because
+            -- its remoteId will not be in validSequence.
+        end
+    end
+    
+    log:info("PublishTask.imposeSortOrderOnPublishedCollection - " .. 
+             #remoteIdSequence .. " published, " .. 
+             #validSequence .. " still match criteria, " ..
+             (#remoteIdSequence - #validSequence) .. " to delete")
+    
+    return validSequence
+end
+
 -- ************************************************
 function PublishTask.validatePublishedCollectionName(name)
     log:info("PublishTask.validatePublishedCollectionName")