tags around the output result would be cool for checking the hierarchy i would say something like this on line 162 in services_admin_browse.inc

$services_admin_browse_test_submit_result = "
".print_r($result,true)."
";


instead of :


//$services_admin_browse_test_submit_result = check_markup(print_r($result, true), 1);


Greetz sjors
CommentFileSizeAuthor
#1 services.patch877 bytesrobloach

Comments

robloach’s picture

robloach
he/him
commented
Status: Needs work » Needs review
StatusFileSize
new services.patch877 bytes

check_markup seems to strip the tags, but we want to escape them. Adding 

 and switching to escaping the html characters seems to help.....
marcingy’s picture

marcingy commented

I had this issue too I'll test today as I have some work to do a service that was affected by this.

marcingy’s picture

marcingy commented
Status: Needs review » Reviewed & tested by the community

RTBC

marcingy’s picture

marcingy commented
Status: Reviewed & tested by the community » Fixed

committed to 5 and 6

robloach’s picture

robloach
he/him
commented
Version: 5.x-1.x-dev » 6.x-1.x-dev
Component: Code » User interface
Priority: Normal » Minor
Status: Fixed » Patch (to be ported)

Thanks, Marc! Is this already in 6 do you know?

marcingy’s picture

marcingy commented
Status: Patch (to be ported) » Fixed

yes I put it 6 last night

robloach’s picture

robloach
he/him
commented

Ahhh, "committed to 5 and 6". Hahaha, thanks.

marcingy’s picture

marcingy commented
Status: Fixed » Closed (fixed)