OrchardCMS / Orchard

Orchard is a free, open source, community-focused Content Management System built on the ASP.NET MVC platform.
https://orchardcore.net
BSD 3-Clause "New" or "Revised" License
2.38k stars 1.12k forks source link

Extension loader should look under any folder called *.Modules #2848

Open orchardbot opened 12 years ago

orchardbot commented 12 years ago

@bleroy created: https://orchard.codeplex.com/workitem/19021

This would enable single-repositories for a set of extensions on local clones, for example MyCompany.Modules.

orchardbot commented 12 years ago

BrandonJoyce commented:

Unless someone beats me to it, I'm happy to implement this. Bertrand suggests looking into the CoreExtensionLoader. We will need to consider the build target that copies files from the modules folder.

orchardbot commented 12 years ago

@bleroy commented:

It would be great. Thanks in advance...

orchardbot commented 12 years ago

tmakaro commented:

So I am a newbie to Orchard but I did some hacking on this and was able to add Modules.CompanyA and Modules.CompanyB by hard coding them into the following 2 classes:

OrchardStarter.cs RazorViewEngineProvider.cs

Its just a hack as there are no wildcards to search for 'Modules.*'

Oh ya, I used 'Modules.' instead of '.Modules' so that when I do a directory listing in windows explorer, they all group together when sorted by name.

orchardbot commented 12 years ago

BrandonJoyce commented:

Cool. My goal is to spend some time on this over the weekend. Maybe we can make the convention configurable too so people who like Modules.* better can go with that. Will keep you updated.

orchardbot commented 12 years ago

@bleroy commented:

Let's just support both.

orchardbot commented 12 years ago

BrandonJoyce commented:

Ya, that sounds good.

orchardbot commented 12 years ago

BrandonJoyce commented:

Orchard.Codegeneration will probably need an optional argument added for folder name. I'll probably go with orchard> codegen Module ModuleName /folder:Modules.YourModules

Probably just going to throw an error if the folder name is not in the format Modules.* or *.Modules. Alternatively, I could just automatically append .Modules, but I don't know if I like that.

orchardbot commented 12 years ago

@bleroy commented:

Good point. Should be fairly easy to add.