New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resource List TV parent field not working #14207

Open
neoneddy opened this Issue Dec 19, 2018 · 6 comments

Comments

Projects
None yet
3 participants
@neoneddy
Copy link

neoneddy commented Dec 19, 2018

Bug report

Summary

When configuring the resource select TV to only display resources from a specific parent it shows all resources in all contexts

Step to reproduce

Create Resource List TV, set parent field to a valid value, assign to Template, try and select.

Tried on Multicontext sites as well.

Observed behavior

Lists all resources

Expected behavior

List only parent resources

Environment

Multicontext site, MODX 2.7 , PHP 7.2, LiquidWeb Cloudsites. Also present in single context site on php 5.6 server.

@sottwell

This comment has been minimized.

Copy link
Contributor

sottwell commented Dec 19, 2018

I can confirm this bug.

Looking at forum posts for several years, it looks like this never worked as expected.

screen shot 2018-12-19 at 19 58 31

screen shot 2018-12-19 at 19 58 08

@neoneddy

This comment has been minimized.

Copy link

neoneddy commented Dec 19, 2018

I'm pretty sure at one point it did work. I used it in an area where I'm sure my client would have made a fit of having to scroll through 200 items.

@sottwell

This comment has been minimized.

Copy link
Contributor

sottwell commented Dec 19, 2018

Could be; I never used it, I usually used one of Sepia River's methods.

@neoneddy

This comment has been minimized.

Copy link

neoneddy commented Dec 19, 2018

I probably do as well ( a listbox with a query building the resources list) But in this case I needed the context awareness.

@Jako

This comment has been minimized.

Copy link
Collaborator

Jako commented Dec 19, 2018

@Jako

This comment has been minimized.

Copy link
Collaborator

Jako commented Dec 20, 2018

Fixing this right would create a BC, so this has to wait until 3.x

@Jako Jako added this to the v3.0.0-alpha milestone Dec 20, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment