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

Version 1.3 does not work with multiple pages on my installation #2

Closed
mpwt opened this issue Jan 13, 2023 · 4 comments
Closed

Version 1.3 does not work with multiple pages on my installation #2

mpwt opened this issue Jan 13, 2023 · 4 comments
Labels
bug Something isn't working

Comments

@mpwt
Copy link

mpwt commented Jan 13, 2023

Hi Carmen, currently I use version 1.2 for two sites. With version 1.3 I have not been able to get more than one singular page to work. When I update my two existing pages, both pages show the same content (of the second page).

I can create another page with version v1.3 in addition to the two pages with v1.2. This new page is displayed correctly.

Is there anything new to note or a trick how I can update to the new version for more than one page?

I use webtrees 2.1.15 with justlight theme, some vesta add-ons, three charts of magicsunday and PHP 8.1. The link to my webtrees page is https://ffp.bauschaffen.de. Thanks in advance.

@JustCarmen
Copy link
Owner

Tested before release and then it worked, I thought. But I can't get it to work again either. I thought it would be easier if it was not necessary to change the constant ROUTE_URL after each upgrade. So I put a variable part in it, but unfortunately that is not the solution.

It's the same problem as JustCarmen/webtrees-fancy-treeview#114. According to Greg, you have to hard code the page type in the ROUTE_URL. And I thought I found a nice solution for this module by making the page type partially variable. So that users with multiple pages don't have to change the code after each upgrade. Unfortunately it doesn't work.

I need to fix this.

@mpwt
Copy link
Author

mpwt commented Jan 14, 2023

Why don't you read the module name from the module settings stored in the database? There per module "jc_simple_menu_X" is stored.

@JustCarmen
Copy link
Owner

JustCarmen commented Jan 14, 2023

That is not possible. It has to be a hard coded name. See Greg's answer here: JustCarmen/webtrees-fancy-treeview#114.

@mpwt
Copy link
Author

mpwt commented Jan 14, 2023

Good, then you should revert the commit 68b4afa and add in the description that an adjustment in the file modul.php is necessary, if more than one page should be created.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants