further work on 0.6 history (not quite complete yet)

Test and fix for #1567, for teh betterer osmChange compliance.

Test and fix for issue #1568. Wasn't testing for end element in the right place.

Fix for #1565. Added a couple of tests and fixed another issue with re-used placeholder IDs.

Fixed bug in changeset closing and querying where the number of elements exceeded the maximum. Added a fixture for this.

splitting and joining tags is gone for good in 0.6

Copying across fix for #1547 from trunk

fixed typo

Set params to sane defaults if unset by client.
Reject creates with no file with 400, model validations will also reject with 400 (as was existing but broken behaviour)
Closes #1510

Require auth on calls to /trace/create, and pass through to form if called without params. Adjust validations on traces to prevent dual error messages on description (validates_presence_of catches the empty string removed from length validation)
More changes to come for api_create
References #1510

Move the ActiveRecord::RecordNotFound to the block, so that it catches the not found in line 327, when a nonexistant way id is searched for.

small versioning fix (plus history rework in progress)

Revert last commit - too many test failures

DateTime => Time for consistency in changeset code, ok by shaun.

Only show changesets that are actually closed in the index view.

removed comments which apparently no-one bothers to read anyway. since no-one reads the comments, changed changeset model to use Time instead of DateTime. will probably have to change it back later, when someone else who doesn't read comments comes along.

Make browse controller index show recently closed changesets, rather than recently changed nodes.

Pass the to_user to the view. Tests may follow. Closes #1394

session management on only for :list

view changesets by user

Relations support

remove debug box

Potlatch for API 0.6 now saving ways happily with node versioning... I think

some more putway fixes - this actually creates new ways with 0.6 Potlatch now

Fix UTF-8 encoding error char in source file that RichardF introduced

bit more putway stuff

yet more rewriting of putway...

Add validations for diary entry lat/lons and tests for diary entry validations. Closes #1395

Some extra stubs in the amf controller test

Fix the transactions, need to set anything that will be returned to nil outside the transaction, otherwise it won't be available after the end of the transaction, when it is returned.

bug fix. Adding some transactions to amf controller, and adding the missing version to putway method signature.

Return, rather than allow a second render to happen. Closes #1407

resync from rails_port 11795:12304

Log the request on a few requests when there is a bad request, probably should do this in a few more places. This makes it easier to see why we are getting errors/bad requests.

removing debug statement

Improving the not found handling of preferences. adding a user preference test. adding some utility methods to the  test helper

Add some extra indexes.

Optimise lookup of traces with a given tag.

capture the too many way nodes error, and give the user an error message

Making amf controller act more like the xml apis

Don't give an error when you access the login page when you are logged in. Instead take you to the page in the referer or the home page.

some more progress towards making amf_controller do version checking of nodes before deleting them

some further progress to amf controller checking the version numbers.

Added missing version to the pure rails AMF controller. More stringent tests on whichways.

Some fixes to the rails part of the AMF controller and associated test.

Some improvements for the XML parsing, as there are somethings that we don't need, like timestamp and visibility, thus hopefully bringing a speedup in the xml parsing. Some more AMF controller fixes, plenty more to go though.

count -> size for compatibility with ruby 1.8.6

the user id may be negative, so that osmosis can create changesets for users that are anonymous, which it uses a negative id for, as a special id.

Adding the ability to see the min/max lat/lon for a changeset in the data browser. Adding file for the nice message that should show when a user is missing in the message controller.

Adding some amf create poi tests. Minor bugfix tweaks to the amf controller

Fix for the refactoring commited yesterday, based on review by Dave Stubbs

Refactoring the amf controller, and all models so that save_with_history! is now private. Everyone should use create_with_history, delete_with_history, and update_from instead. Also gets the amf controller to do version checks in the node/way/relation models. Needs checked. This breaks Potlatch until #1376 is fixed.

turning the session off for the changeset controller, to come in line with the other api controllers

Adding missing fixtures to test helper.

Deal with active record not found exceptions when creating message. Some tidying of html. For some reason in rails 2.1.2 having a comment at the end of a line, in the view,  means that the closing tag needs to be on the next line.

adding link to download the full changeset xml

Actually set the instance var when closing the changeset

ensure that uploads that don't supply a lat and lon for a node. Adding related test and fixing other tests.

better handling of duplicate tags. Extra validation in the tests.

Changed duplicate tags test to test the correct code path - was previously failing in libxml, rather than the duplicate tags check.

Adding pagination to the changesets, thus allowing you to browe large changesets, without MySQL complaining about memory. Probably needs some fine tuning. You currently can only page one of nodes, ways, and relations, without manually setting the relations. Fixing deprection notice for the classic_pagination plugin.

Adding test to check that users can't update changesets that they don't own.

Moved transaction boundary to cover used-by tests on deletion so that the database can help prevent race conditions.

Moved changeset consistency checks to library code.

Renaming the include action to expand_bbox in the changeset controller.

Add a delete method for the user model.

Some improvements to the error messages that are returned by the api.

More model validations. Fixing one test.

various fixes/enhancements, including a couple to the 0.5 version

various fixes/enhancements, including a couple to the 0.5 version

Make the role in relations optional, with a test to make sure it is. Also start moving the errors reading the xml to exceptions, thus making it possible to give meaningful error messages, when bad xml is sent (More work is required on this including doing the same for nodes and ways). With the latest gems update it seems that the lib xml handling was broken, using the newer method. Adding the content type for the exceptions.

cosmetic tweaks to diary interface. icon for new entries link

Fixed bug in changeset idle timeout. Fixed another with a spurious require.

Closes #1350 through updating amf_controller to use the newer closed_at method of determining whether a changeset is closed. Richard please check the rest of the amf controller for more things that may reference the open field that is no longer available. Use is_open? to determine if a changeset is open. Fix indentation in amf controller. Fix minor bug in the links to changesets in the data brower.

Changeset tag unit tests

More tag validations, and finishing the node/way/relation tests.

Remember whether the user set the public checkbox on GPX upload as a user preference. Closes #827

Relation Tag testing. Also sort the belong_to/has_many for user/changeset/old_way.

Require the user to be logged in for those methods which make no
sense otherwise.

Fix syntax of status only renders. Closes #1345.

Validations on the way tags, with unit tests

Make the dates and times that the server deals with UTC. This is probably required for changeset, as they could auto close instantly when the clocks go forward by one hour.

Migration to add close-time to changesets. This replaces the boolean 'open' attribute. Added checks to ensure that the maximum lifetime and number of changes in a changeset are enforced. Added some tests.

Adding validations for the length of node tags, and the presence of the id (and version). More tests, including some boiler plate tests.

Made user input parsing more robust in changeset query method. Added tests.

Verify that the key and value isn't getting mixed up by having them different. verify the error message is the same when returning the user conflict on closing changeset test.

Fixed some bugs in changeset query code. Added more test cases.

Added tests for changeset close method.

Implemented changeset tags updating via the update method.

Altered tests on AMF controller so that they give failures instead of errors. Return format seems to have changed, so someone who knows more about AMF controller should have a look.

changeset comments and a couple of other small fixes

Freezing composite primary key so that we can apply TomH's patch to it, and that way Potlatch will work in Rails mode without a problem. Please test.

finished Rails-friendly amf_controller. Note that this requires Tom's patched composite_primary_keys to work unless you tell Potlatch to use SQL reads

More unit test stubs

Unit tests work again. Fixed the set fixture class that I should. Adding 2 length checks to the user preference controller.

Adding some more unit test stubs that need filling out. Adding changeset tags fixture. Indentation fix in way model.

Adding some more diary entry tests. Making the RSS feed links more portable, by using the SERVER_URL constant. Fixing the validation for the length of the language of diary entries until the i18n branch is integrated.

Covering the case of an entry not being found. More diary entry tests.

replaced "wtf does this do" comment by something explaining what it does.

more diary entry test