James Hawkins : msi: The BS_GROUPBOX style should only be used if the HasBorder attribute is set .

[Alexandre Julliard]

Module: wine
Branch: master
Commit: f4eea10bed0d2145e9fd5403f140156579356702
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=f4eea10bed0d2145e9fd5403f1...

Author: James Hawkins <truiken(a)gmail.com>
Date:   Thu Dec 20 02:44:24 2007 -0600

msi: The BS_GROUPBOX style should only be used if the HasBorder attribute is set.

---

 dlls/msi/dialog.c |    9 ++++++++-
 1 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/dlls/msi/dialog.c b/dlls/msi/dialog.c
index ccb6acc..4994216 100644
--- a/dlls/msi/dialog.c
+++ b/dlls/msi/dialog.c
@@ -1750,13 +1750,20 @@ static UINT msi_dialog_radiogroup_control( msi_dialog *dialog, MSIRECORD *rec )
     radio_button_group_descr group;
     MSIPACKAGE *package = dialog->package;
     WNDPROC oldproc;
+    DWORD attr, style = WS_GROUP;
 
     prop = MSI_RecordGetString( rec, 9 );
 
     TRACE("%p %p %s\n", dialog, rec, debugstr_w( prop ));
 
+    attr = MSI_RecordGetInteger( rec, 8 );
+    if (attr & msidbControlAttributesHasBorder)
+        style |= BS_GROUPBOX;
+    else
+        style |= BS_OWNERDRAW;
+
     /* Create parent group box to hold radio buttons */
-    control = msi_dialog_add_control( dialog, rec, szButton, BS_GROUPBOX|WS_GROUP );
+    control = msi_dialog_add_control( dialog, rec, szButton, style );
     if( !control )
         return ERROR_FUNCTION_FAILED;