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

mod_menu invalid rendering of items of type alias and url #107

Closed
wants to merge 1 commit into from
Closed

mod_menu invalid rendering of items of type alias and url #107

wants to merge 1 commit into from

Conversation

n3t
Copy link
Contributor

@n3t n3t commented Jan 28, 2012

In Joomla 2.5 mod_menu rendering of menu items of type alias and url calls htmlspecialchars for item url.
But this is already processed by helper of mod%menu by calling JRoute::_() function. It results in invalid urls.

@infograf768
Copy link
Member

Please create a tracker on joomlacode

@elinw
Copy link
Contributor

elinw commented Jan 29, 2012

Unfortunately just reverting will simply recreate the security issue of allowing dangerous strings to be saved.

@realityking
Copy link
Contributor

I'm confused, either the string is double encoded or not. If it was encoded before there shouldn't have been an issue. This doesn't make sense to me.

@infograf768
Copy link
Member

Corrected through new patch in tracker

@infograf768 infograf768 closed this Feb 1, 2012
wilsonge pushed a commit that referenced this pull request Jun 4, 2017
* Check the folder name the same way as with files

* The model should return the name of the created file
bembelimen referenced this pull request in bembelimen/joomla-cms Apr 26, 2018
richard67 pushed a commit to richard67/joomla-cms that referenced this pull request Apr 3, 2020
chmst pushed a commit to chmst/joomla-cms that referenced this pull request May 26, 2020
hans2103 added a commit to hans2103/joomla-cms that referenced this pull request Oct 5, 2020
hans2103 added a commit to hans2103/joomla-cms that referenced this pull request Oct 5, 2020
hans2103 pushed a commit to hans2103/joomla-cms that referenced this pull request Oct 5, 2020
…#107--pagination-remove-borders

Remove the white border on active pagination elements
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
obuisard pushed a commit to obuisard/joomla-cms that referenced this pull request Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants