From 0abd65cb71f275e6f40d54059322cc9ce087c063 Mon Sep 17 00:00:00 2001
From: Sergey Yelin <elinsn@gmail.com>
Date: Mon, 31 Aug 2020 19:07:07 +0300
Subject: [PATCH] Review fixes

---
 proto/base.thrift    | 57 +++++++++++++++++++++++++++++++++++++++--
 proto/event.thrift   |  3 +--
 proto/storage.thrift | 60 +++-----------------------------------------
 3 files changed, 59 insertions(+), 61 deletions(-)

diff --git a/proto/base.thrift b/proto/base.thrift
index 3d11128..5af8a2a 100644
--- a/proto/base.thrift
+++ b/proto/base.thrift
@@ -1,10 +1,12 @@
-namespace java com.rbkmoney.pstds.event
-namespace erlang pstds_event
+namespace java com.rbkmoney.pstds.base
+namespace erlang pstds_base
 
 typedef i64 EventID
 typedef i32 SequenceID
 typedef string Timestamp
 typedef string Token
+typedef string PaymentToken
+typedef string EnrollmentID
 
 /**
 * Поддерживаемые платёжные системы
@@ -46,3 +48,54 @@ struct PaymentSystemToken {
     2: required PaymentSystem payment_system
     3: required Revision revision
 }
+
+/** Дата экспирации */
+struct ExpDate {
+    /** Месяц 1..12 */
+    1: required i8 month
+    /** Год 2015..∞ */
+    2: required i16 year
+}
+
+struct PaymentSystemTokenData {
+    /**
+    * Токен МПС:
+    * - VISA: vProvisionedTokenID
+    * - MASTERCARD: tokenUniqueReference
+    * - NSPKMIR: tokenNumber
+    **/
+    1: required PaymentToken tokenID
+
+    /**
+    * Энролмент МПС:
+    * - VISA: vPanEnrollmentID
+    * - MASTERCARD: panUniqueReference
+    * - NSPKMIR: subscriptionID (?)
+    **/
+    2: required EnrollmentID enrollmentID
+
+    /**
+    * Идентификатор МПС
+    **/
+    3: required PaymentSystem payment_system
+
+    /**
+    * Статус токена
+    **/
+    4: required TokenStatus status
+
+    /**
+    * Токен банковской карты, для которого выписан токен МПС
+    **/
+    5: required Token bank_card_token
+
+    /**
+    * Дата экспирации токена
+    **/
+    6: optional ExpDate exp_date
+
+    /**
+    * Уникальный идентификатор карты в МПС (аналоги и замена PAN)
+    **/
+    7: optional string pan_account_reference
+}
diff --git a/proto/event.thrift b/proto/event.thrift
index f6e0384..218dc69 100644
--- a/proto/event.thrift
+++ b/proto/event.thrift
@@ -15,10 +15,9 @@ union Change {
 }
 
 struct TokenCreated {
-    1: base.PaymentSystemToken token
+    1: base.PaymentSystemTokenData token_data
 }
 
 struct StatusChanged {
-    1: required base.PaymentSystemToken token
     2: required base.TokenStatus new_status
 }
diff --git a/proto/storage.thrift b/proto/storage.thrift
index b13b3f6..778f981 100644
--- a/proto/storage.thrift
+++ b/proto/storage.thrift
@@ -3,60 +3,6 @@ namespace erlang pstds
 
 include "base.thrift"
 
-typedef string PaymentToken
-typedef string EnrollmentID
-
-/** Дата экспирации */
-struct ExpDate {
-    /** Месяц 1..12 */
-    1: required i8 month
-    /** Год 2015..∞ */
-    2: required i16 year
-}
-
-struct PaymentSystemTokenData {
-    /**
-    * Токен МПС:
-    * - VISA: vProvisionedTokenID
-    * - MASTERCARD: tokenUniqueReference
-    * - NSPKMIR: tokenNumber
-    **/
-    1: required PaymentToken tokenID
-
-    /**
-    * Энролмент МПС:
-    * - VISA: vPanEnrollmentID
-    * - MASTERCARD: panUniqueReference
-    * - NSPKMIR: subscriptionID (?)
-    **/
-    2: required EnrollmentID enrollmentID
-
-    /**
-    * Идентификатор МПС
-    **/
-    3: required base.PaymentSystem payment_system
-
-    /**
-    * Статус токена
-    **/
-    4: required base.TokenStatus status
-
-    /**
-    * Токен банковской карты, для которого выписан токен МПС
-    **/
-    5: required base.Token bank_card_token
-
-    /**
-    * Дата экспирации токена
-    **/
-    6: optional ExpDate exp_date
-
-    /**
-    * Уникальный идентификатор карты в МПС (аналоги и замена PAN)
-    **/
-    7: optional string pan_account_reference
-}
-
 struct PaymentSystemTokenResult {
     1: required base.PaymentSystemToken payment_system_token
 }
@@ -84,15 +30,15 @@ exception InvalidPaymentSystemToken {
 service Storage {
 
     /** Получить данные платёжного токена */
-    PaymentSystemTokenData GetPaymentSystemTokenData(1: base.PaymentSystemToken token)
+    base.PaymentSystemTokenData GetPaymentSystemTokenData(1: base.PaymentSystemToken token)
         throws (1: PaymentSystemTokenNotFound not_found)
 
     /** Получить данные активного платёжного токена по токену банковской карты */
-    PaymentSystemTokenData GetPaymentSystemTokenByBankCardToken(1: base.Token token)
+    base.PaymentSystemTokenData GetPaymentSystemTokenByBankCardToken(1: base.Token token)
         throws (1: PaymentSystemTokenNotFound not_found)
 
     /** Сохранить платёжный токен */
-    PaymentSystemTokenResult PutPaymentSystemToken(1: PaymentSystemTokenData payment_system_token)
+    PaymentSystemTokenResult PutPaymentSystemToken(1: base.PaymentSystemTokenData payment_system_token)
         throws (1: InvalidPaymentSystemToken invalid)
 
     /** Обновить статус платёжного токена