NGSTACK-461 Use query type instead of controller for ng category

[RandyCupic]

Depends on: netgen/ezplatform-site-api#187

[emodric]

@pspanja Does this look good to you?

[pspanja]

It does, mostly, I'm just not sure about viewParam("childrenLimit", 12), can't figure out where it comes from.

Edit: it seems to be an unused extension point.

[pspanja]

Would it be useful to change fetch_subtree field to depth/relative_depth?

[emodric]

Would it be useful to change fetch_subtree field to depth/relative_depth?

Maybe. We can do it in 1.10 👍

[RandyCupic]

Would it be useful to change fetch_subtree field to depth/relative_depth?

I didn't understand this one: current fetch subtree field only controls whether we will fetch only first level children or everything. I just used relative depth as a way to implement this.

[emodric]

I think @pspanja was referring to changing the field in the database to be called depth or relative_depth, but it cannot be done easily since it can be a breaking change.

[pspanja]

@RandyCupic sorry, it was a kind of a digression on this issue, I meant to ask would it be useful in general, in order to control subtree depth.

[pspanja]

@emodric correct, but not just to change the name, but also change it from checkbox to integer. It's separate topic to this issue in any case.

[emodric]

but also change it from checkbox to integer.

Yes, I figured as much, also :)

It's separate topic to this issue in any case.

👍

[RandyCupic]

I think @pspanja was referring to changing the field in the database to be called depth or relative_depth, but it cannot be done easily since it can be a breaking change.

Ah, I understand now; so it would allow more control by being able to specify any depth.

[emodric]

Thanks @RandyCupic & @pspanja !