Add method TagSystem.get_taggable_realms

[Ryan J Ollos]

While reviewing some of the code, I noticed the possibility for some refactoring by adding a method to the TagSystem. I'll attach the proposed changes as a patch file.

[Steffen Hoffmann]

Yes, I know all these, even if I've been unaware of the three-fold extension point definition.

The new TagSystem API method is straight-forward too. Will push this this weekend with a few more changes. Thanks for taking care.

[Steffen Hoffmann]

In 14153:

TagsPlugin: Add generic getter method for taggable realms to API, refs #11968.

The method replaces similar code in several plugin components.
Thanks to Ryan J Ollos for proposing most of these changes.

[Steffen Hoffmann]

In 14154:

TagsPlugin: More clean-up after API review, refs #11968.

IMHO the only significant change in here is an earlier definition of all_realms
in TagWikiSyntaxProvider, but still it triggered something related to tags
Trac db table setup, so that I've been forced to fix many tests - wired.

[Steffen Hoffmann]

In 14155:

TagsPlugin: Add more permission checks after further review, refs #11968.

Looking at the optional permission-based filtering for taggable realms made me
watch out for places where we might have been too permissive until now.

[Steffen Hoffmann]

Replying to hasienda: One small but significant alteration was using built-in list() for the XMLRPC method. [set(..)] would have returned a list with the set as element.

[Steffen Hoffmann]

Thanks for the inspiration. IMHO this has yielded considerable improvements.

[Ryan J Ollos]

Replying to hasienda:

One small but significant alteration was using built-in list() for the XMLRPC method. [set(..)] would have returned a list with the set as element.

Good catch. I hadn't stopped to consider the difference in behavior before, between list() and [].

[Ryan J Ollos]

In 14945:

0.8: Tag 0.8 release

Fixes #1304, #1344, #3660, #3891, #9064, #9797, #11661, #11690, #11695, #11888, #11950, #11954, #11968, #12202, #12292, #12434, #12486.

Refs #12415.