Some PostConnector (public API) improvements

[roman-yagodin]

Some nice things to have in public API:

• Ability to customize IsPinned then calling SubmitExternalPost() - currently always False, and PostValidation() method is private.
• Ability to get newly created post id somehow - e.g. to generate link to it. PostToDatabase() returns it, but it also private and heavily wrapped in PostValidation().

[juvander]

Need to see, especially this new post id is buried quite deep.

[juvander]

Added another SubmitExternalPost with IsPinned as a new parameter

[roman-yagodin]

Wow, thanks for IsPinned - you are very kind! It will sure help with creating new forum post to discuss from external source!

About new post id: it could be resolved by returning some structure (or class) like that:

public struct SubmitPostResult
{
    public int PostId;
    public PostMessage PostMessage;
}

It can be extended in the future to have more fields to return. But surely it looks like major change in API.

Similar approach could be applied to pass parameters to SubmitExternalPost():

public struct SubmitPostParams
{
    public int TabId;
    public int ModuleId;
    public int ForumId;
    public int UserId;
    ...
    public List<Term> Terms;
}

public SubmitPostResult SubmitExternalPost (SubmitPostParams postParams);

[juvander]

That was my idea as well, but yes, there is lots of places where this is checked. Need to review this carefully.

[juvander]

Did a quick refactoring, see with this https://github.com/juvander/DotNetNuke-Forum/releases/tag/05.01.00_DNN8

[roman-yagodin]

Thanks, I'll check it.

[roman-yagodin]

Seems to work as intended.

[juvander]

Excellent!