What's the difference between _isEnabled and isEnabled in Anguilla?

Question

I\'ve been following GUI extensions and notice examples use either _isEnabled or isEnabled, without the underscore. Both seem to work to extend or

Answer 1

The following methods are called for a command:

1. isAvailable
2. isEnabled
3. invoke

The base class for all commands - Tridion.Core.Command - has a standard implementation of these methods. For the most part, this default implementation allows for extensions to Commands. They also call the underscore methods (_isAvailable, _isEnabled, and _execute).

I don't know why the CME commands only overwrite the underscore methods. Maybe someone thought it was just easier that way. They should be consider private (or the equivalent of "protected" in C#), so it actually seems like a bad practice to me.

It would be cleaner to implement the proper methods (isAvailable, isEnabled, and invoke) and then call the base implementation using this.callBase. However, you might need to stop the pipeline in this case, or also overwrite the underscore methods, in order to avoid your return value getting overwritten by the default underscore methods. It depends on the command you are implementing or extending.

In short: using the underscore methods is probably bad practice, but the Core implementation does seem to make it harder for you to do it "right". So I'd aim to avoid the underscore methods, but not sweat it if it turns out to be too hard to do so.

P.S. isValidSelection is a PowerTools-only method which separates the common logic that they all need from the logic specific to each command.