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

Problem on new Jem settings>venue page #1878

Closed
jojo12 opened this issue Dec 30, 2024 · 14 comments
Closed

Problem on new Jem settings>venue page #1878

jojo12 opened this issue Dec 30, 2024 · 14 comments
Labels
bug Something isn't working fixed Have a commit with the solution.

Comments

@jojo12
Copy link
Contributor

jojo12 commented Dec 30, 2024

There are two settings which seems to be the same:
Event page>Venues and Venue page!

The venue settings in Event page define, how it looks in Event page detail view (at the end of the events, there is the locality.
Meanwhile the settings in Venue page, define how it looks in the separate Venue page detail view!

It seems, that now these settings are the same!

@jojo12 jojo12 added bug Something isn't working change labels Dec 30, 2024
mckillo added a commit that referenced this issue Dec 30, 2024
Delete evvenues template of event page tab (duplicate with the new venue page tab).
@mckillo
Copy link
Collaborator

mckillo commented Dec 30, 2024

Sorry, I didn't delete the old template in the event page tab. See the commit with the fix.

@mckillo mckillo added the fixed Have a commit with the solution. label Dec 30, 2024
@jojo12 jojo12 removed the fixed Have a commit with the solution. label Dec 30, 2024
@jojo12
Copy link
Contributor Author

jojo12 commented Dec 30, 2024

No, there is a misunderstanding!
Sometimes you want venue details on event details view. This settings are in Event Page setting.
But sometimes you want venue details only on venue details view. This settings are in Venue Page setting.
Important: This settings can be different!!! Like there were before!

@mckillo
Copy link
Collaborator

mckillo commented Dec 30, 2024

But they are the same lines 444 =454:
<?php echo $this->loadTemplate('evvenues'); ?>
Then the error is before, there is to change the template name.

@mckillo
Copy link
Collaborator

mckillo commented Dec 30, 2024

Previously, there was no venue tab setting; the venue setting was located in global parameters tab.
I have to implement a new template for venue setting tab (global). I do it...

mckillo added a commit that referenced this issue Dec 30, 2024
Delete evvenues template of event page tab (duplicate with the new venue page tab).
@mckillo
Copy link
Collaborator

mckillo commented Dec 30, 2024

Please test it. I believe I have already fixed it.

@mckillo mckillo added the fixed Have a commit with the solution. label Dec 30, 2024
@jojo12
Copy link
Contributor Author

jojo12 commented Dec 31, 2024

not yet:
in global parameters there are now 2 titles without content: "Edit events" and "Edit venues". I think, you took them without titles to "venue page". But this isnt's so logic, because they are special editor pages, for this I think they are better in global parameters.
(the admin has to decide, if only own venues are proposed in editor form, and how many custom fields are used in editor form or for events or for venues.)
grafik

@mckillo
Copy link
Collaborator

mckillo commented Dec 31, 2024

Here you are.
Let's end the year with this completed.
Happy New Year!!!

@jojo12
Copy link
Contributor Author

jojo12 commented Jan 1, 2025

Happy New Year too!
But there is something, you perhaps never remarked:
There's a reason that there is two times the setting for maps: one is the map in event detail view and there is an other setting for venue detail view.
Like that, you can use e.g. google route planner on one view and openstreetmap on other view.

With your changes, you made the two views identical. Then you can't set two different views!

EDIT: The lines seem to be the same, that's right,
but the relevant field in event part it is e.g. event_show_mapserv
and the relevant field in venue part is e.g. global_show_mapserv

@jojo12 jojo12 removed the fixed Have a commit with the solution. label Jan 3, 2025
@jojo12
Copy link
Contributor Author

jojo12 commented Jan 3, 2025

Some changes to clarify in general JEM settings:

Event page →
COM_JEM_SETTING_EVENT_PART =»Event Part» instead of COM_JEM_EVENTS
COM_JEM_SETTING_VENUE_PART =»Venue Part» instead of COM_JEM_VENUES
Venue page →
COM_JEM_VENUE_DETAIL =»VenueView» instead of COM_JEM_VENUES

@jojo12
Copy link
Contributor Author

jojo12 commented Jan 3, 2025

only on Venue Page: remove «LINK to Venue (page)» ->Nonsense: because you are just on Venue page)

@jojo12
Copy link
Contributor Author

jojo12 commented Jan 11, 2025

#1877

@mckillo
Copy link
Collaborator

mckillo commented Jan 11, 2025

I think that I have undestand your indication.
For Settings - Event Page tab:

image

For Settings - Venues Page tab:

image

If they are correct, please tell me the content for these strings or confirm if they are correct:

COM_JEM_SETTING_EVENT_PART="Event"
COM_JEM_SETTING_VENUE_PART="Venue in event"
COM_JEM_VENUE_DETAIL="Venue Setting"

@mckillo
Copy link
Collaborator

mckillo commented Jan 11, 2025

I think that the tags should be the following to have the same nommenclature of the jem.ini (BE):
COM_JEM_SETTINGS_EVENT_PART="Event"
COM_JEM_SETTINGS_VENUE_PART="Venue in event"
COM_JEM_SETTINGS_VENUE_DETAIL="Venue Setting"

image

image

@mckillo
Copy link
Collaborator

mckillo commented Jan 11, 2025

See the commit, you can update the string if there are to change them. I hope I got it right.

@mckillo mckillo added fixed Have a commit with the solution. and removed change labels Jan 11, 2025
@jojo12 jojo12 closed this as completed Jan 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed Have a commit with the solution.
Projects
None yet
Development

No branches or pull requests

2 participants