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

Cannot Expand folders after upgrade from 1.1 to 1.3.3 #165

Closed
q2apro opened this Issue Aug 8, 2017 · 22 comments

Comments

Projects
None yet
4 participants
@q2apro

q2apro commented Aug 8, 2017

Clicking on a folder does not unfold it. It instead opens the container doc. And then, in the right in the tabs there is "View Children" to access the children documents.

I assume this is not by purpose but the container should show there containing documents when clicking on them?

@Nicola1971

This comment has been minimized.

Show comment
Hide comment
@Nicola1971

Nicola1971 Aug 8, 2017

you must click on the arrow near the folder
download

Nicola1971 commented Aug 8, 2017

you must click on the arrow near the folder
download

@64j

This comment has been minimized.

Show comment
Hide comment
@64j

64j Aug 8, 2017

Member

This can happen if these parameters are selected

image

image

Member

64j commented Aug 8, 2017

This can happen if these parameters are selected

image

image

@q2apro

This comment has been minimized.

Show comment
Hide comment
@q2apro

q2apro Aug 8, 2017

Still no luck.

Neither with the settings above, nor with my previous settings:

former

This happens:

recording

q2apro commented Aug 8, 2017

Still no luck.

Neither with the settings above, nor with my previous settings:

former

This happens:

recording

@64j

This comment has been minimized.

Show comment
Hide comment
@64j

64j Aug 8, 2017

Member

Or so

image

Member

64j commented Aug 8, 2017

Or so

image

@64j

This comment has been minimized.

Show comment
Hide comment
@64j

64j Aug 8, 2017

Member

Clear browser cache
Ctrl + F5 or Ctrl + Shift + R

Member

64j commented Aug 8, 2017

Clear browser cache
Ctrl + F5 or Ctrl + Shift + R

@q2apro

This comment has been minimized.

Show comment
Hide comment
@q2apro

q2apro Aug 8, 2017

Tried all you said. Nothing.

settings

That's weird.

q2apro commented Aug 8, 2017

Tried all you said. Nothing.

settings

That's weird.

@64j

This comment has been minimized.

Show comment
Hide comment
@64j

64j Aug 8, 2017

Member

image

Member

64j commented Aug 8, 2017

image

@Nicola1971

This comment has been minimized.

Show comment
Hide comment
@Nicola1971

Nicola1971 Aug 8, 2017

@64j ..why?
I do not understand why hiding childrens with Enable Stats Tracking disabled

Nicola1971 commented Aug 8, 2017

@64j ..why?
I do not understand why hiding childrens with Enable Stats Tracking disabled

@q2apro

This comment has been minimized.

Show comment
Hide comment
@q2apro

q2apro Aug 8, 2017

I sum up my settings, which do not work:

  1. Interface & Features: Page Click Behavior -> Edit Resource

  2. Sort Tree: Folder, Descending, Default, View children (enabled)

  3. For all containers the "Enable Stats Tracking" is turned off

I also did a hard reload. I further went into private mode (browser) to have no cache/cookie problems etc.

Still not working.

q2apro commented Aug 8, 2017

I sum up my settings, which do not work:

  1. Interface & Features: Page Click Behavior -> Edit Resource

  2. Sort Tree: Folder, Descending, Default, View children (enabled)

  3. For all containers the "Enable Stats Tracking" is turned off

I also did a hard reload. I further went into private mode (browser) to have no cache/cookie problems etc.

Still not working.

@Nicola1971

This comment has been minimized.

Show comment
Hide comment
@Nicola1971

Nicola1971 Aug 8, 2017

@q2apro you must enable "Enable Stats Tracking"

Nicola1971 commented Aug 8, 2017

@q2apro you must enable "Enable Stats Tracking"

@64j

This comment has been minimized.

Show comment
Hide comment
@64j

64j Aug 8, 2017

Member

@Nicola1971 incorrect translation

RU: Показывать дочерние ресурсы
EN: Show child resources

image

Member

64j commented Aug 8, 2017

@Nicola1971 incorrect translation

RU: Показывать дочерние ресурсы
EN: Show child resources

image

@q2apro

This comment has been minimized.

Show comment
Hide comment
@q2apro

q2apro Aug 8, 2017

@Nicola1971 Thanks. That was it!

q2apro commented Aug 8, 2017

@Nicola1971 Thanks. That was it!

@q2apro q2apro closed this Aug 8, 2017

@Nicola1971

This comment has been minimized.

Show comment
Hide comment
@Nicola1971

Nicola1971 commented Aug 8, 2017

@q2apro thanks to @64j :)

@Nicola1971

This comment has been minimized.

Show comment
Hide comment
@Nicola1971

Nicola1971 Aug 8, 2017

@Nicola1971 incorrect translation

@64j ok, but ...where is Enable Stats Tracking ? :D

Nicola1971 commented Aug 8, 2017

@Nicola1971 incorrect translation

@64j ok, but ...where is Enable Stats Tracking ? :D

@64j

This comment has been minimized.

Show comment
Hide comment
@64j

64j Aug 8, 2017

Member

@Nicola1971 Yes.
Need to change the translation
Enable Stats Tracking => Show child resources

( $ph['donthit'] )

https://github.com/evolution-cms/evolution/blob/develop/manager/frames/nodes.functions.inc.php#L353

Member

64j commented Aug 8, 2017

@Nicola1971 Yes.
Need to change the translation
Enable Stats Tracking => Show child resources

( $ph['donthit'] )

https://github.com/evolution-cms/evolution/blob/develop/manager/frames/nodes.functions.inc.php#L353

@Nicola1971

This comment has been minimized.

Show comment
Hide comment
@Nicola1971

Nicola1971 Aug 8, 2017

we need two separated options "Enable Stats Tracking" AND "Show child resources"
There are some Evo extras and plugins based on "Enable Stats Tracking" option (ie: Google Analytics), and is not nice hide childrens when you just want to disable stats tracking.

Nicola1971 commented Aug 8, 2017

we need two separated options "Enable Stats Tracking" AND "Show child resources"
There are some Evo extras and plugins based on "Enable Stats Tracking" option (ie: Google Analytics), and is not nice hide childrens when you just want to disable stats tracking.

@64j

This comment has been minimized.

Show comment
Hide comment
@64j

64j Aug 8, 2017

Member

@Nicola1971
So it turned out that this parameter was not used by anyone

Member

64j commented Aug 8, 2017

@Nicola1971
So it turned out that this parameter was not used by anyone

@Nicola1971

This comment has been minimized.

Show comment
Hide comment
@Nicola1971

Nicola1971 Aug 8, 2017

is used :)

and is in the system settings too
download

better ask to the community before doing some kind of changes.

modx evo to evocms trasition must be at 100% easy.... at least for now ;)

Nicola1971 commented Aug 8, 2017

is used :)

and is in the system settings too
download

better ask to the community before doing some kind of changes.

modx evo to evocms trasition must be at 100% easy.... at least for now ;)

@Nicola1971

This comment has been minimized.

Show comment
Hide comment
@Nicola1971

Nicola1971 Aug 8, 2017

we need to fix

Nicola1971 commented Aug 8, 2017

we need to fix

@64j

This comment has been minimized.

Show comment
Hide comment
@64j

64j Aug 8, 2017

Member

@Nicola1971 Absolutely agree with you.
We need to look for a solution to this problem.

Enabling the global parameter [(track_visitors)], gives the command to use in their tvars, snippets and plug-ins.

Member

64j commented Aug 8, 2017

@Nicola1971 Absolutely agree with you.
We need to look for a solution to this problem.

Enabling the global parameter [(track_visitors)], gives the command to use in their tvars, snippets and plug-ins.

@q2apro

This comment has been minimized.

Show comment
Hide comment
@q2apro

q2apro Aug 8, 2017

The "fix" is to always have the container open on click. No need to hide this feature that is available for ages already.

q2apro commented Aug 8, 2017

The "fix" is to always have the container open on click. No need to hide this feature that is available for ages already.

@Pathologic

This comment has been minimized.

Show comment
Hide comment
@Pathologic

Pathologic Aug 8, 2017

So it turned out that this parameter was not used by anyone

It's Dmi3yy's hack for his custom build. The right solution is to add column "show_children" and according checkbox.

Pathologic commented Aug 8, 2017

So it turned out that this parameter was not used by anyone

It's Dmi3yy's hack for his custom build. The right solution is to add column "show_children" and according checkbox.

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