mirror of
https://github.com/OrchardCMS/Orchard.git
synced 2026-02-09 09:16:41 +08:00
Remove Blogs dependence on HomePageProvider
--HG-- branch : autoroute
This commit is contained in:
@@ -41,7 +41,7 @@ namespace Orchard.Blogs.Commands {
|
|||||||
public string FeedUrl { get; set; }
|
public string FeedUrl { get; set; }
|
||||||
|
|
||||||
[OrchardSwitch]
|
[OrchardSwitch]
|
||||||
public string Slug { get; set; }
|
public int Id { get; set; }
|
||||||
|
|
||||||
[OrchardSwitch]
|
[OrchardSwitch]
|
||||||
public string Owner { get; set; }
|
public string Owner { get; set; }
|
||||||
@@ -72,11 +72,6 @@ namespace Orchard.Blogs.Commands {
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
if(!IsSlugValid(Slug)) {
|
|
||||||
Context.Output.WriteLine(T("Invalid Slug {0} provided. Blog creation failed.", Slug));
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
var blog = _contentManager.New("Blog");
|
var blog = _contentManager.New("Blog");
|
||||||
blog.As<ICommonPart>().Owner = owner;
|
blog.As<ICommonPart>().Owner = owner;
|
||||||
blog.As<TitlePart>().Title = Title;
|
blog.As<TitlePart>().Title = Title;
|
||||||
@@ -94,8 +89,8 @@ namespace Orchard.Blogs.Commands {
|
|||||||
}
|
}
|
||||||
|
|
||||||
[CommandName("blog import")]
|
[CommandName("blog import")]
|
||||||
[CommandHelp("blog import /Slug:<slug> /FeedUrl:<feed url> /Owner:<username>\r\n\t" + "Import all items from <feed url> into the blog at the specified <slug>")]
|
[CommandHelp("blog import /BlogId:<id> /FeedUrl:<feed url> /Owner:<username>\r\n\t" + "Import all items from <feed url> into the blog specified by <id>")]
|
||||||
[OrchardSwitches("FeedUrl,Slug,Owner")]
|
[OrchardSwitches("FeedUrl,Id,Owner")]
|
||||||
public void Import() {
|
public void Import() {
|
||||||
var owner = _membershipService.GetUser(Owner);
|
var owner = _membershipService.GetUser(Owner);
|
||||||
|
|
||||||
@@ -115,10 +110,10 @@ namespace Orchard.Blogs.Commands {
|
|||||||
throw new OrchardException(T("An error occured while loading the feed at {0}.", FeedUrl), ex);
|
throw new OrchardException(T("An error occured while loading the feed at {0}.", FeedUrl), ex);
|
||||||
}
|
}
|
||||||
|
|
||||||
var blog = _blogService.Get(Slug);
|
var blog = _blogService.Get(Id,VersionOptions.Latest);
|
||||||
|
|
||||||
if ( blog == null ) {
|
if ( blog == null ) {
|
||||||
Context.Output.WriteLine(T("Blog not found at specified slug: {0}", Slug));
|
Context.Output.WriteLine(T("Blog not found with specified Id: {0}", Id));
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -130,10 +125,7 @@ namespace Orchard.Blogs.Commands {
|
|||||||
var post = _contentManager.New("BlogPost");
|
var post = _contentManager.New("BlogPost");
|
||||||
post.As<ICommonPart>().Owner = owner;
|
post.As<ICommonPart>().Owner = owner;
|
||||||
post.As<ICommonPart>().Container = blog;
|
post.As<ICommonPart>().Container = blog;
|
||||||
var slug = Slugify(postName);
|
post.As<TitlePart>().Title = postName;
|
||||||
post.As<RoutePart>().Slug = slug;
|
|
||||||
post.As<RoutePart>().Path = post.As<RoutePart>().GetPathWithSlug(slug);
|
|
||||||
post.As<RoutePart>().Title = postName;
|
|
||||||
post.As<BodyPart>().Text = item.Element("description").Value;
|
post.As<BodyPart>().Text = item.Element("description").Value;
|
||||||
_contentManager.Create(post);
|
_contentManager.Create(post);
|
||||||
}
|
}
|
||||||
@@ -142,21 +134,5 @@ namespace Orchard.Blogs.Commands {
|
|||||||
Context.Output.WriteLine(T("Import feed completed."));
|
Context.Output.WriteLine(T("Import feed completed."));
|
||||||
}
|
}
|
||||||
|
|
||||||
private static string Slugify(string slug) {
|
|
||||||
var dissallowed = new Regex(@"[/:?#\[\]@!$&'()*+,;=\s]+");
|
|
||||||
|
|
||||||
slug = dissallowed.Replace(slug, "-");
|
|
||||||
slug = slug.Trim('-');
|
|
||||||
|
|
||||||
if ( slug.Length > 1000 )
|
|
||||||
slug = slug.Substring(0, 1000);
|
|
||||||
|
|
||||||
return slug.ToLowerInvariant();
|
|
||||||
}
|
|
||||||
|
|
||||||
private static bool IsSlugValid(string slug) {
|
|
||||||
// see http://tools.ietf.org/html/rfc3987 for prohibited chars
|
|
||||||
return slug == null || String.IsNullOrEmpty(slug.Trim()) || Regex.IsMatch(slug, @"^[^/:?#\[\]@!$&'()*+,;=\s]+$");
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -26,7 +26,6 @@ namespace Orchard.Blogs.Controllers {
|
|||||||
private readonly IBlogPathConstraint _blogPathConstraint;
|
private readonly IBlogPathConstraint _blogPathConstraint;
|
||||||
private readonly IFeedManager _feedManager;
|
private readonly IFeedManager _feedManager;
|
||||||
private readonly IWorkContextAccessor _workContextAccessor;
|
private readonly IWorkContextAccessor _workContextAccessor;
|
||||||
private readonly IHomePageProvider _routableHomePageProvider;
|
|
||||||
private readonly ISiteService _siteService;
|
private readonly ISiteService _siteService;
|
||||||
|
|
||||||
public BlogController(
|
public BlogController(
|
||||||
@@ -37,7 +36,6 @@ namespace Orchard.Blogs.Controllers {
|
|||||||
IFeedManager feedManager,
|
IFeedManager feedManager,
|
||||||
IShapeFactory shapeFactory,
|
IShapeFactory shapeFactory,
|
||||||
IWorkContextAccessor workContextAccessor,
|
IWorkContextAccessor workContextAccessor,
|
||||||
IEnumerable<IHomePageProvider> homePageProviders,
|
|
||||||
ISiteService siteService) {
|
ISiteService siteService) {
|
||||||
_services = services;
|
_services = services;
|
||||||
_blogService = blogService;
|
_blogService = blogService;
|
||||||
@@ -46,7 +44,6 @@ namespace Orchard.Blogs.Controllers {
|
|||||||
_feedManager = feedManager;
|
_feedManager = feedManager;
|
||||||
_workContextAccessor = workContextAccessor;
|
_workContextAccessor = workContextAccessor;
|
||||||
_siteService = siteService;
|
_siteService = siteService;
|
||||||
_routableHomePageProvider = homePageProviders.SingleOrDefault(p => p.GetProviderName() == RoutableHomePageProvider.Name);
|
|
||||||
Logger = NullLogger.Instance;
|
Logger = NullLogger.Instance;
|
||||||
Shape = shapeFactory;
|
Shape = shapeFactory;
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -17,7 +17,7 @@ namespace Orchard.Blogs.Handlers {
|
|||||||
public class BlogPartHandler : ContentHandler {
|
public class BlogPartHandler : ContentHandler {
|
||||||
private readonly IWorkContextAccessor _workContextAccessor;
|
private readonly IWorkContextAccessor _workContextAccessor;
|
||||||
|
|
||||||
public BlogPartHandler(IRepository<BlogPartRecord> repository, IWorkContextAccessor workContextAccessor, IEnumerable<IHomePageProvider> homePageProviders, IBlogPathConstraint blogPathConstraint) {
|
public BlogPartHandler(IRepository<BlogPartRecord> repository, IWorkContextAccessor workContextAccessor, IBlogPathConstraint blogPathConstraint) {
|
||||||
_workContextAccessor = workContextAccessor;
|
_workContextAccessor = workContextAccessor;
|
||||||
Filters.Add(StorageFilter.For(repository));
|
Filters.Add(StorageFilter.For(repository));
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user