posted 16 years ago
It's nice to include the solution so that when future searchers have a similar problem they're able to also see some solutions.
A couple of notes:
- Why do you use the "apt" prefix for the S2 tags?
- Why don't you use the framework's default result names (lower case)?
Using non-conventional/non-default naming conventions is fine, of course, but make the code more difficult for new developers to deal with.