4

I was using below code in VBScript which check the publishing target and if it is not a mobile targets then it raise an error in publishing queue.

Dim mobileTargets
mobileTargets = "tcm:0-0-0,tcm:0-175-65537,tcm:0-178-65537,tcm:0-179-65537"
If Instr(mobileTargets, PublicationTargetId) = 0 Then
    Err.Raise -1000, "Prevent Publish", "This page should be published only to mobile site"
End If

What is the best way to implement exception handling and Which class we can use in .NET Templating to achieve above functionality?

Thanks.

2 Answers 2

4

In VbScript Err.Raise is used to "raise an error". The equivalent action in .NET is to throw an exception. While these two approaches fulfil more or less the same function, the mechanism for identifying the error/exception is different. In your VbScript example you have used the error number -1000. This can be detected in code higher up the stack with something like:

If Err.Number = -1000 Then
  ' Do something 
End If

In .NET, instead of error numbers we use Type hierarchies. An exception object is derived from System.Exception, and it's usual to have an exception type that specifically identifies the kind of problem which has been encountered. For more sophisticated requirements, you might make a hierarchy that allows you to catch similar kinds of exceptions in the same handler. (For example, the Tridion CM throws exceptions that derive from Tridion.Common.TridionException) When throwing an exception from an event handler like this, you probably just need to create a suitable exception derived directly from Exception.

So for this example, you might write something like:

[Serializable]
public class InappropriatePublicationTargetException : Exception
{
    public InappropriatePublicationTargetException() { }
    public InappropriatePublicationTargetException(string message) : base(message) { }
    public InappropriatePublicationTargetException(string message, Exception inner) 
                                                            : base(message, inner) { }
    protected InappropriatePublicationTargetException(
      System.Runtime.Serialization.SerializationInfo info,
      System.Runtime.Serialization.StreamingContext context)
        : base(info, context) { }
}

Which might seem like a mouthful until you learn to type Ctrl-K-Ctrl-X v tab ex tab InappropriatePublicationTarget Enter to have Visual Studio's exception snippet generate the code for you.

With a class like this available, you can write something like:

throw new InappropriatePublicationTargetException(
                                      "This page should be published only to mobile site");

Of course, you may be asking "what's the point of this extra ritual"? Sure - in each case, what you are probably trying to achieve is having the err.Description or Exception.Message appear in the GUI for the end user to see. That might be enough, but in the general case, you need to consider that the event handler you are writing may just as well be triggered by code as by the GUI, in which case it can be very useful to be able to catch only your own exceptions.

3
  • Thanks Dominic for clarification, so you mean we would require to implement exception handling framework in our existing project where we will be creating the seperate classes for handling specific exceptions, like in your case InappropriatePublicationTargetException, specific for publicationtargets, whcih all be inherited from bas Exception class Commented Jun 12, 2013 at 10:59
  • Can you please clarify little more on this "That might be enough, but in the general case, you need to consider that the event handler you are writing may just as well be triggered by code as by the GUI, in which case it can be very useful to be able to catch only your own exceptions." Commented Jun 12, 2013 at 12:16
  • It's simply that when your error/exception will be processed by a human, then a description or message is appropriate. When it will be processed by code, it's easier to have a specific type. Commented Jun 12, 2013 at 17:41
6

Err.Raise is the VBScript equivalent to 'throw new Exception'. In a .Net C# Template (or Razor template) you would do something like:

throw new Exception("This page should be published only to mobile site");
1
  • Yes that is very clear, however if we wan't to make our generic classes as suggested below by dominic then that would make more clarity while handling the exception...what do you say? Commented Jun 12, 2013 at 12:10

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.