Evan Prodromou [Thu, 3 Feb 2011 15:46:56 +0000 (10:46 -0500)]
BlankAd uses plugins server
Evan Prodromou [Thu, 3 Feb 2011 15:36:25 +0000 (10:36 -0500)]
new methods for paths to plugin static files
Brion Vibber [Tue, 1 Feb 2011 22:37:46 +0000 (14:37 -0800)]
Merge branch '3022' into testing
Brion Vibber [Tue, 1 Feb 2011 22:35:42 +0000 (14:35 -0800)]
Ticket #3022: fix formatting output for ApiAction::clientError and ApiAction::serverError when caller doesn't explicitly pass the format.
Format's already available as a member variable, so use it!
Fixes some error reponses in api/statusnet/groups/leave.json which were coming through as XML.
May fix some others as well.
Zach Copley [Tue, 1 Feb 2011 02:57:19 +0000 (18:57 -0800)]
API - Return integers instead of strings for group IDs and DM sender/recipients in JSON output
Zach Copley [Mon, 31 Jan 2011 23:50:22 +0000 (23:50 +0000)]
Remove old Facebook Plugin (use FacebookBridge now)
Zach Copley [Mon, 31 Jan 2011 23:42:43 +0000 (23:42 +0000)]
FacebookPlugin: Fix up FBML canvas app so it keeps working after
Facebook removed Profile Boxes and some API calls it relied upon.
See: http://developers.facebook.com/roadmap/deprecations
Brion Vibber [Mon, 31 Jan 2011 22:00:22 +0000 (14:00 -0800)]
Fix "$s"s that slipped into double-quoted translatable strings' '%1$s' pattern. Switch to single-quote to fix.
Brion Vibber [Mon, 31 Jan 2011 21:12:56 +0000 (13:12 -0800)]
Performance counters: records number of total and unique cache get/set/incr/deletes and queries, and logs to syslog.
$config['site']['logperf'] = true; // to record & dump total hits of each type and the runtime to syslog
$config['site']['logperf_detail'] = true; // very verbose -- dump the individual cache keys and queries as they get used (may contain private info in some queries)
Seeing 180 cache gets on a timeline page seems not unusual currently; since these run in serial, even relatively small roundtrip times can add up heavily.
We should consider ways to reduce the number of round trips, such as more frequently storing compound objects or the output of processing in memcached.
Doing parallel multi-key lookups could also help by collapsing round-trip times, but might not be easy to fit into SN's object model. (For things like streams this should actually work pretty well -- grab the list, then when it's returned go grab all the individual items in parallel and return the list)
Brion Vibber [Mon, 31 Jan 2011 20:22:50 +0000 (12:22 -0800)]
Use cachedQuery on File::getAttachments, plus other cleanups:
* dropped unnecessary join on notice table
* made the function actually static, since it makes no sense as an instance variable. The only caller (in AttachmentList) is updated.
Brion Vibber [Mon, 31 Jan 2011 19:50:24 +0000 (11:50 -0800)]
Revert "Session GC fix: save session.modified field as UTC so our comparisons work." - no longer needed with ticket #3020 fix to time zone settings
This reverts commit
a7abb2323e7d57125b9fbc903a1cecc06c27944e.
Brion Vibber [Mon, 31 Jan 2011 19:50:06 +0000 (11:50 -0800)]
Merge branch 'master' into testing
Conflicts:
classes/Profile.php
Brion Vibber [Mon, 31 Jan 2011 19:45:19 +0000 (11:45 -0800)]
Fix for ticket #3020: set MySQL session time_zone variable to UTC ('+0:00') so TIMESTAMP column values are comparable against our other UTC timestamp values.
MySQL stores TIMESTAMP columns as UTC, but with a local time interface. (SRSLY?!) DATETIME columns are always bare and assumed to be local time, but we keep only UTC in them.
Forcing the session time_zone to UTC means we won't have to worry as much about what we're sending/receiving in there.
Also will let us remove the hack in master commit
a7abb2323e7d57125b9fbc903a1cecc06c27944e for session tweaks
Brion Vibber [Mon, 31 Jan 2011 19:09:20 +0000 (11:09 -0800)]
normalize line endings in shiny/css/display.css
Brion Vibber [Mon, 31 Jan 2011 19:08:15 +0000 (11:08 -0800)]
Merge branch '0.9.x' into testing
Brion Vibber [Mon, 31 Jan 2011 19:03:46 +0000 (11:03 -0800)]
Merge branch 'testing' of gitorious.org:statusnet/mainline into testing
Brion Vibber [Mon, 31 Jan 2011 19:02:29 +0000 (11:02 -0800)]
Update jquery.form.js to 2.63 -- 2.60 fixed our Opera regression per issue #3015
Thanks to the fine folks at http://forum.jquery.com/topic/regression-form-plugin-ajaxform-cannot-access-xml-return-data-in-opera#
14737000001950332 :D
Evan Prodromou [Mon, 31 Jan 2011 19:02:17 +0000 (14:02 -0500)]
Merge branch 'runtime' into testing
Evan Prodromou [Mon, 31 Jan 2011 18:59:38 +0000 (13:59 -0500)]
add a comment to show runtime at the end of a page
Siebrand Mazeland [Mon, 31 Jan 2011 17:43:39 +0000 (18:43 +0100)]
Update translator documentation, remove FIXME and add comment.
Siebrand Mazeland [Mon, 31 Jan 2011 17:16:55 +0000 (18:16 +0100)]
* add/update translator documentation.
* remove superfluous whitespace.
* update punctuation on form validation messages.
* L10n tweaks.
* add fixme for missing class documentation.
Siebrand Mazeland [Mon, 31 Jan 2011 16:12:39 +0000 (17:12 +0100)]
* tag i18n issue.
* add translator documentation.
* remove superfluous whitespace.
Siebrand Mazeland [Mon, 31 Jan 2011 15:39:54 +0000 (16:39 +0100)]
* update/add translator documentation.
* fix L10n issues.
* update comments and tag missing documentation on some classes.
* remove superfluous whitespace
*
Siebrand Mazeland [Sun, 30 Jan 2011 23:04:03 +0000 (00:04 +0100)]
Localisation updates from http://translatewiki.net.
Siebrand Mazeland [Sun, 30 Jan 2011 18:03:55 +0000 (19:03 +0100)]
* Add/update translator documentation.
* L10n tweaks (mostly domain MENU assignments).
* Remove superfluous whitespace.
Siebrand Mazeland [Sun, 30 Jan 2011 18:01:55 +0000 (19:01 +0100)]
* Add/update translator documentation.
* Update punctuation in form validation message for consistency.
* Remove superfluous whitespace.
Siebrand Mazeland [Sun, 30 Jan 2011 17:48:09 +0000 (18:48 +0100)]
* update punctuation for translator documentation.
* remove superfluous whitespace.
Siebrand Mazeland [Sat, 29 Jan 2011 22:10:15 +0000 (23:10 +0100)]
Localisation updates from http://translatewiki.net.
Siebrand Mazeland [Fri, 28 Jan 2011 23:33:13 +0000 (00:33 +0100)]
* fix some i18n and L10n issues
* update/add translator documentation
* remove superfluous whitespace
Brion Vibber [Thu, 27 Jan 2011 20:27:31 +0000 (12:27 -0800)]
Session GC fix: save session.modified field as UTC so our comparisons work.
Had to tweak statusnet.ini to remove the DB_DATAOBJECT_MYSQLTIMESTAMP bitfield constant on session.modified; while it sounds like a useful and legit setting, it actually just means that DB_DataObject silently fails to pass through any attempts to explicitly set the value. As a result, MySQL does its default behavior which is to insert the current *LOCAL* time, which is useless.
This was leading to early GC west of GMT, or late GC east of it. Early GC could at worst destroy all live sessions (whoever's session *triggered* GC is fine, as the session then gets saved right back.)
Brion Vibber [Thu, 27 Jan 2011 20:07:29 +0000 (12:07 -0800)]
Add $config['sessions']['gc_limit'] to limit how much work we do in each session GC; defaulting to killing 1000 sessions at a time.
Brion Vibber [Wed, 26 Jan 2011 23:53:05 +0000 (15:53 -0800)]
Merge branch 'testing' into filesize
Brion Vibber [Wed, 26 Jan 2011 23:49:57 +0000 (15:49 -0800)]
Fix ticket #3013: MAX_FILE_SIZE hidden fields were incorrectly placed
In order to apply to PHP's POST processing, the MAX_FILE_SIZE field must appear *before* the file upload field. They were incorrectly placed after, where they had no effect on POST processing.
Brion Vibber [Wed, 26 Jan 2011 22:59:35 +0000 (14:59 -0800)]
Merge branch 'testing' of gitorious.org:statusnet/mainline into testing
Brion Vibber [Wed, 26 Jan 2011 22:57:52 +0000 (14:57 -0800)]
Fix ticket #3016: when using non-AJAX form of the ostatus subscription initiation for non-local group joins, show the "group" field instead of a blank "user" field
Brion Vibber [Wed, 26 Jan 2011 22:57:52 +0000 (14:57 -0800)]
Fix ticket #3016: when using non-AJAX form of the ostatus subscription initiation for non-local group joins, show the "group" field instead of a blank "user" field
Zach Copley [Wed, 26 Jan 2011 18:49:14 +0000 (10:49 -0800)]
Add IdentiCurse to notice sources
Zach Copley [Wed, 26 Jan 2011 18:49:14 +0000 (10:49 -0800)]
Add IdentiCurse to notice sources
Zach Copley [Wed, 26 Jan 2011 18:50:05 +0000 (10:50 -0800)]
Merge branch '0.9.x' of gitorious.org:statusnet/mainline into 0.9.x
* '0.9.x' of gitorious.org:statusnet/mainline: (102 commits)
Fix for ticket #3010: blocks are now applied against the original poster of repeats.
Fix XML API output for several profile update methods that returned a <user> entry but didn't set namespaces, causing XML parse failures.
Fix for ticket #3007: .bmp avatar uploads weren't being properly converted to PNG in all cases
Bookmark saving robustness fixes
remove boilerplate from NewMenuPlugin
Localisation updates from http://translatewiki.net.
L10n consistency updates in wording and punctuation. Translator documentation added/updated. Superfluous whitespace removed.
Add translator documentation Fix L10n issues Remove superfluous whitespace
Add correct punctuation for client exceptions.
Add correct punctuation for client exception.
Add correct punctuation for client exception.
Add email field to Twitter registration form; needed when RequireValidatedEmail plugin is present.
Add email field on openid registration; needed to register if RequireValidatedEmail plugin is also present.
Event hook points needed for recaptcha on facebook login form (untested, but should be legit -- same adds as openid & twitter reg forms)
Event hook points needed to run Recaptcha on Twitter registration
Fix inconsistent use of 'name' vs 'fullname' in tw_fields member variable
Add Start/EndRegistrationData event hooks in finishopenidlogin: allows recaptcha to add its captcha display to the form (checked since addition of StartRegistrationTry)
Ticket #2999: RequireValidatedEmail plugin now also prevents group creation by unvalidated users.
Localisation updates from http://translatewiki.net.
Translator comments added L10n updates Remove superfluous whitespace Number parameters in message when two or more are used ClientException and ServerException should end with a period
...
Zach Copley [Wed, 26 Jan 2011 18:49:14 +0000 (10:49 -0800)]
Add IdentiCurse to notice sources
Brion Vibber [Wed, 26 Jan 2011 00:26:56 +0000 (16:26 -0800)]
Ticket #3014: check upload size limits client-side in supporting browsers
Tested successfully in:
* Firefox 4.0b9
* Safari 5.0.5
* Chrome 8.0.522
Samantha Doherty [Tue, 25 Jan 2011 23:33:47 +0000 (18:33 -0500)]
Wee style update for bookmarks plugin.
Samantha Doherty [Tue, 25 Jan 2011 23:32:12 +0000 (18:32 -0500)]
A few wee fixes for rebase and shiny themes.
Samantha Doherty [Tue, 25 Jan 2011 23:29:07 +0000 (18:29 -0500)]
Rounded corner fixes for base theme.
Brion Vibber [Tue, 25 Jan 2011 20:57:49 +0000 (12:57 -0800)]
Merge branch '0.9.x' into testing
Brion Vibber [Mon, 24 Jan 2011 22:16:15 +0000 (14:16 -0800)]
Fix for ticket #3010: blocks are now applied against the original poster of repeats.
Previously, if someone you subscribe to repeats a notice by someone you've blocked, you got the message and had to just roll your eyes.
Now blocks are checked against both the current notice's posting profile, and the poster of the original if it's a repeat.
Brion Vibber [Mon, 24 Jan 2011 20:41:30 +0000 (12:41 -0800)]
Fix XML API output for several profile update methods that returned a <user> entry but didn't set namespaces, causing XML parse failures.
Brion Vibber [Mon, 24 Jan 2011 20:22:47 +0000 (12:22 -0800)]
Fix for ticket #3007: .bmp avatar uploads weren't being properly converted to PNG in all cases
Part of the reported issue was previuosly fixed by
dc497ed0 (smaller size images being blanked).
This commit fixes the remaining bug with original-size avatars being left as BMP (which could include the 96px size for instance, which could cause problems in browsers not supporting BMP natively)
Added ImageFile::copyTo() as a convenient alias for resizeTo() when not resizing; this performs the BMP/XPM/XBM->PNG conversion if needed, or copies the original file.
Copying instead of using move_uploaded_file() is fine here since:
a) the files are cleaned up on script completion anyway (vs moving to remove it)
b) we're already performing getimagesize() and possibly load/resize on the file before this point (vs needing to move the file into a usable area to work with open_basedir restrictions that prevent working directly with uploaded files in the temp dir; since this would fail anyway, we lose nothing)
ImageFile::preferredType() now works on $this->type instead of asking for one, to make it handier to use from outside. (This is still needed in order for calling code to generate a target filename.)
Recommended for future:
* additional consolidation between the various ways of uploading avatars (touched avatarsettings, grouplogo, and apiaccountupdateprofileimage with similar minor changes)
* consolidate type checks and file naming into Avatar class
Evan Prodromou [Mon, 24 Jan 2011 13:54:09 +0000 (06:54 -0700)]
Merge remote branch 'gitorious/testing' into testing
Evan Prodromou [Sun, 23 Jan 2011 17:57:20 +0000 (12:57 -0500)]
Bookmark saving robustness fixes
First, if the tags box is empty, don't save an empty tag for the notice.
Second, if URL shortening fails, just use the regular URL.
Evan Prodromou [Sun, 23 Jan 2011 17:03:28 +0000 (12:03 -0500)]
remove boilerplate from NewMenuPlugin
Siebrand Mazeland [Sat, 22 Jan 2011 19:17:19 +0000 (20:17 +0100)]
Localisation updates from http://translatewiki.net.
Siebrand Mazeland [Fri, 21 Jan 2011 21:45:37 +0000 (22:45 +0100)]
L10n consistency updates in wording and punctuation.
Translator documentation added/updated.
Superfluous whitespace removed.
Siebrand Mazeland [Fri, 21 Jan 2011 15:35:00 +0000 (16:35 +0100)]
Add translator documentation
Fix L10n issues
Remove superfluous whitespace
Siebrand Mazeland [Fri, 21 Jan 2011 00:18:39 +0000 (01:18 +0100)]
Add correct punctuation for client exceptions.
Siebrand Mazeland [Fri, 21 Jan 2011 00:16:30 +0000 (01:16 +0100)]
Add correct punctuation for client exception.
Siebrand Mazeland [Fri, 21 Jan 2011 00:14:53 +0000 (01:14 +0100)]
Add correct punctuation for client exception.
Brion Vibber [Fri, 21 Jan 2011 01:02:34 +0000 (17:02 -0800)]
Add email field to Twitter registration form; needed when RequireValidatedEmail plugin is present.
Since Twitter doesn't provide email address back to us here, we only prefill the field if we have an invite.
Brion Vibber [Fri, 21 Jan 2011 00:56:48 +0000 (16:56 -0800)]
Add email field on openid registration; needed to register if RequireValidatedEmail plugin is also present.
Brion Vibber [Thu, 20 Jan 2011 23:55:36 +0000 (15:55 -0800)]
Event hook points needed for recaptcha on facebook login form (untested, but should be legit -- same adds as openid & twitter reg forms)
Brion Vibber [Thu, 20 Jan 2011 23:49:57 +0000 (15:49 -0800)]
Event hook points needed to run Recaptcha on Twitter registration
Brion Vibber [Thu, 20 Jan 2011 23:47:51 +0000 (15:47 -0800)]
Fix inconsistent use of 'name' vs 'fullname' in tw_fields member variable
Brion Vibber [Thu, 20 Jan 2011 23:40:59 +0000 (15:40 -0800)]
Add Start/EndRegistrationData event hooks in finishopenidlogin: allows recaptcha to add its captcha display to the form (checked since addition of StartRegistrationTry)
Brion Vibber [Thu, 20 Jan 2011 23:12:57 +0000 (15:12 -0800)]
Merge branch '0.9.x' into testing
Brion Vibber [Thu, 20 Jan 2011 23:08:31 +0000 (15:08 -0800)]
Merge branch 'master' into 0.9.x
Brion Vibber [Thu, 20 Jan 2011 21:05:58 +0000 (13:05 -0800)]
Ticket #2999: RequireValidatedEmail plugin now also prevents group creation by unvalidated users.
Siebrand Mazeland [Thu, 20 Jan 2011 20:12:51 +0000 (21:12 +0100)]
Localisation updates from http://translatewiki.net.
Siebrand Mazeland [Thu, 20 Jan 2011 19:00:45 +0000 (20:00 +0100)]
Translator comments added
L10n updates
Remove superfluous whitespace
Number parameters in message when two or more are used
ClientException and ServerException should end with a period
Zach Copley [Thu, 20 Jan 2011 06:55:00 +0000 (22:55 -0800)]
OAuth: Fix rare problem in which request tokens were sometimes being
returned as access tokens.
Zach Copley [Thu, 20 Jan 2011 00:13:42 +0000 (16:13 -0800)]
Fix syntax error in error msg
Zach Copley [Wed, 19 Jan 2011 23:52:18 +0000 (15:52 -0800)]
Fix a couple spelling mistakes in comments and remove redundant statement terminator
Zach Copley [Thu, 20 Jan 2011 18:43:27 +0000 (10:43 -0800)]
Move getConnectedApps() from Profile to User, where it belongs
Conflicts:
classes/User.php
Siebrand Mazeland [Thu, 20 Jan 2011 17:45:13 +0000 (18:45 +0100)]
Update translator documentation
Remove unneeded i18n for debug message
Remove superfluous whitespace
Siebrand Mazeland [Thu, 20 Jan 2011 17:39:46 +0000 (18:39 +0100)]
Update translator documentation
Update L10n
Remove superfluous whitespace
Zach Copley [Thu, 20 Jan 2011 06:55:00 +0000 (22:55 -0800)]
OAuth: Fix rare problem in which request tokens were sometimes being
returned as access tokens.
Zach Copley [Thu, 20 Jan 2011 00:13:42 +0000 (16:13 -0800)]
Fix syntax error in error msg
Zach Copley [Wed, 19 Jan 2011 23:52:18 +0000 (15:52 -0800)]
Fix a couple spelling mistakes in comments and remove redundant statement terminator
Zach Copley [Thu, 20 Jan 2011 06:58:07 +0000 (22:58 -0800)]
Merge branch 'testing' of gitorious.org:statusnet/mainline into testing
* 'testing' of gitorious.org:statusnet/mainline: (63 commits)
Add a scary 'experimental feture' warning & are-you-sure prompt on moveuser.php
fix wrong datatypes (saving string instead of array) in AtomPub notice processing
Account moving is a background activity
return a 409 Conflict when subscription already exists
OStatusPlugin does discovery in Profile::fromURI()
considerably more logging and error checking in AccountMover
add a log method to AccountMover
normalize accounts and check for return in HTTP for moving
move account-moving classes to their own libraries
execution protection on discovery.php
PHPCS discovery.php
Move discovery library from OStatus plugin to core
Revert "Revert "0.9.7alpha1""
first example of moving a user
Parse properties of links in XRD files
Add the Atom username to the XRD output
preserve activities in object
let callers pass in an XMLOutputter to output to
execution protection on discovery.php
PHPCS linkheader.php
...
Zach Copley [Thu, 20 Jan 2011 06:55:00 +0000 (22:55 -0800)]
OAuth: Fix rare problem in which request tokens were sometimes being
returned as access tokens.
Zach Copley [Thu, 20 Jan 2011 00:13:42 +0000 (16:13 -0800)]
Fix syntax error in error msg
Zach Copley [Wed, 19 Jan 2011 23:52:18 +0000 (15:52 -0800)]
Fix a couple spelling mistakes in comments and remove redundant statement terminator
Brion Vibber [Wed, 19 Jan 2011 23:13:46 +0000 (15:13 -0800)]
Add a scary 'experimental feture' warning & are-you-sure prompt on moveuser.php
As with deleteuser.php, can pass -y to bypass the prompt if you're sure.
Brion Vibber [Wed, 19 Jan 2011 22:26:31 +0000 (14:26 -0800)]
fix wrong datatypes (saving string instead of array) in AtomPub notice processing
Brion Vibber [Wed, 19 Jan 2011 18:41:14 +0000 (10:41 -0800)]
Merge branch 'testing' into moveaccount
Brion Vibber [Tue, 18 Jan 2011 23:18:55 +0000 (15:18 -0800)]
add rss.me to notice_source defaults
Brion Vibber [Tue, 18 Jan 2011 21:32:12 +0000 (13:32 -0800)]
SubMirror: add mirrored feeds count & management link to stats section when showing your own profile sidebar.
Brion Vibber [Tue, 18 Jan 2011 21:21:59 +0000 (13:21 -0800)]
Refactor profile sidebar statistics output so plugins can hook it and add entries.
Brion Vibber [Tue, 18 Jan 2011 21:12:08 +0000 (13:12 -0800)]
Refactor profile sidebar stats generation: factor out giant chunk of repeated method calls
Brion Vibber [Tue, 18 Jan 2011 20:52:38 +0000 (12:52 -0800)]
Start cleaning up profile sidebar: link the header text on subscription/subscriber/groups sections to the tabs so users don't have to dance around looking for the link if they don't have enough to trigger a 'more' link.
Consolidated some of that link generation; sooooo much more dupe code to kill in this section!
Brion Vibber [Tue, 18 Jan 2011 20:34:27 +0000 (12:34 -0800)]
Cleanup stray PHP 4-style references in hook calls for navigation bars. We can't replace the live action from here, and don't need a reference to keep the object mutable. Dumping the references helps ensure we don't end up getting errors when things calling the hooks might forget to use the reference and the PHP error reporting settings expose this fact at us.
Brion Vibber [Tue, 18 Jan 2011 20:34:04 +0000 (12:34 -0800)]
SubMirror: switch the mirror setup tab from account settings to subscriptions/groups/invites area
Brion Vibber [Tue, 18 Jan 2011 19:08:32 +0000 (11:08 -0800)]
TwitterBridge fix: merge down remaining 64-bit Snowflake ID fixes for twitterstatusfetcher.php from 0.9.x
Original fixes in
c169dcb5221cf3dd452c291bf97374bb459cc5b9; didn't get merged in
39cad55711897323fac5f651c003c4d815a51ae0 because the code had been broken out to another file, but manual merge went smooth.
These affect twitterstatusfetcher.php on all 32-bit installs and some 64-bit installs (depending on whether the version of the JSON library reads the large numbers as long or double internally). 64-bit bug is harder to see as it tends to manifest as off-by-one due to losing a bit of precision off the end.
Evan Prodromou [Mon, 17 Jan 2011 23:04:10 +0000 (18:04 -0500)]
wrap account actions in a section
Evan Prodromou [Mon, 17 Jan 2011 22:34:03 +0000 (17:34 -0500)]
Merge branch 'testing' into moveaccount
Evan Prodromou [Mon, 17 Jan 2011 22:32:44 +0000 (17:32 -0500)]
Erroneous code ensuring Webfinger accounts
Ostatus_profile::ensureProfileURI() was accidentally falling through to
the default switch case, and was also calling common_log() incorrectly.
Evan Prodromou [Mon, 17 Jan 2011 22:31:56 +0000 (17:31 -0500)]
Account moving is a background activity
Evan Prodromou [Mon, 17 Jan 2011 21:04:46 +0000 (16:04 -0500)]
Merge remote branch 'origin/moveaccount' into moveaccount
Conflicts:
lib/accountmover.php
Evan Prodromou [Mon, 17 Jan 2011 20:57:04 +0000 (15:57 -0500)]
return a 409 Conflict when subscription already exists
Evan Prodromou [Mon, 17 Jan 2011 20:45:03 +0000 (15:45 -0500)]
OStatusPlugin does discovery in Profile::fromURI()
Evan Prodromou [Mon, 17 Jan 2011 20:25:58 +0000 (15:25 -0500)]
considerably more logging and error checking in AccountMover