<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "http://pkp.sfu.ca/bugzilla/bugzilla.dtd">

<bugzilla version="4.2.5+"
          urlbase="http://pkp.sfu.ca/bugzilla/"
          
          maintainer="pkp-hosted@sfu.ca"
>

    <bug>
          <bug_id>4113</bug_id>
          
          <creation_ts>2009-03-05 10:30:00 -0800</creation_ts>
          <short_desc>Password reset fails if journal contact is not set</short_desc>
          <delta_ts>2009-05-01 10:58:54 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>OJS</product>
          <component>Site</component>
          <version>2.2.3</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Mac OS X 10.3</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P5</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Matthew Crider">mattcrider</reporter>
          <assigned_to name="PKP Support">pkp-support</assigned_to>
          <cc>alec</cc>
          
          

      

      

      

          <long_desc isprivate="0">
            <commentid>13926</commentid>
              <attachid>1544</attachid>
            <who name="Matthew Crider">mattcrider</who>
            <bug_when>2009-03-05 10:30:53 -0800</bug_when>
            <thetext>Created attachment 1544
Patch against OJS pre-2.2.3

OJS-devel does not fail because PKPLoginHandler only uses the site contact address.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>13953</commentid>
            <who name="Alec Smecher">alec</who>
            <bug_when>2009-03-09 07:14:57 -0700</bug_when>
            <thetext>Looks good, Matt -- go ahead.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>13956</commentid>
            <who name="Matthew Crider">mattcrider</who>
            <bug_when>2009-03-09 12:59:51 -0700</bug_when>
            <thetext>Fixed.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>14438</commentid>
              <attachid>1719</attachid>
            <who name="Matthew Crider">mattcrider</who>
            <bug_when>2009-05-01 10:58:54 -0700</bug_when>
            <thetext>Created attachment 1719
Additional patch against OJS pre-2.2.4 CVS

We should also check if supportEmail is set in the resetPassword function.  Fixed in CVS.</thetext>
          </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>1544</attachid>
            <date>2009-03-05 10:30:00 -0800</date>
            <delta_ts>2009-03-05 10:30:53 -0800</delta_ts>
            <desc>Patch against OJS pre-2.2.3</desc>
            <filename>ojsResetPass.patch</filename>
            <type>text/plain</type>
            <size>774</size>
            <attacher>mattcrider</attacher>
            
              <data encoding="base64">SW5kZXg6IHBhZ2VzL2xvZ2luL0xvZ2luSGFuZGxlci5pbmMucGhwCj09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KUkNTIGZp
bGU6IC9jdnMvb2pzMi9wYWdlcy9sb2dpbi9BdHRpYy9Mb2dpbkhhbmRsZXIuaW5jLnBocCx2CnJl
dHJpZXZpbmcgcmV2aXNpb24gMS4yNi4yLjEKZGlmZiAtdSAtcjEuMjYuMi4xIExvZ2luSGFuZGxl
ci5pbmMucGhwCi0tLSBwYWdlcy9sb2dpbi9Mb2dpbkhhbmRsZXIuaW5jLnBocAkxOCBGZWIgMjAw
OSAxODo0MDoxMSAtMDAwMAkxLjI2LjIuMQorKysgcGFnZXMvbG9naW4vTG9naW5IYW5kbGVyLmlu
Yy5waHAJNSBNYXIgMjAwOSAxODoyNTozMCAtMDAwMApAQCAtMTc4LDcgKzE3OCw3IEBACiAJCQkk
bWFpbCA9JiBuZXcgTWFpbFRlbXBsYXRlKCdQQVNTV09SRF9SRVNFVF9DT05GSVJNJyk7CiAKIAkJ
CS8vIFNldCB0aGUgc2VuZGVyIGJhc2VkIG9uIHRoZSBjdXJyZW50IGNvbnRleHQKLQkJCWlmICgk
am91cm5hbCkgeworCQkJaWYgKCRqb3VybmFsICYmICRqb3VybmFsLT5nZXRTZXR0aW5nKCdzdXBw
b3J0RW1haWwnKSkgewogCQkJCSRlbWFpbC0+c2V0RnJvbSgkam91cm5hbC0+Z2V0U2V0dGluZygn
c3VwcG9ydEVtYWlsJyksICRqb3VybmFsLT5nZXRTZXR0aW5nKCdzdXBwb3J0TmFtZScpKTsKIAkJ
CX0gZWxzZSB7CiAJCQkJJG1haWwtPnNldEZyb20oJHNpdGUtPmdldFNpdGVDb250YWN0RW1haWwo
KSwgJHNpdGUtPmdldFNpdGVDb250YWN0TmFtZSgpKTsK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>1719</attachid>
            <date>2009-05-01 10:58:00 -0700</date>
            <delta_ts>2009-05-01 10:58:54 -0700</delta_ts>
            <desc>Additional patch against OJS pre-2.2.4 CVS</desc>
            <filename>passwordResetEmail.patch</filename>
            <type>text/plain</type>
            <size>764</size>
            <attacher>mattcrider</attacher>
            
              <data encoding="base64">SW5kZXg6IHBhZ2VzL2xvZ2luL0xvZ2luSGFuZGxlci5pbmMucGhwCj09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KUkNTIGZp
bGU6IC9jdnMvb2pzMi9wYWdlcy9sb2dpbi9BdHRpYy9Mb2dpbkhhbmRsZXIuaW5jLnBocCx2CnJl
dHJpZXZpbmcgcmV2aXNpb24gMS4yNi4yLjQKZGlmZiAtdSAtcjEuMjYuMi40IExvZ2luSGFuZGxl
ci5pbmMucGhwCi0tLSBwYWdlcy9sb2dpbi9Mb2dpbkhhbmRsZXIuaW5jLnBocAkxIE1heSAyMDA5
IDE3OjUwOjEwIC0wMDAwCTEuMjYuMi40CisrKyBwYWdlcy9sb2dpbi9Mb2dpbkhhbmRsZXIuaW5j
LnBocAkxIE1heSAyMDA5IDE3OjUzOjI4IC0wMDAwCkBAIC0yNTAsNyArMjUwLDcgQEAKIAkJCSRt
YWlsID0mIG5ldyBNYWlsVGVtcGxhdGUoJ1BBU1NXT1JEX1JFU0VUJyk7CiAKIAkJCS8vIFNldCB0
aGUgc2VuZGVyIGJhc2VkIG9uIHRoZSBjdXJyZW50IGNvbnRleHQKLQkJCWlmICgkam91cm5hbCkg
eworCQkJaWYgKCRqb3VybmFsICYmICRqb3VybmFsLT5nZXRTZXR0aW5nKCdzdXBwb3J0RW1haWwn
KSkgewogCQkJCSRtYWlsLT5zZXRGcm9tKCRqb3VybmFsLT5nZXRTZXR0aW5nKCdzdXBwb3J0RW1h
aWwnKSwgJGpvdXJuYWwtPmdldFNldHRpbmcoJ3N1cHBvcnROYW1lJykpOwogCQkJfSBlbHNlIHsK
IAkJCQkkbWFpbC0+c2V0RnJvbSgkc2l0ZS0+Z2V0U2l0ZUNvbnRhY3RFbWFpbCgpLCAkc2l0ZS0+
Z2V0U2l0ZUNvbnRhY3ROYW1lKCkpOwo=
</data>

          </attachment>
      

    </bug>

</bugzilla>