<?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>6629</bug_id>
          
          <creation_ts>2011-05-02 12:18:00 -0700</creation_ts>
          <short_desc>Registration &quot;Registered&quot; date incorrectly updated</short_desc>
          <delta_ts>2011-05-03 12:31:09 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>OCS</product>
          <component>Registrations</component>
          <version>2.3.4</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P3</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="James MacGregor">jmacgreg</reporter>
          <assigned_to name="James MacGregor">jmacgreg</assigned_to>
          <cc>pkp-support</cc>
          <cf_reportedversion>2.3.3</cf_reportedversion>
          <cf_alsoaffects>OCS 2.3.3</cf_alsoaffects>

      

      

      

          <long_desc isprivate="0">
            <commentid>23693</commentid>
              <attachid>3500</attachid>
            <who name="James MacGregor">jmacgreg</who>
            <bug_when>2011-05-02 12:18:20 -0700</bug_when>
            <thetext>Created attachment 3500
patch against master

If a user registers eg on 2011-01-01, and the Editor edits (or even views and saves without editing) their registration, the &quot;Registered&quot; date changes to the current date. 

I&apos;ve modified wuyh&apos;s fix at http://pkp.sfu.ca/support/forum/viewtopic.php?f=3&amp;t=2602&amp;p=28892#p28892; see attached. I think it works fine, but I&apos;d like someone to take a look. 

If it looks good, I&apos;ll commit to master and stable.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>23707</commentid>
            <who name="Michael Felczak">michael.pkp</who>
            <bug_when>2011-05-03 10:26:39 -0700</bug_when>
            <thetext>Looks good, James.

Can you please provide an updated patch against OCS 2.3.3-1 (stable) with the lib/pkp subcommit info removed + add to recommended patches list for OCS. Thanks!</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>23711</commentid>
              <attachid>3503</attachid>
            <who name="James MacGregor">jmacgreg</who>
            <bug_when>2011-05-03 12:30:31 -0700</bug_when>
            <thetext>Created attachment 3503
Patch against OCS 2.3.3-1 (stable)</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>23712</commentid>
            <who name="James MacGregor">jmacgreg</who>
            <bug_when>2011-05-03 12:31:09 -0700</bug_when>
            <thetext>Thanks, Michael! Committed to stable and master, and added a patch here against 2.3.3-1.</thetext>
          </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>3500</attachid>
            <date>2011-05-02 12:18:00 -0700</date>
            <delta_ts>2011-05-02 12:18:20 -0700</delta_ts>
            <desc>patch against master</desc>
            <filename>registration.diff</filename>
            <type>text/plain</type>
            <size>1256</size>
            <attacher>jmacgreg</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL2NsYXNzZXMvcmVnaXN0cmF0aW9uL2Zvcm0vUmVnaXN0cmF0aW9uRm9ybS5p
bmMucGhwIGIvY2xhc3Nlcy9yZWdpc3RyYXRpb24vZm9ybS9SZWdpc3RyYXRpb25Gb3JtLmluYy5w
aHAKaW5kZXggZDRkYzc5My4uNDFmOGQ0NCAxMDA3NTUKLS0tIGEvY2xhc3Nlcy9yZWdpc3RyYXRp
b24vZm9ybS9SZWdpc3RyYXRpb25Gb3JtLmluYy5waHAKKysrIGIvY2xhc3Nlcy9yZWdpc3RyYXRp
b24vZm9ybS9SZWdpc3RyYXRpb25Gb3JtLmluYy5waHAKQEAgLTE4MSw2ICsxODEsNyBAQCBjbGFz
cyBSZWdpc3RyYXRpb25Gb3JtIGV4dGVuZHMgRm9ybSB7CiAKIAkJaWYgKCFpc3NldCgkcmVnaXN0
cmF0aW9uKSkgewogCQkJJHJlZ2lzdHJhdGlvbiA9IG5ldyBSZWdpc3RyYXRpb24oKTsKKwkJCSRy
ZWdpc3RyYXRpb24tPnNldERhdGVSZWdpc3RlcmVkKHRpbWUoKSk7CiAJCX0KIAogCQkkcmVnaXN0
cmF0aW9uLT5zZXRTY2hlZENvbmZJZCgkc2NoZWRDb25mLT5nZXRJZCgpKTsKQEAgLTE5MCw3ICsx
OTEsNiBAQCBjbGFzcyBSZWdpc3RyYXRpb25Gb3JtIGV4dGVuZHMgRm9ybSB7CiAJCSRyZWdpc3Ry
YXRpb24tPnNldERvbWFpbigkdGhpcy0+Z2V0RGF0YSgnZG9tYWluJykgPyAkdGhpcy0+Z2V0RGF0
YSgnZG9tYWluJykgOiBudWxsKTsKIAkJJHJlZ2lzdHJhdGlvbi0+c2V0SVBSYW5nZSgkdGhpcy0+
Z2V0RGF0YSgnaXBSYW5nZScpID8gJHRoaXMtPmdldERhdGEoJ2lwUmFuZ2UnKSA6IG51bGwpOwog
CQkkcmVnaXN0cmF0aW9uLT5zZXRTcGVjaWFsUmVxdWVzdHMoJHRoaXMtPmdldERhdGEoJ3NwZWNp
YWxSZXF1ZXN0cycpID8gJHRoaXMtPmdldERhdGEoJ3NwZWNpYWxSZXF1ZXN0cycpIDogbnVsbCk7
Ci0JCSRyZWdpc3RyYXRpb24tPnNldERhdGVSZWdpc3RlcmVkKHRpbWUoKSk7CiAKIAkJLy8gU2Vu
ZCBhbiBlbWFpbCB0byB0aGUgcmVnaXN0cmFudCBpbmZvcm1pbmcgdGhlbSB0aGF0IHRoZWlyIHBh
eW1lbnQgd2FzIHJlY2VpdmVkCiAJCWlmICgkdGhpcy0+Z2V0RGF0YSgnbm90aWZ5UGF5bWVudEVt
YWlsJykpIHsKZGlmZiAtLWdpdCBhL2xpYi9wa3AgYi9saWIvcGtwCmluZGV4IDkyNjQ5ODEuLmRh
OGQ1NjcgMTYwMDAwCi0tLSBhL2xpYi9wa3AKKysrIGIvbGliL3BrcApAQCAtMSArMSBAQAotU3Vi
cHJvamVjdCBjb21taXQgOTI2NDk4MTU1Y2RiZDhkYjkyMWU0Y2IxNDg3N2EyZWExNTA3MmM2OAor
U3VicHJvamVjdCBjb21taXQgZGE4ZDU2NzYzZDAyNzM1ZjJjZDExYTU4NzRkODYzYmQzOWNjZjNm
Zgo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>3503</attachid>
            <date>2011-05-03 12:30:00 -0700</date>
            <delta_ts>2011-05-03 12:30:31 -0700</delta_ts>
            <desc>Patch against OCS 2.3.3-1 (stable)</desc>
            <filename>6629.diff</filename>
            <type>text/plain</type>
            <size>1035</size>
            <attacher>jmacgreg</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL2NsYXNzZXMvcmVnaXN0cmF0aW9uL2Zvcm0vUmVnaXN0cmF0aW9uRm9ybS5p
bmMucGhwIGIvY2xhc3Nlcy9yZWdpc3RyYXRpb24vZm9ybS9SZWdpc3RyYXRpb25Gb3JtLmluYy5w
aHAKaW5kZXggZDRkYzc5My4uNDFmOGQ0NCAxMDA3NTUKLS0tIGEvY2xhc3Nlcy9yZWdpc3RyYXRp
b24vZm9ybS9SZWdpc3RyYXRpb25Gb3JtLmluYy5waHAKKysrIGIvY2xhc3Nlcy9yZWdpc3RyYXRp
b24vZm9ybS9SZWdpc3RyYXRpb25Gb3JtLmluYy5waHAKQEAgLTE4MSw2ICsxODEsNyBAQCBjbGFz
cyBSZWdpc3RyYXRpb25Gb3JtIGV4dGVuZHMgRm9ybSB7CiAKIAkJaWYgKCFpc3NldCgkcmVnaXN0
cmF0aW9uKSkgewogCQkJJHJlZ2lzdHJhdGlvbiA9IG5ldyBSZWdpc3RyYXRpb24oKTsKKwkJCSRy
ZWdpc3RyYXRpb24tPnNldERhdGVSZWdpc3RlcmVkKHRpbWUoKSk7CiAJCX0KIAogCQkkcmVnaXN0
cmF0aW9uLT5zZXRTY2hlZENvbmZJZCgkc2NoZWRDb25mLT5nZXRJZCgpKTsKQEAgLTE5MCw3ICsx
OTEsNiBAQCBjbGFzcyBSZWdpc3RyYXRpb25Gb3JtIGV4dGVuZHMgRm9ybSB7CiAJCSRyZWdpc3Ry
YXRpb24tPnNldERvbWFpbigkdGhpcy0+Z2V0RGF0YSgnZG9tYWluJykgPyAkdGhpcy0+Z2V0RGF0
YSgnZG9tYWluJykgOiBudWxsKTsKIAkJJHJlZ2lzdHJhdGlvbi0+c2V0SVBSYW5nZSgkdGhpcy0+
Z2V0RGF0YSgnaXBSYW5nZScpID8gJHRoaXMtPmdldERhdGEoJ2lwUmFuZ2UnKSA6IG51bGwpOwog
CQkkcmVnaXN0cmF0aW9uLT5zZXRTcGVjaWFsUmVxdWVzdHMoJHRoaXMtPmdldERhdGEoJ3NwZWNp
YWxSZXF1ZXN0cycpID8gJHRoaXMtPmdldERhdGEoJ3NwZWNpYWxSZXF1ZXN0cycpIDogbnVsbCk7
Ci0JCSRyZWdpc3RyYXRpb24tPnNldERhdGVSZWdpc3RlcmVkKHRpbWUoKSk7CiAKIAkJLy8gU2Vu
ZCBhbiBlbWFpbCB0byB0aGUgcmVnaXN0cmFudCBpbmZvcm1pbmcgdGhlbSB0aGF0IHRoZWlyIHBh
eW1lbnQgd2FzIHJlY2VpdmVkCiAJCWlmICgkdGhpcy0+Z2V0RGF0YSgnbm90aWZ5UGF5bWVudEVt
YWlsJykpIHsK
</data>

          </attachment>
      

    </bug>

</bugzilla>