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.

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

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.

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

more diary entry tests, and a minor fix to the relation member that makes the code a bit more readable.

slight more work on the diary test

Postgres adapter, and make migration 022 db-agnostic

nil protection - mysql however has a default for enums, which is why it wasn't a problem before

Made relations ordered. Added some tests for this. Otherwise interface is unchanged.

Ooops, forgot to commit the routes for the changeset query and include calls.

boolean correctness

forgot the test change from 11866

In YAML yes is a boolean keyword: also fix test that was erroneously asserting success

0.6-friendly Potlatch (work in progress). DO NOT SET POTLATCH_USE_SQL=false as it won't work

adding first integration for testing user diaries, as multiple controllers are required. more work required.

remove indexes before renaming tables (some dbs have database-level index namespaces) and unsigned is also a mysqlism

amf get*_history tests and related fixes

a few more amf tests and associated bug fixes

fixing the tests due to a osm -> diffChange and moving to floats

update nodes/ways/relation XML to include user id as well as display name

return user id AND name with changeset get request

migrations: add specific method for nodes/ways/relations enum column

fix SCALE, make changeset GET use floats

require auth for close

cosmetic tweaks to 'login' and 'create account' screens

mini fix

make diff upload result a "diffResult" rather than "osm" document

move index dropping to remove_index method

bumping the version of composite primary keys for the amf controler as there are bugs in it

oops remove my test

Update DB readme for PgSQL a bit

fix accidental commit of local config changes

fix version number return for "modify" sections of diff uploads

1st amf_controller tests, and some basic sanity check fixes

Ignore the bits in tmp/

CREATE FUNCTION statement for pgsql

Support building a postgresql binding for maptile_for_point

cosmetic tweaks to user settings form + move 'Mapper since' field onto user profile view

correcting typo on gps_points fixtures

move fulltext indexes to monkeypatch, since not all dbs have a fulltext analogue

Move db engine changing into db adaptor monkeypatch

last of the auto_increments removed from the migrations

Fixing boo-boo in r11802 - but you can assign integers to booleans on all dbs

Moving AUTO_INCREMENTs to monkeypatching. Seems a bit hacky though

message with a logo and link to Contact page

Boolean correctness in migrations

Make the confrm button into a confirm button

Error message for version mismatch is now more informative.

move bigint_pk id column into table def, no need to be an add_column

fixing one cmd that was missed in the resync

Resyncing from head 10895:11795