Return-Path: X-Original-To: apmail-cordova-dev-archive@www.apache.org Delivered-To: apmail-cordova-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 5E251119C1 for ; Mon, 9 Jun 2014 22:06:09 +0000 (UTC) Received: (qmail 10307 invoked by uid 500); 9 Jun 2014 22:06:04 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 10275 invoked by uid 500); 9 Jun 2014 22:06:03 -0000 Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cordova.apache.org Delivered-To: mailing list dev@cordova.apache.org Received: (qmail 10264 invoked by uid 99); 9 Jun 2014 22:06:03 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 09 Jun 2014 22:06:03 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of purplecabbage@gmail.com designates 209.85.220.180 as permitted sender) Received: from [209.85.220.180] (HELO mail-vc0-f180.google.com) (209.85.220.180) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 09 Jun 2014 22:05:59 +0000 Received: by mail-vc0-f180.google.com with SMTP id hq11so6899492vcb.39 for ; Mon, 09 Jun 2014 15:05:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=qdrsF/CcoMX1PEl8TXcmfnnFVgaGAKVsjVL26tKheAY=; b=B1FpRKfzw8bFlRGtriMw2sTt8rEAqIQnCxQRIz5pTuvDwkOoJQMMgiQTkvJL+6v6EC 5Wv8QOXUUn9MxLX4kzQYyXMD9zRo5vdwWJp0dbaJbRk5wN7IMjpXPNdsm5CsjudpiDOn aMf+4nyFKOujJvuJ9H5L2hbY9GMfSq7dnANjYfjPrjzYiJc723wiNUGtMTQNMHXS3gv6 wP6mA24yoVjKbTDaOI82Ro4nfPoky/ui9x1ywLd8QyiEBP1HGnBvYLU99uB95Qvfaxlk xdy07ziSaSqZjcw8NSxxdoWwGgkMk2SgvlR4/mAcXE9PbBmMrQQwFHdLEXatPvlj7YTd 2+SA== MIME-Version: 1.0 X-Received: by 10.58.30.1 with SMTP id o1mr27373672veh.37.1402351538783; Mon, 09 Jun 2014 15:05:38 -0700 (PDT) Received: by 10.58.197.97 with HTTP; Mon, 9 Jun 2014 15:05:38 -0700 (PDT) In-Reply-To: References: Date: Mon, 9 Jun 2014 15:05:38 -0700 Message-ID: Subject: Re: [WP8][cordova-plugin-file] filesystem: null after use IsolatedStorageFile.MoveDirectory From: Jesse To: "dev@cordova.apache.org" Content-Type: multipart/alternative; boundary=089e013cbe5e51cd0e04fb6e69c0 X-Virus-Checked: Checked by ClamAV on apache.org --089e013cbe5e51cd0e04fb6e69c0 Content-Type: text/plain; charset=UTF-8 Yes, thanks Martin. The issue you created is sufficient. CB-6901 @purplecabbage risingj.com On Mon, Jun 9, 2014 at 3:00 PM, Shazron 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 > 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":"", "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 > --089e013cbe5e51cd0e04fb6e69c0--