CompareVersion not implemented

VERIFIED FIXED in M9

Status

Core Graveyard
Installer: XPInstall Engine
P2
major
VERIFIED FIXED
19 years ago
2 years ago

People

(Reporter: Jimmy Lee, Assigned: dveditz)

Tracking

Trunk
All
Windows NT

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
Build: 7/9/99 SeaMonkey build

1. Use trigger, http://jimbob/trigger2.html
2. Trigger jar at http://jimbob/jars/a_compareversion_2_string.jar

regName = "a_compareversion_2_string";
jarSrc = "smrtupdt.txt";
var vi = "5.5.5.5";

Install.StartInstall("Acceptance: a_compareversion_2_string", "acceptance", vi,
0);
f = Install.GetFolder("Communicator", "a_compareversion_2_string");
Install.AddSubcomponent(regName, vi, jarSrc, f, jarSrc);

if (0 == Install.GetLastError())
	Install.FinalizeInstall();
else
	Install.AbortInstall();

if (!InstallTrigger.UpdateEnabled())
	Install.LogComment("UpdateEnabled() is false.  SmartUpdate is not
enabled for this client.");
else
   {
	compValue = InstallTrigger.CompareVersion("acceptance" + "/" + regName,
"6.6.6.6");
	Install.LogComment("CompareVersion should return -4 = " + compValue);
   }

RESULT:
The Install.log shows:
---------------------------------------------------------------------------
Acceptance: a_compareversion_2_string
---------------------------------------------------------------------------

Starting Installation at 07/13/1999 10:05:46

Item [1/1] Installing:
C:\Program Files\moz79\x86rel\a_compareversion_2_string\smrtupdt.txt
Comment: CompareVersion should return -4 = 7393600

Finished Installation  07/13/1999 10:05:46

EXPECTED RESULT:
InstallTrigger.CompareVersion returns -4 because the major version of the
registered component is smaller than the one compared to, "6.6.6.6".
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M9
(Assignee)

Updated

19 years ago
Priority: P3 → P2
(Assignee)

Comment 1

19 years ago
adjusting priority
(Assignee)

Updated

19 years ago
Summary: CompareVersion: Incorrect major number received → CompareVersion not implemented
(Assignee)

Comment 2

19 years ago
Turns out this isn't even implemented below the top level.

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED

Comment 3

19 years ago
I have made numberous bug fixes which might cause some of these bugs to go away.
 I am going to mark them as FIXED so that they may be verified.
(Reporter)

Comment 4

19 years ago
Bug 10180 is similar to this problem.  I will mark this Verified when 10180 is
verified fix.
(Reporter)

Updated

19 years ago
Status: RESOLVED → VERIFIED
(Reporter)

Comment 5

19 years ago
Bug 10180 is fixed.

Comment 6

19 years ago
Bulk move of XPInstall (component to be deleted) bugs to Installer: XPInstall
Engine
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.