Reply to thread

Message

Hey u28151,

my workaround is little different: but have the same issue as your workaround; my catalog in Ant Movie Catalog is in xml format (and not .amc format: I like my files editable by text editor), which already has full path as source as relative path as Picture; I just copied this xml file to local drive where mediaPortal is, and pointed myFilms plug-in to that file.

the issue then remains: two copies, and I have to be responsible for keeping mediaPortal copy up-to-date.

also note: even with MP MyFilms running, I don't see a tmp file from my database file is - and thus can't compare. Log doesn't seems to indicate that it copied to a tmp file, but indicates reading from the main xml file. Can you show me what your new file structure is (that myfilms is creating)[DOUBLEPOST=1358501522][/DOUBLEPOST]Hi Guzzi,

thanks for your reply; I will post a feature request for "Security" - as it clearly is. I run quite a tight ship at home where domain users has specific privileges to certain folder - and I definately don't want an user using MediaPortal be able to update any of the databases.

I had a quick look at the source: at a cursory glance through MyFilms project, I can see three instances where a file is being created/open with a .tmp extension. Correct me if I am wrong, but seems like that file is ONLY created for locking purpose - i.e. stopping multiple myFilms instances to update the database. I found these instances on method "SaveMyFilmsToDisk", "LoadMyFilmsFromDisk" and "UpdateMovies".

If that is indeed the case, apart from "LoadMyFilmsFromDisk", the other two methods already opens the stream in "FileShare.None". So if my analysis is correct, by updating "LoadMyFilmsFromDisk" to open in exclusive mode, there would be no need to use the temporary file as a lock (btw, I see there is also a global lock - didn't check those methods yet)

<blockquote data-quote="RanaUK" data-source="post: 957233" data-attributes="member: 55312">Hey u28151,&nbsp;my workaround is little different: but have the same issue as your workaround; my catalog in Ant Movie Catalog is in xml format (and not .amc format: I like my files editable by text editor), which already has full path as source as relative path as Picture; I just copied this xml file to local drive where mediaPortal is, and pointed myFilms plug-in to that file.&nbsp;the issue then remains: two copies, and I have to be responsible for keeping mediaPortal copy up-to-date.&nbsp;also note: even with MP MyFilms running, I don't see a tmp file from my database file is - and thus can't compare. Log doesn't seems to indicate that it copied to a tmp file, but indicates reading from the main xml file. Can you show me what your new file structure is (that myfilms is creating)[DOUBLEPOST=1358501522][/DOUBLEPOST]Hi Guzzi,&nbsp;thanks for your reply; I will post a feature request for &quot;Security&quot; - as it clearly is. I run quite a tight ship at home where domain users has specific privileges to certain folder - and I definately don't want an user using MediaPortal be able to update any of the databases.&nbsp;I had a quick look at the source: at a cursory glance through MyFilms project, I can see three instances where a file is being created/open with a .tmp extension. Correct me if I am wrong, but seems like that file is ONLY created for locking purpose - i.e. stopping multiple myFilms instances to update the database. I found these instances on method &quot;SaveMyFilmsToDisk&quot;, &quot;LoadMyFilmsFromDisk&quot; and &quot;UpdateMovies&quot;.&nbsp;If that is indeed the case, apart from &quot;LoadMyFilmsFromDisk&quot;, the other two methods already opens the stream in &quot;FileShare.None&quot;. So if my analysis is correct, by updating &quot;LoadMyFilmsFromDisk&quot; to open in exclusive mode, there would be no need to use the temporary file as a lock (btw, I see there is also a global lock - didn't check those methods yet)</blockquote>

[QUOTE="RanaUK, post: 957233, member: 55312"] Hey u28151, my workaround is little different: but have the same issue as your workaround; my catalog in Ant Movie Catalog is in xml format (and not .amc format: I like my files editable by text editor), which already has full path as source as relative path as Picture; I just copied this xml file to local drive where mediaPortal is, and pointed myFilms plug-in to that file. the issue then remains: two copies, and I have to be responsible for keeping mediaPortal copy up-to-date. also note: even with MP MyFilms running, I don't see a tmp file from my database file is - and thus can't compare. Log doesn't seems to indicate that it copied to a tmp file, but indicates reading from the main xml file. Can you show me what your new file structure is (that myfilms is creating)[DOUBLEPOST=1358501522][/DOUBLEPOST]Hi Guzzi, thanks for your reply; I will post a feature request for "Security" - as it clearly is. I run quite a tight ship at home where domain users has specific privileges to certain folder - and I definately don't want an user using MediaPortal be able to update any of the databases. I had a quick look at the source: at a cursory glance through MyFilms project, I can see three instances where a file is being created/open with a .tmp extension. Correct me if I am wrong, but seems like that file is ONLY created for locking purpose - i.e. stopping multiple myFilms instances to update the database. I found these instances on method "SaveMyFilmsToDisk", "LoadMyFilmsFromDisk" and "UpdateMovies". If that is indeed the case, apart from "LoadMyFilmsFromDisk", the other two methods already opens the stream in "FileShare.None". So if my analysis is correct, by updating "LoadMyFilmsFromDisk" to open in exclusive mode, there would be no need to use the temporary file as a lock (btw, I see there is also a global lock - didn't check those methods yet) [/QUOTE]