Browse Source

Merge pull request #153 from siman/alex-improve-name-consistency

Rename Channel.publishing_status -> publication_status
Mokhtar Naamani 5 years ago
parent
commit
081553f89c
2 changed files with 20 additions and 20 deletions
  1. 15 15
      src/content_working_group/lib.rs
  2. 5 5
      src/content_working_group/tests.rs

+ 15 - 15
src/content_working_group/lib.rs

@@ -542,19 +542,19 @@ impl Default for ChannelContentType {
 /// Is only meant to affect visibility, mutation of channel and child content
 /// is unaffected on runtime.
 #[derive(Encode, Decode, Debug, Clone, PartialEq)]
-pub enum ChannelPublishingStatus {
+pub enum ChannelPublicationStatus {
     /// Compliant UIs should render.
-    Published,
+    Public,
 
     /// Compliant UIs should not render it or any child content.
-    NotPublished,
+    Unlisted,
 }
 
 /// Must be default constructible because it indirectly is a value in a storage map.
 /// ***SHOULD NEVER ACTUALLY GET CALLED, IS REQUIRED TO DUE BAD STORAGE MODEL IN SUBSTRATE***
-impl Default for ChannelPublishingStatus {
+impl Default for ChannelPublicationStatus {
     fn default() -> Self {
-        ChannelPublishingStatus::Published
+        ChannelPublicationStatus::Public
     }
 }
 
@@ -610,7 +610,7 @@ pub struct Channel<MemberId, AccountId, BlockNumber, PrincipalId> {
     pub role_account: AccountId,
 
     /// Publication status of channel.
-    pub publishing_status: ChannelPublishingStatus,
+    pub publication_status: ChannelPublicationStatus,
 
     /// Curation status of channel.
     pub curation_status: ChannelCurationStatus,
@@ -1092,7 +1092,7 @@ decl_module! {
             description: OptionalText,
             avatar: OptionalText,
             banner: OptionalText,
-            publishing_status: ChannelPublishingStatus
+            publication_status: ChannelPublicationStatus
         ) {
 
             // Ensure that it is signed
@@ -1145,7 +1145,7 @@ decl_module! {
                 content: content,
                 owner: owner,
                 role_account: role_account,
-                publishing_status: publishing_status,
+                publication_status: publication_status,
                 curation_status: ChannelCurationStatus::Normal,
                 created: <system::Module<T>>::block_number(),
                 principal_id: principal_id
@@ -1175,7 +1175,7 @@ decl_module! {
         /// An owner transfers channel ownership to a new owner.
         ///
         /// Notice that working group participants cannot do this.
-        /// Notice that censored or unpublished channel may still be transferred.
+        /// Notice that censored or unlisted channel may still be transferred.
         /// Notice that transfers are unilateral, so new owner cannot block. This may be problematic: https://github.com/Joystream/substrate-runtime-joystream/issues/95
         pub fn transfer_channel_ownership(origin, channel_id: ChannelId<T>, new_owner: T::MemberId, new_role_account: T::AccountId) {
 
@@ -1225,7 +1225,7 @@ decl_module! {
             new_description: Option<OptionalText>,
             new_avatar: Option<OptionalText>,
             new_banner: Option<OptionalText>,
-            new_publishing_status: Option<ChannelPublishingStatus>
+            new_publication_status: Option<ChannelPublicationStatus>
         ) {
 
             // Ensure channel owner has signed
@@ -1268,7 +1268,7 @@ decl_module! {
                 &new_description,
                 &new_avatar,
                 &new_banner,
-                &new_publishing_status,
+                &new_publication_status,
                 &None // curation_status
             );
         }
@@ -1297,7 +1297,7 @@ decl_module! {
                 &None, // description,
                 &None, // avatar
                 &None, // banner
-                &None, // publishing_status
+                &None, // publication_status
                 &new_curation_status
             );
         }
@@ -2612,7 +2612,7 @@ impl<T: Trait> Module<T> {
         new_description: &Option<OptionalText>,
         new_avatar: &Option<OptionalText>,
         new_banner: &Option<OptionalText>,
-        new_publishing_status: &Option<ChannelPublishingStatus>,
+        new_publication_status: &Option<ChannelPublicationStatus>,
         new_curation_status: &Option<ChannelCurationStatus>,
     ) {
         // Update channel id to handle mapping, if there is a new handle.
@@ -2651,8 +2651,8 @@ impl<T: Trait> Module<T> {
                 channel.banner = banner.clone();
             }
 
-            if let Some(ref publishing_status) = new_publishing_status {
-                channel.publishing_status = publishing_status.clone();
+            if let Some(ref publication_status) = new_publication_status {
+                channel.publication_status = publication_status.clone();
             }
 
             if let Some(ref curation_status) = new_curation_status {

+ 5 - 5
src/content_working_group/tests.rs

@@ -215,7 +215,7 @@ impl UpdateChannelAsCurationActorFixture {
             content: old_channel.content,
             owner: old_channel.owner,
             role_account: old_channel.role_account,
-            publishing_status: old_channel.publishing_status,
+            publication_status: old_channel.publication_status,
             curation_status: upd_curation_status,
             created: old_channel.created,
             principal_id: old_channel.principal_id,
@@ -1825,7 +1825,7 @@ struct CreateChannelFixture {
     pub avatar: OptionalText,
     pub banner: OptionalText,
     pub content: ChannelContentType,
-    pub publishing_status: ChannelPublishingStatus,
+    pub publication_status: ChannelPublicationStatus,
 }
 
 impl CreateChannelFixture {
@@ -1851,7 +1851,7 @@ impl CreateChannelFixture {
             banner: Some(generate_valid_length_buffer(&ChannelBannerConstraint::get())),
             description: Some(generate_valid_length_buffer(&ChannelDescriptionConstraint::get())),
             content: ChannelContentType::Video,
-            publishing_status: ChannelPublishingStatus::NotPublished,
+            publication_status: ChannelPublicationStatus::Unlisted,
         }
     }
 
@@ -1866,7 +1866,7 @@ impl CreateChannelFixture {
             self.description.clone(),
             self.avatar.clone(),
             self.banner.clone(),
-            self.publishing_status.clone(),
+            self.publication_status.clone(),
         )
     }
 
@@ -1914,7 +1914,7 @@ impl CreateChannelFixture {
             content: self.content.clone(),
             owner: self.channel_creator_member_id,
             role_account: self.channel_creator_role_account,
-            publishing_status: self.publishing_status.clone(),
+            publication_status: self.publication_status.clone(),
             curation_status: ChannelCurationStatus::Normal,
             created: 1, // <== replace with now()