From: Richard Fairhurst Date: Wed, 8 Jul 2009 22:29:45 +0000 (+0000) Subject: "list mode" should really be "live mode" :) X-Git-Tag: live~6890 X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/8eb77ef5e133e90423ac5048f1e21933d657db01 "list mode" should really be "live mode" :) --- diff --git a/app/controllers/amf_controller.rb b/app/controllers/amf_controller.rb index 0b5e3a3ce..3fa47d088 100644 --- a/app/controllers/amf_controller.rb +++ b/app/controllers/amf_controller.rb @@ -598,14 +598,6 @@ class AmfController < ApplicationController # 6. hash of node versions (node=>version) def putway(renumberednodes, usertoken, changeset_id, wayversion, originalway, pointlist, attributes, nodes, deletednodes) #:doc: - -logger.info("renumberednodes is #{renumberednodes.inspect}") -logger.info("wayversion is #{wayversion}") -logger.info("pointlist is #{pointlist.inspect}") -logger.info("sendnodes is #{nodes.inspect}") -logger.info("deletednodes is #{deletednodes.inspect}") - - amf_handle_error("'putway' #{originalway}") do # -- Initialise diff --git a/config/locales/en.yml b/config/locales/en.yml index c1a0d17f5..7b9d40e28 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -604,7 +604,7 @@ en: anon_edits_link: "http://wiki.openstreetmap.org/wiki/Disabling_anonymous_edits" anon_edits_link_text: "Find out why this is the case." flash_player_required: 'You need a Flash player to use Potlatch, the OpenStreetMap Flash editor. You can download Flash Player from Adobe.com. Several other options are also available for editing OpenStreetMap.' - potlatch_unsaved_changes: "You have unsaved changes. (To save in Potlatch, you should deselect the current way or point, if editing in list mode, or click save if you have a save button.)" + potlatch_unsaved_changes: "You have unsaved changes. (To save in Potlatch, you should deselect the current way or point, if editing in live mode, or click save if you have a save button.)" sidebar: search_results: Search Results close: Close diff --git a/public/potlatch/potlatch.swf b/public/potlatch/potlatch.swf index cd73235f9..e633ccd2e 100755 Binary files a/public/potlatch/potlatch.swf and b/public/potlatch/potlatch.swf differ diff --git a/test/functional/amf_controller_test.rb b/test/functional/amf_controller_test.rb index fbc0af883..ab0f5d5ad 100644 --- a/test/functional/amf_controller_test.rb +++ b/test/functional/amf_controller_test.rb @@ -311,21 +311,18 @@ class AmfControllerTest < ActionController::TestCase history = amf_result("/1") # ['node',nodeid,history] + # note that (as per getway_history) we actually round up + # to the next second assert_equal history[0], 'node', 'first element should be "node"' assert_equal history[1], latest.id, 'second element should be the input node ID' - # NOTE: changed this test to match what amf_controller actually - # outputs - which may or may not be what potlatch is expecting. - # someone who knows potlatch (i.e: richard f) should review this. - # NOTE2: wow - this is the second time this has changed in the - # API and the tests are being patched up. assert_equal history[2].first[0], - latest.timestamp.strftime("%d %b %Y, %H:%M:%S"), - 'first part of third element should be the latest version' + latest.timestamp.succ.strftime("%d %b %Y, %H:%M:%S"), + 'first element in third element (array) should be the latest version' assert_equal history[2].last[0], - nodes(:node_with_versions_v1).timestamp.strftime("%d %b %Y, %H:%M:%S"), - 'second part of third element should be the initial version' + nodes(:node_with_versions_v1).timestamp.succ.strftime("%d %b %Y, %H:%M:%S"), + 'last element in third element (array) should be the initial version' end def test_getnode_history_nonexistent