Fixed bug that stan noted:

* Readme says the plugin has only been tested with 2.1, but that should be 2.2.
* Modify source code so that all lines in each file uses the same end-of-line markers.
* Parent categories do not display the number of incidents in its child categories. It displays 0, which isn't helpful when categories are collapsed.
* It would also be useful to explain the %20 in the help file and readme. Most users probably don't know that is how to URLify a space.
* controllers/admin/kml_settings.php comments: 1) Purpose, Inputs, and Outputs comments are too vague and don't convey any information. Please also address these questions in comments: 1) what is the purpose of the temp files? If files are written/read, that should be included as an inputs/outputs. 2) The POST parameters comment doesn't supply any information. You should list each POST parameter -- something like that is very hard to pick our of the code. Variables that are visible form fields may not need further explanation (beyond naming), but other variables probably do 3) Validation could benefit from a few words. It isn't at all clear from the Kohana code what is going on here, and we want users of all levels to understand the code.
* controllers/admin/kml.php comments: 1) date in header and changelog is not up to date. 2) This is a good example for params comments (and other comments, too).
* Brokering System - Completed: The readme file says that completed=true excludes completed reports. That is backwards. It also says that completed=false does no filtering. That does not meet the requirement to show just incomplete reports (all minus complete) (req. 6.12). Ideally, the KML file would exclude completed reports by default (i.e., no extra GET parameters) whenever the brokering plugin is active. Also, line 238 of kml.php "$incidents = $incidents->where($table_prefix.'qn_responsible.completed', '0');", the 0 should be a 1. When an incident is completed, the value is 1.
* kml_helper.php: the Purpose, Input, and Output comments are vague and non-informative. I didn't go through every file looking at these particular comments. Please review the files I didn't call out specifically and see if the comments can be made a little more informative.
* The readme and help page should mention that the l paramerter returns the n most recent reports (I think that is the way it works).
* The readme and help file should also mention the necessity of using "&" when using more than one parameter. Help the less-savvy user…
* yoursite/kml/help does not generate valid html. There is no header section, the "headings are not contained in valid tags (should be <p>, <h>, or even <li> tags. <strong> is not a block-level container.
* Please remove the Ushahidi logo from the KML. It inteferes with display on non-Google platforms (GeoIQ), and it just takes up precious space in the bubble.
* Icons (place markers) don't display in Google Earth when type=kmz&style=plain
* When the brokering system is installed put [c] and [I] in front of report names if they are complete or incomplete.
some more modifications to the comments making it clear that most classes, in and of themselves, don't do anything until you call a method in that class.