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

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