62832 - Make an option for "newemailtech" to be default

Status: RESOLVED FIXED

(Bugzilla :: Bugzilla-General, defect, P3)

Description

[Jacob Steenhagen]

I wrote a patch that will allow newemailtech to be set as the default mailing
scheme via editparams.cgi.  This patch does not change any existing users
preference, it only makes it so all new users have newemailtech turned on by
default (they still have the option to turn it off).  Also, the default value
for this is "off" so nothing happens unless you specifcally change the setting
in editparams.cgi.

Comment 1

[Jacob Steenhagen]

Attachment: [PATCH] Add an option for "newemailtechdefault"

Comment 2

[Jacob Steenhagen]

As mentioned above, this patch does not change any users default settings.  If
you wish to change all your users prefs to the newemailtech setting, you must
issue the following SQL command inside the "bugs" database:

update profiles set newemailtech = '1';

Comment 3

[Tara Hernandez]

See comment in bug #63272

Comment 4

[Dan Mosedale (:dmosedale, :dmose)]

*** Bug 63272 has been marked as a duplicate of this bug. ***

Comment 5

[Dan Mosedale (:dmosedale, :dmose)]

Attachment: patch v2

Comment 6

[Dan Mosedale (:dmosedale, :dmose)]

Ok, I've revised the patch in the following ways:

	* use Param() instead of $::param(), so that defparams.pl gets read as
	  necessary

	* ignore newemailtechdefault if newemailtech is turned off

	* default both newemailtech and newemailtechdefault on in defparams.pl

	* change verbiage in defparams.pl to reflect new defaults

	* give Jake credit in the boilerplate contributor list.

Comment 7

[Jacob Steenhagen]

All makes sence to me...

Thanks for the boiler plate credit, BTW... I always forget to do that.  We may
also want to change the verbage in userprefs.cgi?bank=diffs to reflect to the
users that this is no longer bleeding edge and is recommended, blah, blah...

Comment 8

[Dan Mosedale (:dmosedale, :dmose)]

Attachment: patch v3

Comment 9

[Dan Mosedale (:dmosedale, :dmose)]

Jake: excellent point.  I've uploaded a new patch with that change made.

I've also added several folks to the CC list.  If one of you guys could give me
an r=, I'll check this in.

Comment 10

[Dan Mosedale (:dmosedale, :dmose)]

Patch checked in; r=endico & cyeh.

Comment 11

[Tara Hernandez]

Sorry for the spam, but I needed to be able to query for all of these correctly.

Comment 12

[Dave Miller [:justdave]]

Moving closed bugs to Bugzilla product