Thanks for the module guys, I've just installed it and found on the admin page that the "Delete old statuses" select does not retain a previously submitted value.

I'll attach a very small patch to fix this.

I'm not sure it matters that much because it doesn't appear that this variable is actually used anywhere else?

CommentFileSizeAuthor
#1 602016_twitter_delete_old_statuses.patch675 bytesmatason

Comments

matason’s picture

matason commented
Status: Active » Needs review
StatusFileSize
new 602016_twitter_delete_old_statuses.patch675 bytes

Here's the patch, seems the same issue exists in 6.x-3.x-dev

And yes, this variable is used when deleting old statuses.

aaronbauman’s picture

aaronbauman
he/him
commented
Version: 6.x-2.6 » 6.x-3.x-dev
Priority: Minor » Normal
Status: Needs review » Reviewed & tested by the community

yup, it's a one-liner and it's breaking a piece of functionality.
let's get it in.

sirkitree’s picture

sirkitree commented 
+++ twitter.pages.inc	12 Oct 2009 06:50:44 -0000
@@ -43,7 +43,7 @@ function twitter_admin_form() {
-    '#default_value' => variable_get('amazon_refresh_schedule', 0),

lol

++

walkah’s picture

walkah commented
Status: Reviewed & tested by the community » Fixed

this is fixed, thanks :-)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.