I discovered this issue while working on a jump menu that launches applications such as Oracle applications. The jump menu converts the links that have a secondary colon (ASCII character) in it to an HTML equivalent (i.e., url ending in 102:1 is converted to url ending in 102%3A1). This is a problem.

I don't know how to write patches but here is my recommendation to 7.x-3.3.

After line 113: $path = url($url_options['path'], $url_options);
Add line and write on line 114 : $path=htmlspecialchars(strip_tags(urldecode($path)));

I tested this and it works.
E.

Comments

erok415’s picture

Sorry to not mention which file to apply this to.

Apply to /plugins/views_plugin_style_jump_menu.inc

erok415’s picture

Assigned: Unassigned » erok415
Status: Active » Needs review
StatusFileSize
new687 bytes

I created a patch that seems to fix the issue.

dawehner’s picture

Status: Needs review » Needs work
+++ b/plugins/views_plugin_style_jump_menu.incundefined
@@ -112,6 +112,8 @@ class views_plugin_style_jump_menu extends views_plugin_style {
+		// Fix bad urls due to ASCII character mis interpretation
+		$path=htmlspecialchars(strip_tags(urldecode($path)));

Drupal uses spaces instead of tabs ...

If we change something here we might have to change this in style_jump_menu_summary as well. Can't we use similar code then below?

akshita’s picture

can anyone got this resolved? if so please help.

I am on drupal 7. on one of my Jobpostings nodes I have a field to enter the URL but when entered the URL (APEX URL Format)

I am getting the error : The value in Link is not a valid url.

URL : http://j2eetest:7782/pls/apex/f?p=136:10:0::NO::P10_LGR_JOB_ID,P10_POSITIONTYPE,P10_LGR_WRITING_TST_IND:18,7,Y

Please do the needful.