<?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>6667</bug_id>
          
          <creation_ts>2011-06-01 18:00:00 -0700</creation_ts>
          <short_desc>Upgrade does not mark old version as no longer current</short_desc>
          <delta_ts>2011-06-21 07:04:28 -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>Installer</component>
          <version>2.3.6</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="Michael Felczak">michael.pkp</reporter>
          <assigned_to name="Michael Felczak">michael.pkp</assigned_to>
          <cc>colin.prince</cc>
    
    <cc>pkp-support</cc>
    
    <cc>utkuh</cc>
          <cf_reportedversion>2.3.5</cf_reportedversion>
          

      

      

      

          <long_desc isprivate="0">
            <commentid>23883</commentid>
            <who name="Michael Felczak">michael.pkp</who>
            <bug_when>2011-06-01 18:00:55 -0700</bug_when>
            <thetext>Only the most recently installed version should be marked as current.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>23884</commentid>
              <attachid>3531</attachid>
            <who name="Michael Felczak">michael.pkp</who>
            <bug_when>2011-06-01 18:04:37 -0700</bug_when>
            <thetext>Created attachment 3531
Patch against OJS 2.3.5</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>23885</commentid>
              <attachid>3532</attachid>
            <who name="Michael Felczak">michael.pkp</who>
            <bug_when>2011-06-01 19:00:40 -0700</bug_when>
            <thetext>Created attachment 3532
Patch against OJS 2.3.5 (apply against lib/pkp + see db fix below)</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>23886</commentid>
            <who name="Michael Felczak">michael.pkp</who>
            <bug_when>2011-06-01 19:04:06 -0700</bug_when>
            <thetext>Patch marks old version as no longer current for manual install.

Patch restores marking of old version as no longer current in VersionDAO.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>23887</commentid>
            <who name="Michael Felczak">michael.pkp</who>
            <bug_when>2011-06-01 19:07:19 -0700</bug_when>
            <thetext>Need to manually update versions table to fix this bug after an upgrade to OJS 2.3.5:

UPDATE versions SET current = 0 WHERE product = &apos;ojs2&apos;;

UPDATE versions SET current = 1 WHERE product = &apos;ojs2&apos; AND major = 2 AND minor = 3 AND revision = 5;</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>23888</commentid>
            <who name="Michael Felczak">michael.pkp</who>
            <bug_when>2011-06-01 19:12:23 -0700</bug_when>
            <thetext>Committed to ojs_stable-2_3 and master:

https://github.com/pkp/pkp-lib/commit/de0224443d7c683b7506120e4cdd228343ff20bb
https://github.com/pkp/pkp-lib/commit/3cd2a49075f7ef0346aca6ac6296446dfb8aa4e4</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>23992</commentid>
            <who name="Michael Felczak">michael.pkp</who>
            <bug_when>2011-06-13 09:34:21 -0700</bug_when>
            <thetext>Additional notes:

The fix here reverts patch 3490 in Bug 6579 (Can&apos;t move custom blocks).

Tested custom blocks on a clean OJS install and customBlockManager version is correctly set to 1 upon install. Custom blocks can also be moved up/down and left/right.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <commentid>23993</commentid>
            <who name="Michael Felczak">michael.pkp</who>
            <bug_when>2011-06-13 09:34:43 -0700</bug_when>
            <thetext>Closing.</thetext>
          </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>3531</attachid>
            <date>2011-06-01 18:04:00 -0700</date>
            <delta_ts>2011-06-01 19:00:40 -0700</delta_ts>
            <desc>Patch against OJS 2.3.5</desc>
            <filename>6667.diff</filename>
            <type>text/plain</type>
            <size>1430</size>
            <attacher>michael.pkp</attacher>
            
              <data encoding="base64">LS0tIGEvY2xhc3Nlcy9pbnN0YWxsL0luc3RhbGxlci5pbmMucGhwCisrKyBiL2NsYXNzZXMvaW5z
dGFsbC9JbnN0YWxsZXIuaW5jLnBocApAQCAtMjkwLDEwICsyOTAsMTYgQEAgY2xhc3MgSW5zdGFs
bGVyIHsKIAlmdW5jdGlvbiB1cGRhdGVWZXJzaW9uKCkgewogCQlpZiAoJHRoaXMtPm5ld1ZlcnNp
b24tPmNvbXBhcmUoJHRoaXMtPmN1cnJlbnRWZXJzaW9uKSA+IDApIHsKIAkJCWlmICgkdGhpcy0+
Z2V0UGFyYW0oJ21hbnVhbEluc3RhbGwnKSkgeworCQkJCS8vIE1hcmsgb2xkIHZlcnNpb24gYXMg
bm8gbG9uZ2VyIGN1cnJlbnQKKwkJCQkkdGhpcy0+ZXhlY3V0ZVNRTChzcHJpbnRmKCdVUERBVEUg
dmVyc2lvbnMgU0VUIGN1cnJlbnQgPSAwIFdIRVJFIHByb2R1Y3QgPSAlcycsICR0aGlzLT5kYmNv
bm4tPnFzdHIoJHRoaXMtPmN1cnJlbnRWZXJzaW9uLT5nZXRQcm9kdWN0KCkpKSk7CisKIAkJCQkv
LyBGSVhNRSBXb3VsZCBiZSBiZXR0ZXIgdG8gaGF2ZSBhIG1vZGUgd2hlcmUgJGRiY29ubi0+ZXhl
Y3V0ZSgpIHNhdmVzIHRoZSBxdWVyeQogCQkJCXJldHVybiAkdGhpcy0+ZXhlY3V0ZVNRTChzcHJp
bnRmKCdJTlNFUlQgSU5UTyB2ZXJzaW9ucyAobWFqb3IsIG1pbm9yLCByZXZpc2lvbiwgYnVpbGQs
IGRhdGVfaW5zdGFsbGVkLCBjdXJyZW50LCBwcm9kdWN0X3R5cGUsIHByb2R1Y3QsIHByb2R1Y3Rf
Y2xhc3NfbmFtZSwgbGF6eV9sb2FkLCBzaXRld2lkZSkgVkFMVUVTICglZCwgJWQsICVkLCAlZCwg
Tk9XKCksIDEsICVzLCAlcywgJXMsICVkLCAlZCknLCAkdGhpcy0+bmV3VmVyc2lvbi0+Z2V0TWFq
b3IoKSwgJHRoaXMtPm5ld1ZlcnNpb24tPmdldE1pbm9yKCksICR0aGlzLT5uZXdWZXJzaW9uLT5n
ZXRSZXZpc2lvbigpLCAkdGhpcy0+bmV3VmVyc2lvbi0+Z2V0QnVpbGQoKSwgJHRoaXMtPmRiY29u
bi0+cXN0cigkdGhpcy0+bmV3VmVyc2lvbi0+Z2V0UHJvZHVjdFR5cGUoKSksICR0aGlzLT5kYmNv
bm4tPnFzdHIoJHRoaXMtPm5ld1ZlcnNpb24tPmdldFByb2R1Y3QoKSksICR0aGlzLT5kYmNvbm4t
PnFzdHIoJHRoaXMtPm5ld1ZlcnNpb24tPmdldFByb2R1Y3RDbGFzc05hbWUoKSksICgkdGhpcy0+
bmV3VmVyc2lvbi0+Z2V0TGF6eUxvYWQoKT8xOjApLCAoJHRoaXMtPm5ld1ZlcnNpb24tPmdldFNp
dGV3aWRlKCk/MTowKSkpOwogCQkJfSBlbHNlIHsKIAkJCQkkdmVyc2lvbkRhbyA9JiBEQU9SZWdp
c3RyeTo6Z2V0REFPKCdWZXJzaW9uREFPJyk7CisJCQkJLy8gTWFyayBvbGQgdmVyc2lvbiBhcyBu
byBsb25nZXIgY3VycmVudAorCQkJCSR2ZXJzaW9uRGFvLT5kaXNhYmxlVmVyc2lvbigkdGhpcy0+
Y3VycmVudFZlcnNpb24tPmdldFByb2R1Y3QoKSk7CisKIAkJCQlpZiAoISR2ZXJzaW9uRGFvLT5p
bnNlcnRWZXJzaW9uKCR0aGlzLT5uZXdWZXJzaW9uKSkgewogCQkJCQlyZXR1cm4gZmFsc2U7CiAJ
CQkJfQo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>3532</attachid>
            <date>2011-06-01 19:00:00 -0700</date>
            <delta_ts>2011-06-01 19:07:55 -0700</delta_ts>
            <desc>Patch against OJS 2.3.5 (apply against lib/pkp + see db fix below)</desc>
            <filename>6667.diff</filename>
            <type>text/plain</type>
            <size>2136</size>
            <attacher>michael.pkp</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL2NsYXNzZXMvaW5zdGFsbC9JbnN0YWxsZXIuaW5jLnBocCBiL2NsYXNzZXMv
aW5zdGFsbC9JbnN0YWxsZXIuaW5jLnBocAppbmRleCBlN2Y3MDgyLi4yNDg5ZGRjIDEwMDY0NAot
LS0gYS9jbGFzc2VzL2luc3RhbGwvSW5zdGFsbGVyLmluYy5waHAKKysrIGIvY2xhc3Nlcy9pbnN0
YWxsL0luc3RhbGxlci5pbmMucGhwCkBAIC0yOTAsNiArMjkwLDkgQEAgY2xhc3MgSW5zdGFsbGVy
IHsKIAlmdW5jdGlvbiB1cGRhdGVWZXJzaW9uKCkgewogCQlpZiAoJHRoaXMtPm5ld1ZlcnNpb24t
PmNvbXBhcmUoJHRoaXMtPmN1cnJlbnRWZXJzaW9uKSA+IDApIHsKIAkJCWlmICgkdGhpcy0+Z2V0
UGFyYW0oJ21hbnVhbEluc3RhbGwnKSkgeworCQkJCS8vIE1hcmsgb2xkIHZlcnNpb24gYXMgbm8g
bG9uZ2VyIGN1cnJlbnQKKwkJCQkkdGhpcy0+ZXhlY3V0ZVNRTChzcHJpbnRmKCdVUERBVEUgdmVy
c2lvbnMgU0VUIGN1cnJlbnQgPSAwIFdIRVJFIHByb2R1Y3QgPSAlcycsICR0aGlzLT5kYmNvbm4t
PnFzdHIoJHRoaXMtPmN1cnJlbnRWZXJzaW9uLT5nZXRQcm9kdWN0KCkpKSk7CisKIAkJCQkvLyBG
SVhNRSBXb3VsZCBiZSBiZXR0ZXIgdG8gaGF2ZSBhIG1vZGUgd2hlcmUgJGRiY29ubi0+ZXhlY3V0
ZSgpIHNhdmVzIHRoZSBxdWVyeQogCQkJCXJldHVybiAkdGhpcy0+ZXhlY3V0ZVNRTChzcHJpbnRm
KCdJTlNFUlQgSU5UTyB2ZXJzaW9ucyAobWFqb3IsIG1pbm9yLCByZXZpc2lvbiwgYnVpbGQsIGRh
dGVfaW5zdGFsbGVkLCBjdXJyZW50LCBwcm9kdWN0X3R5cGUsIHByb2R1Y3QsIHByb2R1Y3RfY2xh
c3NfbmFtZSwgbGF6eV9sb2FkLCBzaXRld2lkZSkgVkFMVUVTICglZCwgJWQsICVkLCAlZCwgTk9X
KCksIDEsICVzLCAlcywgJXMsICVkLCAlZCknLCAkdGhpcy0+bmV3VmVyc2lvbi0+Z2V0TWFqb3Io
KSwgJHRoaXMtPm5ld1ZlcnNpb24tPmdldE1pbm9yKCksICR0aGlzLT5uZXdWZXJzaW9uLT5nZXRS
ZXZpc2lvbigpLCAkdGhpcy0+bmV3VmVyc2lvbi0+Z2V0QnVpbGQoKSwgJHRoaXMtPmRiY29ubi0+
cXN0cigkdGhpcy0+bmV3VmVyc2lvbi0+Z2V0UHJvZHVjdFR5cGUoKSksICR0aGlzLT5kYmNvbm4t
PnFzdHIoJHRoaXMtPm5ld1ZlcnNpb24tPmdldFByb2R1Y3QoKSksICR0aGlzLT5kYmNvbm4tPnFz
dHIoJHRoaXMtPm5ld1ZlcnNpb24tPmdldFByb2R1Y3RDbGFzc05hbWUoKSksICgkdGhpcy0+bmV3
VmVyc2lvbi0+Z2V0TGF6eUxvYWQoKT8xOjApLCAoJHRoaXMtPm5ld1ZlcnNpb24tPmdldFNpdGV3
aWRlKCk/MTowKSkpOwogCQkJfSBlbHNlIHsKZGlmZiAtLWdpdCBhL2NsYXNzZXMvc2l0ZS9WZXJz
aW9uREFPLmluYy5waHAgYi9jbGFzc2VzL3NpdGUvVmVyc2lvbkRBTy5pbmMucGhwCmluZGV4IGQ0
MzhkNTIuLjFlOGMxZDAgMTAwNjQ0Ci0tLSBhL2NsYXNzZXMvc2l0ZS9WZXJzaW9uREFPLmluYy5w
aHAKKysrIGIvY2xhc3Nlcy9zaXRlL1ZlcnNpb25EQU8uaW5jLnBocApAQCAtMTM5LDcgKzEzOSw4
IEBAIGNsYXNzIFZlcnNpb25EQU8gZXh0ZW5kcyBEQU8gewogCQkJCQkkaXNOZXdWZXJzaW9uID0g
ZmFsc2U7CiAJCQkJfSBlbHNlaWYgKCR2ZXJzaW9uLT5jb21wYXJlKCRvbGRWZXJzaW9uKSA9PSAx
KSB7CiAJCQkJCS8vIFZlcnNpb24gdG8gaW5zZXJ0IGlzIG5ld2VyIHRoYW4gdGhlIGV4aXN0aW5n
IHZlcnNpb24gZW50cnkuCi0JCQkJCS8vIFdlJ2xsIHVwZGF0ZSB0aGUgZXhpc3RpbmcgZW50cnkg
YmVsb3cuCisJCQkJCS8vIFdlIHJlc2V0IGV4aXN0aW5nIGVudHJ5LgorCQkJCQkkdGhpcy0+dXBk
YXRlKCdVUERBVEUgdmVyc2lvbnMgU0VUIGN1cnJlbnQgPSAwIFdIRVJFIGN1cnJlbnQgPSAxIEFO
RCBwcm9kdWN0ID0gPycsICR2ZXJzaW9uLT5nZXRQcm9kdWN0KCkpOwogCQkJCX0gZWxzZSB7CiAJ
CQkJCS8vIFdlIGRvIG5vdCBzdXBwb3J0IGRvd25ncmFkZXMuCiAJCQkJCWZhdGFsRXJyb3IoJ1lv
dSBhcmUgdHJ5aW5nIHRvIGRvd25ncmFkZSB0aGUgcHJvZHVjdCAiJy4kdmVyc2lvbi0+Z2V0UHJv
ZHVjdCgpLiciIGZyb20gdmVyc2lvbiBbJy4kb2xkVmVyc2lvbi0+Z2V0VmVyc2lvblN0cmluZygp
LiddIHRvIHZlcnNpb24gWycuJHZlcnNpb24tPmdldFZlcnNpb25TdHJpbmcoKS4nXS4gRG93bmdy
YWRlcyBhcmUgbm90IHN1cHBvcnRlZC4nKTsK
</data>

          </attachment>
      

    </bug>

</bugzilla>