New Module to download photos from Apple iCloud

[Bungle68]

@msimon360 Thats brilliant, I’ve been hoping for this for a while.
Will give it a go.

[msimon360]

@Bungle68 Thanks let me know how it works for you. A few things I have found so far. I am using it with MMM-ImagesPhotos. When I add photos to the Album they sync to the uploads directory but they don’t display until I restart mm. It may just be my config values. Still testing. Also if I remove a photo from the Album a blank is shown for the removed photo until MMM-ImagesPhotos refreshes. I guess this is expected. I am looking into sending a Notification to force a refresh.

[sdetweil]

@msimon360 imagesphotos should refresh its list of images every 60 seconds by default
I use it linked to a file share on my nas
Because there are so many pics no idea when the new pic will appear, add random on to that and it’s worse for predictably

[msimon360]

@sdetweil I was testing on an album with only 4 photos and set to sequential. What I found, with help of Claude, was Before the Dec 2023 fetch rewrite, every completed request did:

if (retry) {
  self.scheduleUpdate(self.loaded ? -1 : self.config.retryDelay);
}

After the rewrite it became:

if (!photosResponse.ok) {
  if (retry) self.scheduleUpdate(...);
}

So the directory list only refreshed on HTTP failure, not every getInterval. I have an updated MMM-ImagesPhotos.js with a fix but I don’t know the best way to send it. Would you like a diff, patch, GitHub branch, other?

[sdetweil]

@msimon360 ImagePhotos isn’t mine, I have a fork…

so, generally how you do this , for any change to any repo on github

on github, find the source module repo
make a fork (copy on YOUR github)
checkout your fork
make a new branch
make the changes
commit to local branch
push to your github fork
github will see this and offer to create a PULL request to the original source , and the author can execute that PULL , from yours to theirs (merge is the operation)

once its merged, then you can delete your branch
and then sync the source to your fork (if you still want the fork)
git pull on your fork to get the update,
OR
delete your local copy of the fork, and redownload the original repo again
and maybe delete your fork on github, if you think you will never need it again

as for the bug

retry is a bad choice of variable, if its not in a failure path!.. lol…

[msimon360]

@sdetweil Thanks for the detailed instructions. I am using your fork and I made the pull request. The only files changed are MMM-ImagesPhotos.js and I put my GitHub link in the README.md.

[sdetweil]

@msimon360 merged… also merged support for apple HEIC images…

git pull
to get the latest
or sync your fork, then git pull

[msimon360]

@sdetweil I gave it a try and no photos displayed. I went back to my repo. The explanation I got from AI was this;
Why photos disappeared
non-fullscreen (bottom_center) crashes in getDomnotFS because EXIF is appended to this.wrapper (fullscreen-only) instead of the local wrapper. With default showExifDate: true, getDom throws → blank module.

[sdetweil]

@msimon360 ok. Set the showexif: false

Will review

[sdetweil]

I fixed the crash, git pull