[FAB-9851] Enable PvtChanData by default in v1.2

In v1.1, the private data feature within a channel
is an experimental feature. Hence, one needs to set
privateChannelData capability in config tx to
enable it. In v1.2, private data feature is
enabled by default, i.e., non-experimental.

As a result, privateChannelData() capability API in
v1.2 should return (privateChannelData || v1.2) to
support rolling upgrades. In v1.3, we can remove the
privateChannelData capability flag.

Change-Id: I9f1adbe87b8bab1d98b89602cca410fba18aba86
Signed-off-by: senthil <cendhu@gmail.com>

Author: cendhu

common/capabilities/application.go

@@ -71,8 +71,10 @@ func (ap *ApplicationProvider) ForbidDuplicateTXIdInBlock() bool {
 }
 
 // PrivateChannelData returns true if support for private channel data (a.k.a. collections) is enabled.
+// In v1.1, the private channel data is experimental and has to be enabled explicitly.
+// In v1.2, the private channel data is enabled by default.
 func (ap *ApplicationProvider) PrivateChannelData() bool {
-	return ap.v11PvtDataExperimental
+	return ap.v11PvtDataExperimental || ap.v12
 }
 
 // V1_1Validation returns true is this channel is configured to perform stricter validation

common/capabilities/application_test.go

@@ -44,6 +44,12 @@ func TestApplicationPvtDataExperimental(t *testing.T) {
 		ApplicationPvtDataExperimental: {},
 	})
 	assert.True(t, op.PrivateChannelData())
+
+	op = NewApplicationProvider(map[string]*cb.Capability{
+		ApplicationV1_2: {},
+	})
+	assert.True(t, op.PrivateChannelData())
+
 }
 
 func TestChaincodeLifecycleExperimental(t *testing.T) {

common/channelconfig/api.go

@@ -127,6 +127,8 @@ type ApplicationCapabilities interface {
 	ResourcesTree() bool
 
 	// PrivateChannelData returns true if support for private channel data (a.k.a. collections) is enabled.
+	// In v1.1, the private channel data is experimental and has to be enabled explicitly.
+	// In v1.2, the private channel data is enabled by default.
 	PrivateChannelData() bool
 
 	// V1_1Validation returns true is this channel is configured to perform stricter validation