-
Notifications
You must be signed in to change notification settings - Fork 10.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Microsoft.Web.Xdt #3077
Comments
@glennc @vijayrkn The bug occurred when the base source have an xml namespace while the target remove it. <configuration xmlns="http://schemas.microsoft.com/.NetConfiguration/v2.0" xdt:Transform="RemoveAttributes(xmlns)" xmlns:xdt="http://schemas.microsoft.com/XML-Document-Transform"> I am not sure when you remove the namespace from source(maybe in memory), but when I remove the namespace from source, the exception disappeared. |
@chucklu - I will take a look at the issue you mentioned and will get back. |
Hi @vijayrkn , any progress? |
@vijayrkn any update? private XmlNodeList GetAttributesFrom(XmlNode node, IList<string> arguments, bool warnIfEmpty) {
string[] array = new string[arguments.Count];
arguments.CopyTo(array, 0);
string xpath = String.Concat("@", String.Join("|@", array));
XmlNodeList attributes = node.SelectNodes(xpath);
if (attributes.Count == 0 && warnIfEmpty) {
Debug.Assert(arguments.Count == 1, "Should only call warnIfEmpty==true with one argument");
if (arguments.Count == 1) {
Log.LogWarning(SR.XMLTRANSFORMATION_TransformArgumentFoundNoAttributes, arguments[0]);
}
}
return attributes;
} XmlNodeList attributes = node.SelectNodes(xpath); |
The node above is a target node, however when you record the log, public void LogWarning(string message, params object[] messageArgs) {
if (SupressWarnings)
{
// SupressWarnings downgrade the Warning to LogMessage
LogMessage(message, messageArgs);
}
else
{
if (CurrentReferenceNode != null)
{
LogWarning(CurrentReferenceNode, message, messageArgs);
}
else if (externalLogger != null)
{
externalLogger.LogWarning(message, messageArgs);
}
}
} |
Closing this issue because this is not part of ASP.NET Core. |
Hi @Eilon, who I supposed to contact? |
@vijayrkn - what is the recommended place to report issues with XDT? |
At this point, aspnet/websdk would be the best place. |
Here's the full link: https://github.com/aspnet/websdk/issues |
Hi team,
I find a library on nuget https://www.nuget.org/packages/Microsoft.Web.Xdt/2.1.2
The project site is https://xdt.codeplex.com.
when I try to clone it, but failed with following info:
Could you tell me the correct url?
I find a fork on github https://github.com/davidalpert/xdt. but it's not the latest version released two months ago.
Is it possible for you to publish this project on github https://github.com/aspnet
The text was updated successfully, but these errors were encountered: