Skip to content
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

Grav subpage issues : replicated fault (possibly flex object) #3365

Closed
spamhater-grav opened this issue May 20, 2021 · 4 comments
Closed

Grav subpage issues : replicated fault (possibly flex object) #3365

spamhater-grav opened this issue May 20, 2021 · 4 comments
Assignees

Comments

@spamhater-grav
Copy link

Had a couple of people identify an issue of strange folder numbering prefixing on the forum.

I now been able to identify a couple of linked issues

If you create a numbered sub page / folder off the root under the admin panel, all is ok (due to the existing pages (01.home / 02.typography ) this look ok.

If you then try and create a child to the subfolder / page, when the 'add page' or 'add folder' is displayed selecting the sub folder as a parent, it doesnt update correctly and will still state / display "root"

If you then 'enable foldering numbering' and the save the page. It will display the page with a prefix of 999999.pagename , and this is displayed in the ordering. If you then press the save button, it increments the folder name by 1 everytime.

It doesnt happen if there is a previous or manually created or renamed folder with the correct number (01)
Any further depth down with also produce this issue.

You see the result in the screen recording, a few users have noticed and seem to be related to the recent flex-box updates.

All tested and issue is repeatable on a clean install of grav and users have queried the issue on the community forum.

grav-subpage-numbering-issue.mp4

Many thanks

@pamtbaau
Copy link
Contributor

pamtbaau commented May 21, 2021

Is indeed reproducible. Happens since Grav Core + Admin Plugin v1.7.8

Possibly caused by "Optimized saving visible/ordered pages when there are a lot of siblings #3231" on Grav v1.7.8 ? With commit: 27aff76

Steps to reproduce:

  1. $ wget https://getgrav.org/download/core/grav-admin/1.7.8 .
  2. $ unzip 1.7.8 -d .
  3. Login to site and go to admin/pages/home to open 'Home' page.
  4. Click +Add
  5. In modal form, enter name 'Child' and leave 'Parent page' as 'Home'
  6. Click Continue
  7. Click Enable for field 'Folder Numeric Prefix'
  8. Click Save
  9. Field 'Sortable Pages' will show 999999.Child
  10. Repeat step 8.
    • Field 'Sortable Pages' will show 1000000.Child
    • Etc.

@w00fz w00fz added the bug label May 21, 2021
@spamhater-grav spamhater-grav changed the title Grav subpage issues : replicated fault (possibly flex ojbect) Grav subpage issues : replicated fault (possibly flex object) May 21, 2021
@mahagr
Copy link
Member

mahagr commented May 26, 2021

The first issue is duplicate of #2136 (parent not changing).

@mahagr
Copy link
Member

mahagr commented May 26, 2021

@spamhater-grav @pamtbaau Can you test the fix?

@pamtbaau
Copy link
Contributor

pamtbaau commented May 26, 2021

Using Grav development branch:

  1. $ git clone https://github.com/getgrav/grav -b develop .
  2. $ bin/grav install
  3. $ bin/gpm install admin
  4. Running script as mentioned above
    • Field 'Sortable Pages' shows 1.Child
    • Saving again still shows 1.Child

Ergo, faulty behaviour shown before can no longer be reproduced. Seems you have fixed it...

@mahagr mahagr closed this as completed May 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants