Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Moving Microsoft.ServiceModel.Syndication from wcf to corefx. #24202

Merged
merged 58 commits into from
Sep 22, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
58 commits
Select commit Hold shift + click to select a range
e48777c
Microsoft.ServiceModel.Syndication skeleton project
mconnew Jun 7, 2017
1d9f27f
Adding the existing classes of SyndicationFeed from .net fx
ElJerry Jun 8, 2017
9c278a1
Added the needed references to get the code to compile
ElJerry Jun 8, 2017
aa05946
Changed some namespaces
ElJerry Jun 8, 2017
398649d
Fixed errors when reading feeds and replaced some buffers
ElJerry Jun 12, 2017
0594aff
Cleaning code for PR
ElJerry Jun 13, 2017
9f6c78e
Deleted some unused files
ElJerry Jun 15, 2017
37bf505
Added the posibility that the user creates his own date parser when r…
ElJerry Jun 16, 2017
62f2980
Added correct testnames and Copyright
ElJerry Jun 17, 2017
91a06a4
Initial changes to add custom parsers
ElJerry Jun 19, 2017
adcb15b
Added more delegates as parsers
ElJerry Jun 20, 2017
01e52b9
Added custom parser delegates
ElJerry Jun 21, 2017
7ece449
Save changes
ElJerry Jun 21, 2017
898bff5
Initial SyndicationFeed
ElJerry Jun 8, 2017
6f2b69b
Removed the dependence of the old SR class
ElJerry Jun 21, 2017
c7be030
Cleaned the code, deleted Diagnostics, fixed throwing resources
ElJerry Jun 21, 2017
709a1ab
Cleaned code from most of the unnecesary comments
ElJerry Jun 22, 2017
8d9968f
Formatted the code with CodeFormatter Tool
ElJerry Jun 22, 2017
ab8c7ae
Moved the call of itemParser to be called with each item
ElJerry Jun 22, 2017
681f0fb
Test using custom parsers
ElJerry Jun 22, 2017
ae10738
Merge branch 'CustomParser' into InitialSyndicationFeed
ElJerry Jun 23, 2017
25309a2
Fixed image issue where image tag was using original title and url, f…
ElJerry Jun 23, 2017
b4d2d1b
Test clase jorge
ElJerry Jun 24, 2017
21f972f
Save changes with Jorge cass
ElJerry Jun 26, 2017
5dacea0
Initial Jorge
ElJerry Jun 26, 2017
77738fd
saving changes
ElJerry Jun 27, 2017
28b6ef6
Save changes
ElJerry Jun 27, 2017
52a29fd
Fixed image and items issue
ElJerry Jun 27, 2017
0f514e5
Fixed disjoint items in atom
ElJerry Jun 27, 2017
a8a8dfe
Run codeFormatter
ElJerry Jun 28, 2017
0b75b86
Adding parsing to optional elements int the spec
ElJerry Jun 28, 2017
c7e2224
Added Icon parsing to Atom10 feedFormatter. Added unit test
ElJerry Jun 28, 2017
e9344e6
Adding Async Writing
ElJerry Jun 29, 2017
7756539
Merge branch async into lab
ElJerry Jul 1, 2017
ddc0a71
Added Icon writing on Arom Writer
ElJerry Jul 3, 2017
71d8993
Fixed some warnings
ElJerry Jul 3, 2017
7a992b1
Improved custom parsing for RSS
ElJerry Jul 3, 2017
9e0f815
Added custom parsing for Atom feed formatter, added test
ElJerry Jul 5, 2017
c4abc36
added nameof() to all exceptions when needed.
ElJerry Jul 5, 2017
3795eda
Adding Extension Methods for XmlReader
ElJerry Jul 6, 2017
362f483
Fixing code for PR
ElJerry Jul 6, 2017
ed909e0
Fixing code for PR
ElJerry Jul 6, 2017
3f9cc1d
Added check for skip days to allow only accepted days.
ElJerry Jul 7, 2017
0ed721c
Improved flexibility for Default dateparser
ElJerry Jul 8, 2017
3557125
Add wrong dates example
ElJerry Jul 10, 2017
12175b6
Fixing code review
ElJerry Jul 12, 2017
68e73ff
Fixed warnings on some unawaited methods.
ElJerry Jul 17, 2017
14c5c59
Added async extension methods for XmlReader
ElJerry Jul 18, 2017
7cd87fc
Add XmlWriterWrapper method extensions
ElJerry Jul 18, 2017
813cd55
Changed ReadCategoryFromAsync to return a SyndicationCategory
ElJerry Jul 19, 2017
8df46f7
Fixed sync method exposed GetReader.
ElJerry Jul 31, 2017
2f16e88
Edited XmlReaderWrapper, moved methods to extension methods.
ElJerry Jul 31, 2017
755906b
Removed unnecesary variables from Wrappers.
ElJerry Jul 31, 2017
1752cc9
Fixed bug from ServiceModel.Syndication
ElJerry Aug 17, 2017
594afa6
Fix ServiceModel.SyndicationFeed project dependencies
zhenlan Sep 13, 2017
c7b485f
Move Syndication to root/src
shmao Sep 21, 2017
0115259
Merge branch 'lab' of C:\GitMV\wcf into AddSyndication
shmao Sep 21, 2017
e35e0cb
Disable Build of S.SM.Syndication.
shmao Sep 21, 2017
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@

Microsoft Visual Studio Solution File, Format Version 12.00
# Visual Studio 15
VisualStudioVersion = 15.0.26430.12
MinimumVisualStudioVersion = 10.0.40219.1
Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Microsoft.ServiceModel.Syndication", "src\Microsoft.ServiceModel.Syndication.csproj", "{B615835F-C286-4FB5-A2BF-C3B967AB9EC6}"
EndProject
Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Microsoft.ServiceModel.Syndication.Tests", "tests\Microsoft.ServiceModel.Syndication.Tests.csproj", "{A622B2C0-DD74-4218-9CF0-F9B2E52F4E91}"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm sure you're aware but the project references and name of this solution file should be changed from Microsoft to System.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The csproj currently does not build in corefx. The PR is mainly for porting the code from wcf to corefx. I planned to enable the build of the project later and fix the namespace later for better code review experience.

EndProject
Global
GlobalSection(SolutionConfigurationPlatforms) = preSolution
Debug|Any CPU = Debug|Any CPU
Release|Any CPU = Release|Any CPU
EndGlobalSection
GlobalSection(ProjectConfigurationPlatforms) = postSolution
{B615835F-C286-4FB5-A2BF-C3B967AB9EC6}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{B615835F-C286-4FB5-A2BF-C3B967AB9EC6}.Debug|Any CPU.Build.0 = Debug|Any CPU
{B615835F-C286-4FB5-A2BF-C3B967AB9EC6}.Release|Any CPU.ActiveCfg = Release|Any CPU
{B615835F-C286-4FB5-A2BF-C3B967AB9EC6}.Release|Any CPU.Build.0 = Release|Any CPU
{A622B2C0-DD74-4218-9CF0-F9B2E52F4E91}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{A622B2C0-DD74-4218-9CF0-F9B2E52F4E91}.Debug|Any CPU.Build.0 = Debug|Any CPU
{A622B2C0-DD74-4218-9CF0-F9B2E52F4E91}.Release|Any CPU.ActiveCfg = Release|Any CPU
{A622B2C0-DD74-4218-9CF0-F9B2E52F4E91}.Release|Any CPU.Build.0 = Release|Any CPU
EndGlobalSection
GlobalSection(SolutionProperties) = preSolution
HideSolutionNode = FALSE
EndGlobalSection
EndGlobal
3 changes: 3 additions & 0 deletions src/System.ServiceModel.Syndication/dir.props
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
<?xml version="1.0" encoding="utf-8"?>
<Project ToolsVersion="14.0" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
</Project>
3 changes: 3 additions & 0 deletions src/System.ServiceModel.Syndication/dir.targets
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
<?xml version="1.0" encoding="utf-8"?>
<Project ToolsVersion="14.0" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
</Project>
20 changes: 20 additions & 0 deletions src/System.ServiceModel.Syndication/src/App10Constants.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
// Licensed to the .NET Foundation under one or more agreements.
// The .NET Foundation licenses this file to you under the MIT license.
// See the LICENSE file in the project root for more information.


namespace Microsoft.ServiceModel.Syndication
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Microsoft -> System (applies throughout)

{
internal static class App10Constants
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be consistent with the other components in corefx, the source files should be nested under their namespace components (for example, see System.Collections' layout).

Instead of:

src/System.ServiceModel.Syndication/src/App10Constants.cs

Change to:

src/System.ServiceModel.Syndication/src/System/ServiceModel/Syndication/App10Constants.cs

(applies throughout)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, will change this later in separate PR.

{
public const string Accept = "accept";
public const string Categories = "categories";
public const string Collection = "collection";
public const string Fixed = "fixed";
public const string Href = "href";
public const string Namespace = "http://www.w3.org/2007/app";
public const string Prefix = "app";
public const string Service = "service";
public const string Workspace = "workspace";
}
}
53 changes: 53 additions & 0 deletions src/System.ServiceModel.Syndication/src/Atom10Constants.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,53 @@
// Licensed to the .NET Foundation under one or more agreements.
// The .NET Foundation licenses this file to you under the MIT license.
// See the LICENSE file in the project root for more information.



namespace Microsoft.ServiceModel.Syndication
{
public static class Atom10Constants
{
public const string AlternateTag = "alternate";
public const string Atom10Namespace = "http://www.w3.org/2005/Atom";
public const string Atom10Prefix = "a10";
public const string AtomMediaType = "application/atom+xml";
public const string AuthorTag = "author";
public const string CategoryTag = "category";
public const string ContentTag = "content";
public const string ContributorTag = "contributor";
public const string EmailTag = "email";
public const string EntryTag = "entry";
public const string FeedTag = "feed";
public const string GeneratorTag = "generator";
public const string HrefTag = "href";
public const string HtmlMediaType = "text/html";
public const string HtmlType = "html";
public const string IdTag = "id";
public const string LabelTag = "label";
public const string LengthTag = "length";
public const string LinkTag = "link";
public const string LogoTag = "logo";
public const string NameTag = "name";
public const string PlaintextType = "text";
public const string PublishedTag = "published";
public const string RelativeTag = "rel";
public const string RightsTag = "rights";
public const string SchemeTag = "scheme";
public const string SelfTag = "self";
public const string SourceFeedTag = "source";
public const string SourceTag = "src";
public const string SpecificationLink = "http://atompub.org/2005/08/17/draft-ietf-atompub-format-11.html";
public const string SubtitleTag = "subtitle";
public const string SummaryTag = "summary";
public const string TermTag = "term";
public const string TitleTag = "title";
public const string TypeTag = "type";
public const string UpdatedTag = "updated";
public const string UriTag = "uri";
public const string XHtmlMediaType = "application/xhtml+xml";
public const string XHtmlType = "xhtml";
public const string XmlMediaType = "text/xml";
public const string IconTag = "icon";
}
}
Loading