[
https://issues.apache.org/jira/browse/CB-12163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15747722#comment-15747722
]
ASF GitHub Bot commented on CB-12163:
-------------------------------------
Github user vladimir-kotikov commented on a diff in the pull request:
https://github.com/apache/cordova-windows/pull/213#discussion_r92344562
--- Diff: template/cordova/lib/JsprojManager.js ---
@@ -121,7 +121,12 @@ jsprojManager.prototype = {
copyToOutputDirectory.text = 'Always';
children.push(copyToOutputDirectory);
- var item = createItemGroupElement('ItemGroup/Content', sourcePath,
targetConditions, children);
+ var item;
--- End diff --
IMO it'd be better to calculate path for `content` element outside this
method (in `'resource-file'.install`) - this way you will avoid checking the
same condition (`if (targetConditions.reference) ...`) twice and won't need to
modify this method at all.
> plugin.xml 'resource-file' does not copy file to target for Windows
> -------------------------------------------------------------------
>
> Key: CB-12163
> URL: https://issues.apache.org/jira/browse/CB-12163
> Project: Apache Cordova
> Issue Type: Bug
> Components: Windows
> Affects Versions: Master
> Environment: Windows 8/Windows10
> Reporter: Karen Tran
> Assignee: Karen Tran
> Labels: Windows
>
> From plugin.xml:
> {noformat}<resource-file src="windows/shared/myprops.properties"
> target="myprops.properties"/>{noformat}
> <resource-file> does not copy the file from src to target anymore. It used
> to, but was changed. This creates a couple of issues since I expect it to be
> in the application, but it technically isn't. This means that I cannot update
> the file that doesn't exist and very much limits the functionality of
> <resource-file> (Also the documentation is wrong for Windows since it doesn't
> copy any files). Visual Studio also only shows the file from the src path.
> This doesn't make much sense since that file wouldn't be in the application.
> The expected behavior should be if the target is specified, <resource-file>
> should copy the file to the target location. In turn, Visual Studio should
> show the target file instead of the src file because the target is part of
> the application whereas the src file is not.
> This used to be expected behavior, but was changed as a result of CB-10326.
> The intention of that issue is limited to use cases of only having the src
> attribute, ignoring the possibility that we would want to update files in the
> target instead (which imo makes more sense). Resource files are not all .dll
> files, so it's not very user friendly to limit the <resource-file> tag to
> handling just those kinds of files.
> Proposed changes in JsprojManager.js and PluginHandler.js:
> If target attribute is specified, copy the file to the target and point the
> Include attribute to the path of the target.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org
