Hi!

Looks like there's a bug in the Drupal 7 display of the xWeb Testing form. In certain cases, the parameters form does not display. Methods with simple arguments like GetQuery work just fine. But the parameters form doesn't display at all if the method arguments are nested objects -- like WEBCentralizedShoppingCartGetPaymentOptions.

The cause seems to be a change in the way collapsable field sets are put together in D7. Collapse.js now expects span.fieldset-legend and div.fieldset-wrapper inside the field set. See http://drupal.org/node/1424350#comment-5548762.

The patch provided adds these elements and makes the test form appear once again.

Even with this patch, one problem remains -- collapsing does not work. Collapse.js doesn't seem to add the collapse links to the field set legends. FireFox throws a slow script execution warning, so it may be that the form is just too complex.

In any case, this patch will at least show the NetForum arguments once again.

CommentFileSizeAuthor
#3 netforum-xweb_test_fails_to_show_objects-1513398-3.patch2.5 KBelstudio
#2 netforum-xweb_test_fails_to_show_objects-1513398-2.patch10.7 KBelstudio
#1 netforum-xweb_test_fails_to_show_objects-1513398.patch10.89 KBelstudio

Comments

elstudio’s picture

elstudio commented
StatusFileSize
new netforum-xweb_test_fails_to_show_objects-1513398.patch10.89 KB

Patch attached.

elstudio’s picture

elstudio commented
StatusFileSize
new netforum-xweb_test_fails_to_show_objects-1513398-2.patch10.7 KB

Let's try that patch again, without the junk.

elstudio’s picture

elstudio commented
StatusFileSize
new netforum-xweb_test_fails_to_show_objects-1513398-3.patch2.5 KB

Third time's a charm?

Let's ignore whitespace differences in the patch so you can see what's important.

elstudio’s picture

elstudio commented
Status: Patch (to be ported) » Needs work
james.michael-hill’s picture

james.michael-hill commented
Status: Needs work » Closed (fixed)

Awesome, thanks for putting this in. I've applied it and I hope, fixed some of the issues using it in the admin overlay. There is a 1.1 build scheduled that should include this.