Modify

Opened 13 years ago

Closed 13 years ago

#9032 closed defect (duplicate)

AttributeError: 'AccountChangeNotification' object has no attribute 'smtp_server'

Reported by: Born4Developing Owned by: Steffen Hoffmann
Priority: normal Component: AccountManagerPlugin
Severity: normal Keywords: change notification
Cc: Trac Release: 0.11

Description

How to Reproduce

While doing a POST operation on /prefs/account, Trac issued an internal error.

(please provide additional details here)

Request parameters:

{'__FORM_TOKEN': u'd08ffb504b3328d5959faa25',
 'action': u'save',
 'old_password': u'AdNyOfBb',
 'panel_id': u'account',
 'password': u'Xy-04#eA@3271',
 'password_confirm': u'Xy-04#eA@3271'}

User agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/534.30 (KHTML, like Gecko) Chrome/12.0.742.122 Safari/534.30

System Information

Trac 0.12.2
Babel 0.9.5
Genshi 0.6
mod_wsgi 2.5 (WSGIProcessGroup WSGIApplicationGroup %{GLOBAL})
MySQL server: "5.1.56-0.dotdeb.0", client: "5.0.51a", thread-safe: 1
MySQLdb 1.2.2
Python 2.5.2 (r252:60911, Jan 24 2010, 18:02:01)
[GCC 4.3.2]
setuptools 0.6c8
Subversion 1.5.1 (r32289)
jQuery 1.4.2

Enabled Plugins

TracAccountManager 0.2.1dev-r7163
TracWysiwyg 0.11.0.2-r8148

Python Traceback

Traceback (most recent call last):
  File "build/bdist.linux-x86_64/egg/trac/web/main.py", line 511, in _dispatch_request
    dispatcher.dispatch(req)
  File "build/bdist.linux-x86_64/egg/trac/web/main.py", line 237, in dispatch
    resp = chosen_handler.process_request(req)
  File "build/bdist.linux-x86_64/egg/trac/prefs/web_ui.py", line 77, in process_request
    template, data = chosen_provider.render_preference_panel(req, panel_id)
  File "build/bdist.linux-x86_64/egg/acct_mgr/web_ui.py", line 189, in render_preference_panel
    data = {'account': self._do_account(req)}
  File "build/bdist.linux-x86_64/egg/acct_mgr/web_ui.py", line 239, in _do_account
    data.update(self._do_change_password(req))
  File "build/bdist.linux-x86_64/egg/acct_mgr/web_ui.py", line 316, in _do_change_password
    mgr.set_password(user, password)
  File "build/bdist.linux-x86_64/egg/acct_mgr/api.py", line 131, in set_password
    self._notify('password_changed', user, password)
  File "build/bdist.linux-x86_64/egg/acct_mgr/api.py", line 238, in _notify
    getattr(l, func)(*args)
  File "build/bdist.linux-x86_64/egg/acct_mgr/notification.py", line 46, in user_password_changed
    notifier.notify(username, 'Password reset for user')
  File "build/bdist.linux-x86_64/egg/acct_mgr/notification.py", line 79, in notify
    NotifyEmail.notify(self, username, subject)
  File "build/bdist.linux-x86_64/egg/trac/notification.py", line 324, in notify
    Notify.notify(self, resid)
  File "build/bdist.linux-x86_64/egg/trac/notification.py", line 226, in notify
    self.send(torcpts, ccrcpts)
  File "build/bdist.linux-x86_64/egg/acct_mgr/notification.py", line 149, in send
    % (self.smtp_server, self.smtp_port, recipients))
AttributeError: 'AccountChangeNotification' object has no attribute 'smtp_server'

Attachments (0)

Change History (2)

comment:1 Changed 13 years ago by Ryan J Ollos

Component: SELECT A HACKAccountManagerPlugin
Owner: changed from anonymous to Steffen Hoffmann

comment:2 Changed 13 years ago by Steffen Hoffmann

Keywords: change notification added
Resolution: duplicate
Status: newclosed

First, thanks for taking care to report issues. That's much appreciated.

Second, please follow general advice not to report issues twice. We don't hide issue, and there is a good search here, admittedly not as powerful as in Trac 0.11 or current revision, but searching for smtp_server alone would have done the job quite nicely for you. See and follow up on #4895 is the appropriate way, please.

Third, free developers time by providing correct detail. More details are generally good as well, but plain wrong details are especially bad, no matter, how it happened to go wrong. Wouldn't it be for user rjollos, I'd probably have never noticed your report.

Last for now: Please don't continue with such outdated software. It kind of hurts me to see 0.2.1dev-r7163 in production, while I've done more than 140 changes meanwhile. Using the announced stable release is really better than any old dev version that old. Still use 0.4 dev, if you feel like using the latest, or want to help with debugging and testing latest fixes.

Hope you don't feel embarrass, no intention from my side. I already had to take a duplicate down lately, so I'm well aware of the problem and hope to get all good/new additional information into the existing bucket.

Thanks again for taking care.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Steffen Hoffmann.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.