PHP and ASP AJAX viewer fixes needed for simple search on NULL values - MgProxyFeatureReader.GetProperty line 448

[cgountanis]

AJAX Viewer Fixes for PHP and ASP.NET - Value for property is null. - MgProxyFeatureReader.GetProperty line 448

Will these changes or something like this be implemented in the next release of the basic AJAX viewer? We really should not have to edit the ajaxviewer* code every implementation. Causes many problems with services packs updates and organization.

PHP - Add if statement around the switch($propType) area.

Line 158
if ($features->IsNull($propName))
{
$val="(null)";
}
else
{

ASP.NET - Add if statement around the switch (propType) area. Line 163

if(features.IsNull(prop))
{
val = "(Null)";
}
else
{

[jbirch]

minor formatting

[tomfukushima]

Thanks for suggested fix. I had to rework it so that it didn't have constant strings (globalization) and also added it to the Java environment. As there are no steps or data to reproduce the problem, I will have to rely on someone to test this.

1. Please put the "en" file in the WebServerExtensions/www/localized folder.
2. Put the .php/.asp/.jsp file in the WebServerExtensions/www/mapviewerphp or mapviewernet or mapviewerjava folder (depends on your install).

Once I have all three versions verified, I will get it reviewed further and submit. If you are testing it please make sure you have a case where the "(Null)" strings show up.

[cgountanis]

I could only test on MapGuide Ent 2009 at the molment.

search.aspx Line 72 There is a : before ;. Also the get setclientip and setclientagent did not work on this version of MapGuide. THis may only be an issue with Enterprise version 2009.

Line 154 There is a PHP style $ sign on locale which I think will casue issues.

search.php When you run a search you get internal error. I assume this is due to me running MapGuide 2009 Enterprise and not the latest version of MapGuide Open Source as well. I don't think the older version have SetClient or GetLocalizedString.

To simplify there is no reason to show (null) or any text for that matter. The important part is the IsNull if statement preventing the viewer from crashing. A simple IsNull = true {val=""} would do fine. That might simplify versioning right now.

[tomfukushima]

Goes to mapviewernet directory

[tomfukushima]

Goes to mapviewerjava directory

[tomfukushima]

Goes to mapviewerphp directory

[tomfukushima]

Sorry about that, I didn't realize these files were updated; I removed the use of the problem methods (SetClient, and one other one --- these are not required for testing this anyway). And I've removed the "(null)" string; so now we don't have to worry about globalization. They should now work when you drop them in. Thanks for testing this, Tom

[cgountanis]

Beautiful work! I was able to confirm that the ASPX and the PHP do work with Null and !Null fields from the same feature source. Thank you very much! I will not be able to test JSP without complicating my IIS install.

[tomfukushima]

r3317 for trunk. r3318 for 2.0.2.

I got someone here to test the .jsp page and it worked as well.

Chris, thanks for your help. Tom

[amorsell]

For FUSION. Replace \WebServerExtensions\www\fusion\widgets\Search\Search.php