fix: Commerce product group bug./

Backported version tag fix to 7.1.2

docs/changelog/7.x.x.txt

@@ -3,12 +3,13 @@
  - Added configurable sales tax. (Tiffany Patterson / Elite Marketing)
  - change: made Text::Aspell optional, nullifying spellchecker if not present
  - Fixed a bug where logging in/out would cause a blank page display.
- - Fix: Version tags could not be create()d because no default values set.
 
 7.1.2
  - Fixed a bug that caused workflows to fail if collaboration systems and
     posts for that CS were in the same version tag at commit time.
-  - fix: minor assetsToHide implementation bug in dashboard
+ - fix: minor assetsToHide implementation bug in dashboard
+ - fix: Version tags could not be create()d because no default values set.
+ - fix: Commerce items were required to have a group.
 
 7.1.1
  - fix: some issues with asset exports not handling URLs with dots correctly

docs/upgrades/upgrade_7.1.1-7.1.2.pl

@@ -21,6 +21,8 @@ my $quiet; # this line required
 my $session = start(); # this line required
 
 # upgrade functions go here
+fixAssetVersionTag($session);
+
 
 finish($session); # this line required
 
@@ -32,6 +34,16 @@ finish($session); # this line required
 #	# and here's our code
 #}
 
+#-------------------------------------------------
+sub fixAssetVersionTag {
+	my $session = shift;
+	print "\tFixing assetVersionTag database table.\n" unless ($quiet);
+	
+	$session->db->write("alter table assetVersionTag alter lockedBy set default ''");
+	$session->db->write("alter table assetVersionTag alter groupToUse set default ''");
+	$session->db->write("alter table assetVersionTag alter workflowId set default ''");
+}
+
 
 
 # ---- DO NOT EDIT BELOW THIS LINE ----

docs/upgrades/upgrade_7.1.2-7.2.0.pl

@@ -59,17 +59,6 @@ sub createDictionaryStorage {
 }
 
 
-#-------------------------------------------------
-sub fixAssetVersionTag {
-	my $session = shift;
-	print "\tFixing assetVersionTag database table.\n" unless ($quiet);
-	
-	$session->db->write("alter table assetVersionTag alter lockedBy set default ''");
-	$session->db->write("alter table assetVersionTag alter groupToUse set default ''");
-	$session->db->write("alter table assetVersionTag alter workflowId set default ''");
-}
-
-
 # ---- DO NOT EDIT BELOW THIS LINE ----
 
 #-------------------------------------------------

lib/WebGUI/Commerce/Item/Product.pm

@@ -48,7 +48,7 @@ sub handler {
 
        ### Add to group action
        # If group is 'everyone', skip
-       unless ($self->{_product}->get('groupId') eq '7') {
+       unless ($self->{_product}->get('groupId') && $self->{_product}->get('groupId') eq '7') {
                my $g = WebGUI::Group->new($self->session,$self->{_product}->get('groupId'));
                my $expiresOffset;