I have updated the patch to version 2. This update adds a warning message that the scene name contains capital letters which will prevent the course from loading past 50% in JNPG. The link at the bottom of this message has the updated link to the patch.
A couple months ago I became frustrated when I built a course, loaded it into JNPG, only to realize I had turned the meshes off and found Hacker Harry and the ball were floating above a strange looking terrain. Sound familiar? After this happened several times, I decided to look at the scripts to see if a warning message could be displayed during the pre-build tests. While in the scripts, I decided to fix the test for pins not placed on a green spline. It was giving me hole 59, 36, etc., even though I only had 18 holes defined. Of course I began looking for the bad pin starting on hole 1 only to find it was on hole 18. What was happening is it was using the index of the pin in the Pins table as the hole number rather than the actual hole number.
An additional modification was to rename the tee, pin, shot point, etc. identifiers for holes 1 thru 9 to have a leading zero. That way when you turn DEBUG ON, a search for h01 gives you only the objects for hole 1, and they are not intermixed in a list of objects for holes 10 thru 18 as they were previously.
Larry (axe360) recently started to use the patched scripts and located a bug in the original CF script that terminated the pre-build tests if a pin was accidentally placed on the terrain, not on a spline. I had to fire up Visual Studio and learn how to activate the debugger, and fortunately found the culprit line of code. Then I decided to expand the tests for tees that were not placed on a tee spline. We now have tests to display all Rogue pins and tees.
This image demonstrates all of the enhanced messages:
The patch overwrites the "FileUtilUI.cs" and "CourseBase.Plant.cs" scripts located in the Project->Perfect Parallel folder. I believe Challenge tees will be reported as Rogue tees. If that is a problem, I can probably exclude them from the tees test.