Issue 12239: msilib VT_EMPTY SummaryInformation properties raise an error (suggest returning None) (original) (raw)

Issue12239

Created on 2011-06-02 14:13 by markm, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
support_vt_empty_in_summary_getproperty.patch markm,2011-06-12 01:20 Patch to return None for VT_EMPTY properties of SummaryInformation review
Pull Requests
URL Status Linked Edit
PR 4539 merged berker.peksag,2017-11-24 11:13
PR 4543 merged berker.peksag,2017-11-24 15:19
Messages (5)
msg137478 - (view) Author: Mark Mc Mahon (markm) * Date: 2011-06-02 14:13
Using "some.msi" where the first property is VT_EMPTY Using COM: >>> from win32com.client import gencache >>> com_lib = gencache.EnsureModule('{000C1092-0000-0000-C000-000000000046}', 409, 1, 0) >>> com_msi = com_lib.Installer() >>> db = com_msi.OpenDatabase('some.msi', 0) >>> si = db.GetSummaryInformation(0) >>> repr(si.Property(0)) 'None' Using msilib: >>> import msilib >>> db = msilib.OpenDatabase(r'some.msi', 0) >>> si = db.GetSummaryInformation(0) >>> si.GetProperty(0) Traceback (most recent call last): File "", line 1, in NotImplementedError: result of type 0 I aim to submit a patch that so that SummaryInformation.GetProperty() with a type of VT_EMPTY will return None.
msg235997 - (view) Author: Mark Lawrence (BreamoreBoy) * Date: 2015-02-14 22:06
Can we have a patch review please.
msg306899 - (view) Author: Berker Peksag (berker.peksag) * (Python committer) Date: 2017-11-24 15:11
New changeset 19fb134185ce155bc53f517116fca73093ba55e9 by Berker Peksag in branch 'master': bpo-12239: Make GetProperty() return None for VT_EMPTY (GH-4539) https://github.com/python/cpython/commit/19fb134185ce155bc53f517116fca73093ba55e9
msg306901 - (view) Author: Berker Peksag (berker.peksag) * (Python committer) Date: 2017-11-24 16:04
New changeset 412f00b839eae2bc07ca08a8e615c3d7dc870646 by Berker Peksag in branch '3.6': [3.6] bpo-12239: Make GetProperty() return None for VT_EMPTY (GH-4539) https://github.com/python/cpython/commit/412f00b839eae2bc07ca08a8e615c3d7dc870646
msg306903 - (view) Author: Berker Peksag (berker.peksag) * (Python committer) Date: 2017-11-24 16:05
Thanks for the patch, Mark. I updated the test code and commit it to 3.6 and master branches.
History
Date User Action Args
2022-04-11 14:57:18 admin set github: 56448
2017-11-24 16:05:42 berker.peksag set status: open -> closedversions: + Python 3.6, Python 3.7, - Python 3.4, Python 3.5messages: + resolution: fixedstage: patch review -> resolved
2017-11-24 16:04:42 berker.peksag set messages: +
2017-11-24 15:19:47 berker.peksag set pull_requests: + <pull%5Frequest4476>
2017-11-24 15:11:20 berker.peksag set messages: +
2017-11-24 11:13:44 berker.peksag set stage: needs patch -> patch reviewpull_requests: + <pull%5Frequest4473>
2017-11-19 07:05:32 berker.peksag set nosy: + berker.peksag
2015-02-14 23:13:55 brian.curtin set nosy: - brian.curtin
2015-02-14 22:06:11 BreamoreBoy set nosy: + BreamoreBoymessages: +
2014-06-24 03:08:44 BreamoreBoy set nosy: + loewis, steve.dowerversions: + Python 3.4, Python 3.5, - Python 3.1, Python 2.7, Python 3.2
2011-06-12 01:20:59 markm set files: + support_vt_empty_in_summary_getproperty.patchkeywords: + patch
2011-06-02 14:14:49 brian.curtin set stage: needs patchtype: behaviorversions: + Python 3.1, Python 2.7
2011-06-02 14:14:08 brian.curtin set nosy: + brian.curtin
2011-06-02 14:13:43 markm create