Issue 13: PATCH - more verbose preferences, adds a 'helper' warning for those that don't load their preferences, ensures that AM preferences are toggled right after each save.
Status:  Fixed
Owner: ----
Closed:  Aug 2008
Reported by rogerpack2005, Feb 15, 2008
This patch is mostly for usability, with some small code changes (mostly of setting the AM billing 
style after each preferences save, just in case).  It makes the preferences page more verbose 
(hopefully in a good way), and adds a method that would have helped me when I came across 
"Nil.is_true? is undefined" errors.
Thanks!
-Roger
This diff was taken from the substruct directory from what I believe is the latest TRUNK but it's hard 
to tell since it was a recent SVN export.
diff1.diff
5.1 KB   View   Download
Feb 16, 2008
#1 rogerpack2005
if the patch fails on one chunk in preferences.rb that's ok--it was just the "Nil.is_true" part which can be 
ignored, anyway.
Mar 13, 2008
#2 rogerpack2005
This patch is probably not necessary if and after the authorize.net SIM patch were applied.  In fact, any Active 
Merchant initialization could be taken out entirely then, as it is re-read per transaction in that patch.
Apr 8, 2008
Project Member #3 subim...@gmail.com
Roger - which patch were you talking about with authorize.net SIM? Have an issue number?
Apr 8, 2008
Project Member #4 subim...@gmail.com
(No comment was entered for this change.)
Labels: -Priority-Medium Priority-Low
Apr 9, 2008
Project Member #5 subim...@gmail.com
Roger, I've changed a ton of stuff and this is failing.
I'd like to apply this if possible. Can you give me a patch that applies cleanly to trunk?
Summary: PATCH - more verbose preferences, adds a 'helper' warning for those that don't load their preferences, ensures that AM preferences are toggled right after each save.
Status: Accepted
Apr 19, 2008
#6 rogerpack2005
the SIM patch is in  issue 34 .  I will send in an updated version of this sometime,
too.  Sorry for the late response--google doesn't email me on changes, for some reason.
Aug 11, 2008
Project Member #7 subim...@gmail.com
This change seems pretty outdated by now. Rolled in the text changes and skipped the
rest with r109
Status: Fixed