fpexif OneDrive "file not found" issue / patch

[DJMaster]

Hello,
I found a bug accessing jpg image files stored in OneDive directory with fpexif component.

To fix it I applied the attached patch, please review and apply to lazarus-ccr trunk if ok.

Thank you.

[wp]

Thanks. But I don't know how to reproduce the issue.

I put the new parameter of the "FileExists" function into a {$IF FPC_FullVersion >= 30200} because is does not exist in fpc 3.0.x. Then I copied an image onto my OneDrive. With a right-click on the file and "copy as path" I copied the path to the clipboard and pasted it back into the filename edit of the ExifSimpleDemo example program which is contained in the fpexif installation.

The demo was able to find the image and display its EXIF data when it was compiled with Laz trunk/fpc 3.04 and with Laz trunk/fpc trunk. But the image was also found when I used the unpatched code (with 1-argument FileExists call) with fpc trunk.

So I am a bit lost how to see the issue.

[DJMaster]

Sorry, I forgot to mention that I'm using Lazarus trunk version compiled with fpc fixes_3_2 branch.

I have problems with "OneDrive for business" version. I confirm right now that it works well with "OneDrive home".

Tomorrow I will do more tests with the ExifSimpleDemo example program.

Thank you for your reply.

[DJMaster]

ExifSimpleDemo.exe fails on my workstation at work accessing "OneDrive for business" files... so I can try to investigate.

[wp]

I updated svn with your (slightly modified) patch. Please test, I can't because I do not have access to OneDrive for Business.

[DJMaster]

Now it works with "OneDrive for business" files too, using Lazarus trunk / fpc fixes_3_2 branch. Thank you.

[DJMaster]

Please read this message/thread: https://forum.lazarus.freepascal.org/index.php/topic,42155.msg376343.html#msg376343

That patch should fix globally FileExists and all related FPC 3.2.0+ issues with OneDrive for Business files.

[wp]

This patch applies to FPC, not to fpexif. Please file a bug report to BugTracker (link in the left panel of the forum) and attach your patch so that the fpc team can see it. Otherwise it will be lost here.

[DJMaster]

SymLinks patch has been applied to fpc trunk and fixes_3_2 branch (https://svn.freepascal.org/cgi-bin/viewvc.cgi?view=revision&revision=47100).

Pleaase revert revision 7416 in fpexif lazarus-ccr (https://sourceforge.net/p/lazarus-ccr/svn/7416).

[wp]

I reverted that change. Please test and report back (I cannot test myself since I don't have access to OneDrive for Business).

[DJMaster]

I reverted that change. Please test and report back (I cannot test myself since I don't have access to OneDrive for Business).

Did you push the revert commit to Lazarus CCR? 

[wp]

Oh I'm sorry... --> r7778

[DJMaster]

Oh I'm sorry... --> r7778

Everything is working fine with OneDrive for Business files, thank you.