Return-Path: 
 <issues-return-30514-apmail-cordova-issues-archive=cordova.apache.org@cordova.apache.org>
X-Original-To: apmail-cordova-issues-archive@minotaur.apache.org
Delivered-To: apmail-cordova-issues-archive@minotaur.apache.org
Received: from mail.apache.org (hermes.apache.org [140.211.11.3])
	by minotaur.apache.org (Postfix) with SMTP id AAB48111B7
	for <apmail-cordova-issues-archive@minotaur.apache.org>;
 Tue, 26 Aug 2014 13:37:58 +0000 (UTC)
Received: (qmail 92174 invoked by uid 500); 26 Aug 2014 13:37:58 -0000
Delivered-To: apmail-cordova-issues-archive@cordova.apache.org
Received: (qmail 92154 invoked by uid 500); 26 Aug 2014 13:37:58 -0000
Mailing-List: contact issues-help@cordova.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:issues-help@cordova.apache.org>
List-Unsubscribe: <mailto:issues-unsubscribe@cordova.apache.org>
List-Post: <mailto:issues@cordova.apache.org>
List-Id: <issues.cordova.apache.org>
Reply-To: dev@cordova.apache.org
Delivered-To: mailing list issues@cordova.apache.org
Received: (qmail 92135 invoked by uid 99); 26 Aug 2014 13:37:58 -0000
Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28)
    by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 26 Aug 2014 13:37:58 +0000
Date: Tue, 26 Aug 2014 13:37:58 +0000 (UTC)
From: "Ian Clelland (JIRA)" <jira@apache.org>
To: issues@cordova.apache.org
Message-ID: <JIRA.12736659.1409008204035.13876.1409060278409@arcas>
In-Reply-To: <JIRA.12736659.1409008204035@arcas>
References: <JIRA.12736659.1409008204035@arcas>
Subject: [jira] [Commented] (CB-7375) Investigate breaking changes at file
 plugin
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394


    [ https://issues.apache.org/jira/browse/CB-7375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14110697#comment-14110697 ] 

Ian Clelland commented on CB-7375:
----------------------------------

To make this work for both Android/iOS *and* WP8, we can do this:

When JavaScript code is expecting an Entry structure to be returned from an exec() call, the structure returned should include these fields:

{code}
    isFile: Boolean
    isDirectory: Boolean

    name: String name of the file
    fullPath: String path to this file, relative to the filesystem root
    nativeURL: [Optional] String URL to this file that can be used natively by the webview (Important for <video src> attributes on Android)

    filesystemName: [Optional] String name of filesystem, eg "persistent", "temporary", "cache"
    filesystem: [Optional] Actual FileSystem object in which this file can be found
{code}

Only one of {{filesystemName}} or {{filesystem}} should be provided. If {{filesystemName}} is provided, then the JavaScript should construct an appropriate {{FileSystem}} object. If no {{filesystemName}} is provided, but {{filesystem}} is, then {{filesystem}} should be used as the {{FileSystem}} object for the new {{Entry}}.

I suggest that filesystemName, if provided, should take precedence, since it is not part of the standard, and that makes it more difficult to include it accidentally in the exec response.

Then to build a filesystem object for an Entry object, the code would look like this:

{code}
// Set a default
var filesystem = <some default filesystem to use>;
// Find an explicit name, if one was provided
var newFSName = entry.filesystemName || (entry.filesystem && entry.filesystem.name);
// Use the name *or* the provided FS *or* the default fs
var fs = (newFSName && new FileSystem(newFSName, { name: "", fullPath: "/" })) || entry.filesystem || filesystem;
{code}

> Investigate breaking changes at file plugin
> -------------------------------------------
>
>                 Key: CB-7375
>                 URL: https://issues.apache.org/jira/browse/CB-7375
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Plugin File
>    Affects Versions: 3.5.0
>            Reporter: Martin Gonzalez
>            Assignee: Ian Clelland
>              Labels: file
>             Fix For: 3.6.0
>
>
> Some of the changes made lately to file plugin with this commit: https://git-wip-us.apache.org/repos/asf?p=cordova-plugin-file.git;a=commit;h=0ffb96988c0539222887f7756a653e7f0a1ed12c
> Are breaking changes to this plugin, affecting other platforms as android.
> Running mobile spec after that specific commit, 4 file tests are failing (test 125 -128).



--
This message was sent by Atlassian JIRA
(v6.2#6252)