You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cordova.apache.org by Martin Gonzalez <ma...@gmail.com> on 2014/06/09 19:39:42 UTC
[WP8][cordova-plugin-file] filesystem: null after use IsolatedStorageFile.MoveDirectory
I'm working on fix mobile spec automated tests, specifically on those which
involves the File plugin. In the test # 67 of that test suite, I've found
out that the problem is not that the directory is not moved, the problem is
that after this line is executed:
https://github.com/apache/cordova-plugin-file/blob/master/src/wp/File.cs#L1412
The DispatchCommandResult sends the entry file with the CallbackID, but the
problem is that the information about the filesystem is missing.
This situation it doesn't happen when isoFile.MoveFile(newPath),
IsolatedStorageFile.CreateDirectory(path) or
IsolatedStorageFile.CreateFile(path) are used the filesystem information it
remains with those but not with MoveDirectory.
During the 67 test and several others, it uses the filesystem information
of the recent moved directory to determine if the file exists, and when it
gets to GeFileOrDirectory on the options this is the array obtained:
"[null,"file1","{\"create\":false}","File619232322"]"
When Create or MoveFile:
"["\/\/entry,move.dsp.srcDir","file1","{\"create\":false}","File619232322"]"
>From the JS side object:
MoveDirectory:
{"isFile":false, "isDirectory":true, "name" : "entry.move.dsp.dstDir",
"filesystem":null, "nativeURL":null}
MoveFile:
{"isFile":true, "isDirectory":false, "name" : "entry.move.dsp.dstDir",
"filesystem":"<FileSystem: persistent>", "nativeURL":null}
That's the reason why some automated tests on the MoveTo section are
failing.
Tested on Windows Phone 8 and 8.1, emulator and device. Same behavior.
Any thoughts about this?
--
Regards,
Martin Gonzalez
Re: [WP8][cordova-plugin-file] filesystem: null after use IsolatedStorageFile.MoveDirectory
Posted by Martin Gonzalez <ma...@gmail.com>.
Sure no problem, and CB-6901 is the issue with all the information related.
On Jun 9, 2014 5:06 PM, "Jesse" <pu...@gmail.com> wrote:
> Yes, thanks Martin.
> The issue you created is sufficient. CB-6901
>
> @purplecabbage
> risingj.com
>
>
> On Mon, Jun 9, 2014 at 3:00 PM, Shazron <sh...@gmail.com> wrote:
>
> > Thanks Martin, is this captured in a JIRA issue just in case people
> > are away and this thread gets buried?
> >
> > On Mon, Jun 9, 2014 at 10:39 AM, Martin Gonzalez
> > <ma...@gmail.com> wrote:
> > > I'm working on fix mobile spec automated tests, specifically on those
> > which
> > > involves the File plugin. In the test # 67 of that test suite, I've
> found
> > > out that the problem is not that the directory is not moved, the
> problem
> > is
> > > that after this line is executed:
> > >
> >
> https://github.com/apache/cordova-plugin-file/blob/master/src/wp/File.cs#L1412
> > > The DispatchCommandResult sends the entry file with the CallbackID, but
> > the
> > > problem is that the information about the filesystem is missing.
> > > This situation it doesn't happen when isoFile.MoveFile(newPath),
> > > IsolatedStorageFile.CreateDirectory(path) or
> > > IsolatedStorageFile.CreateFile(path) are used the filesystem
> information
> > it
> > > remains with those but not with MoveDirectory.
> > > During the 67 test and several others, it uses the filesystem
> information
> > > of the recent moved directory to determine if the file exists, and when
> > it
> > > gets to GeFileOrDirectory on the options this is the array obtained:
> > > "[null,"file1","{\"create\":false}","File619232322"]"
> > >
> > > When Create or MoveFile:
> > >
> >
> "["\/\/entry,move.dsp.srcDir","file1","{\"create\":false}","File619232322"]"
> > >
> > > From the JS side object:
> > > MoveDirectory:
> > > {"isFile":false, "isDirectory":true, "name" : "entry.move.dsp.dstDir",
> > > "filesystem":null, "nativeURL":null}
> > >
> > > MoveFile:
> > > {"isFile":true, "isDirectory":false, "name" : "entry.move.dsp.dstDir",
> > > "filesystem":"<FileSystem: persistent>", "nativeURL":null}
> > >
> > > That's the reason why some automated tests on the MoveTo section are
> > > failing.
> > >
> > > Tested on Windows Phone 8 and 8.1, emulator and device. Same behavior.
> > >
> > > Any thoughts about this?
> > >
> > > --
> > > Regards,
> > > Martin Gonzalez
> >
>
Re: [WP8][cordova-plugin-file] filesystem: null after use IsolatedStorageFile.MoveDirectory
Posted by Jesse <pu...@gmail.com>.
Yes, thanks Martin.
The issue you created is sufficient. CB-6901
@purplecabbage
risingj.com
On Mon, Jun 9, 2014 at 3:00 PM, Shazron <sh...@gmail.com> wrote:
> Thanks Martin, is this captured in a JIRA issue just in case people
> are away and this thread gets buried?
>
> On Mon, Jun 9, 2014 at 10:39 AM, Martin Gonzalez
> <ma...@gmail.com> wrote:
> > I'm working on fix mobile spec automated tests, specifically on those
> which
> > involves the File plugin. In the test # 67 of that test suite, I've found
> > out that the problem is not that the directory is not moved, the problem
> is
> > that after this line is executed:
> >
> https://github.com/apache/cordova-plugin-file/blob/master/src/wp/File.cs#L1412
> > The DispatchCommandResult sends the entry file with the CallbackID, but
> the
> > problem is that the information about the filesystem is missing.
> > This situation it doesn't happen when isoFile.MoveFile(newPath),
> > IsolatedStorageFile.CreateDirectory(path) or
> > IsolatedStorageFile.CreateFile(path) are used the filesystem information
> it
> > remains with those but not with MoveDirectory.
> > During the 67 test and several others, it uses the filesystem information
> > of the recent moved directory to determine if the file exists, and when
> it
> > gets to GeFileOrDirectory on the options this is the array obtained:
> > "[null,"file1","{\"create\":false}","File619232322"]"
> >
> > When Create or MoveFile:
> >
> "["\/\/entry,move.dsp.srcDir","file1","{\"create\":false}","File619232322"]"
> >
> > From the JS side object:
> > MoveDirectory:
> > {"isFile":false, "isDirectory":true, "name" : "entry.move.dsp.dstDir",
> > "filesystem":null, "nativeURL":null}
> >
> > MoveFile:
> > {"isFile":true, "isDirectory":false, "name" : "entry.move.dsp.dstDir",
> > "filesystem":"<FileSystem: persistent>", "nativeURL":null}
> >
> > That's the reason why some automated tests on the MoveTo section are
> > failing.
> >
> > Tested on Windows Phone 8 and 8.1, emulator and device. Same behavior.
> >
> > Any thoughts about this?
> >
> > --
> > Regards,
> > Martin Gonzalez
>
Re: [WP8][cordova-plugin-file] filesystem: null after use IsolatedStorageFile.MoveDirectory
Posted by Shazron <sh...@gmail.com>.
Thanks Martin, is this captured in a JIRA issue just in case people
are away and this thread gets buried?
On Mon, Jun 9, 2014 at 10:39 AM, Martin Gonzalez
<ma...@gmail.com> wrote:
> I'm working on fix mobile spec automated tests, specifically on those which
> involves the File plugin. In the test # 67 of that test suite, I've found
> out that the problem is not that the directory is not moved, the problem is
> that after this line is executed:
> https://github.com/apache/cordova-plugin-file/blob/master/src/wp/File.cs#L1412
> The DispatchCommandResult sends the entry file with the CallbackID, but the
> problem is that the information about the filesystem is missing.
> This situation it doesn't happen when isoFile.MoveFile(newPath),
> IsolatedStorageFile.CreateDirectory(path) or
> IsolatedStorageFile.CreateFile(path) are used the filesystem information it
> remains with those but not with MoveDirectory.
> During the 67 test and several others, it uses the filesystem information
> of the recent moved directory to determine if the file exists, and when it
> gets to GeFileOrDirectory on the options this is the array obtained:
> "[null,"file1","{\"create\":false}","File619232322"]"
>
> When Create or MoveFile:
> "["\/\/entry,move.dsp.srcDir","file1","{\"create\":false}","File619232322"]"
>
> From the JS side object:
> MoveDirectory:
> {"isFile":false, "isDirectory":true, "name" : "entry.move.dsp.dstDir",
> "filesystem":null, "nativeURL":null}
>
> MoveFile:
> {"isFile":true, "isDirectory":false, "name" : "entry.move.dsp.dstDir",
> "filesystem":"<FileSystem: persistent>", "nativeURL":null}
>
> That's the reason why some automated tests on the MoveTo section are
> failing.
>
> Tested on Windows Phone 8 and 8.1, emulator and device. Same behavior.
>
> Any thoughts about this?
>
> --
> Regards,
> Martin Gonzalez