From ff163df61ebe8d8e92643fc2f72d1a800c6e48f5 Mon Sep 17 00:00:00 2001
From: diminator <dimi.dejonghe@gmail.com>
Date: Thu, 17 Sep 2015 14:20:46 +0200
Subject: [PATCH] PR update

---
 js/components/ascribe_detail/edition.js          |  5 +++--
 js/components/ascribe_detail/piece_container.js  |  5 ++---
 .../ascribe_forms/form_copyright_association.js  |  2 +-
 js/components/ascribe_forms/form_login.js        |  2 +-
 js/components/ascribe_forms/form_signup.js       |  2 +-
 .../ascribe_detail/cyland_piece_container.js     |  5 ++---
 .../ascribe_detail/ikonotv_piece_container.js    |  5 ++---
 .../ikonotv/ikonotv_contract_notifications.js    | 16 +++++++---------
 .../wallet/components/ikonotv/ikonotv_landing.js |  3 +--
 .../components/ikonotv/ikonotv_register_piece.js | 10 ++--------
 js/fetchers/ownership_fetcher.js                 |  8 ++++++++
 11 files changed, 30 insertions(+), 33 deletions(-)

diff --git a/js/components/ascribe_detail/edition.js b/js/components/ascribe_detail/edition.js
index 1e69a7ab..f2dbf682 100644
--- a/js/components/ascribe_detail/edition.js
+++ b/js/components/ascribe_detail/edition.js
@@ -152,8 +152,9 @@ let Edition = React.createClass({
 
                     <CollapsibleParagraph
                         title="Notes"
-                        show={(!!this.state.currentUser.username) ||
-                                (this.props.edition.acl.acl_edit || this.props.edition.public_note)}>
+                        show={!!(this.state.currentUser.username
+                                || this.props.edition.acl.acl_edit
+                                || this.props.edition.public_note)}>
                         <Note
                             id={() => {return {'bitcoin_id': this.props.edition.bitcoin_id}; }}
                             label={getLangText('Personal note (private)')}
diff --git a/js/components/ascribe_detail/piece_container.js b/js/components/ascribe_detail/piece_container.js
index 4f8d8d8d..f14cf743 100644
--- a/js/components/ascribe_detail/piece_container.js
+++ b/js/components/ascribe_detail/piece_container.js
@@ -237,12 +237,11 @@ let PieceContainer = React.createClass({
                     </CollapsibleParagraph>
                     <CollapsibleParagraph
                         title={getLangText('Notes')}
-                        show={(!!this.state.currentUser.username) ||
-                                (this.state.piece.public_note)}>
+                        show={!!(this.state.currentUser.username || this.state.piece.public_note)}>
                         <Note
                             id={this.getId}
                             label={getLangText('Personal note (private)')}
-                            defaultValue={this.state.piece.private_note ? this.state.piece.private_note : null}
+                            defaultValue={this.state.piece.private_note || null}
                             placeholder={getLangText('Enter your comments ...')}
                             editable={true}
                             successMessage={getLangText('Private note saved')}
diff --git a/js/components/ascribe_forms/form_copyright_association.js b/js/components/ascribe_forms/form_copyright_association.js
index d510a252..53c7245f 100644
--- a/js/components/ascribe_forms/form_copyright_association.js
+++ b/js/components/ascribe_forms/form_copyright_association.js
@@ -49,7 +49,7 @@ let CopyrightAssociationForm = React.createClass({
                         style={{paddingBottom: 0}}>
                         <select name="contract">
                             <option disabled selected={selectedState === -1}>
-                                {getLangText(' -- select an association -- ')}
+                                {' -- ' + getLangText('select an association') + ' -- '}
                             </option>
                             {AppConstants.copyrightAssociations.map((association, i) => {
                                 return (
diff --git a/js/components/ascribe_forms/form_login.js b/js/components/ascribe_forms/form_login.js
index 1c6fd0ad..79bced6c 100644
--- a/js/components/ascribe_forms/form_login.js
+++ b/js/components/ascribe_forms/form_login.js
@@ -95,7 +95,7 @@ let LoginForm = React.createClass({
     },
 
     render() {
-        let email = this.getQuery().email ? this.getQuery().email : null;
+        let email = this.getQuery().email || null;
         return (
             <Form
                 className="ascribe-form-bordered"
diff --git a/js/components/ascribe_forms/form_signup.js b/js/components/ascribe_forms/form_signup.js
index 928afe38..b713b4f8 100644
--- a/js/components/ascribe_forms/form_signup.js
+++ b/js/components/ascribe_forms/form_signup.js
@@ -77,7 +77,7 @@ let SignupForm = React.createClass({
         let tooltipPassword = getLangText('Your password must be at least 10 characters') + '.\n ' +
             getLangText('This password is securing your digital property like a bank account') + '.\n ' +
             getLangText('Store it in a safe place') + '!';
-        let email = this.getQuery().email ? this.getQuery().email : null;
+        let email = this.getQuery().email || null;
         return (
             <Form
                 className="ascribe-form-bordered"
diff --git a/js/components/whitelabel/wallet/components/cyland/ascribe_detail/cyland_piece_container.js b/js/components/whitelabel/wallet/components/cyland/ascribe_detail/cyland_piece_container.js
index 5b605007..dab5eda4 100644
--- a/js/components/whitelabel/wallet/components/cyland/ascribe_detail/cyland_piece_container.js
+++ b/js/components/whitelabel/wallet/components/cyland/ascribe_detail/cyland_piece_container.js
@@ -98,12 +98,11 @@ let CylandPieceContainer = React.createClass({
                     </CollapsibleParagraph>
                     <CollapsibleParagraph
                         title={getLangText('Notes')}
-                        show={(!!this.state.currentUser.username) ||
-                                (this.state.piece.public_note)}>
+                        show={!!(this.state.currentUser.username || this.state.piece.public_note)}>
                         <Note
                             id={() => {return {'id': this.state.piece.id}; }}
                             label={getLangText('Personal note (private)')}
-                            defaultValue={this.state.piece.private_note ? this.state.piece.private_note : null}
+                            defaultValue={this.state.piece.private_note || null}
                             placeholder={getLangText('Enter your comments ...')}
                             editable={true}
                             successMessage={getLangText('Private note saved')}
diff --git a/js/components/whitelabel/wallet/components/ikonotv/ascribe_detail/ikonotv_piece_container.js b/js/components/whitelabel/wallet/components/ikonotv/ascribe_detail/ikonotv_piece_container.js
index 9ee94245..3efa2185 100644
--- a/js/components/whitelabel/wallet/components/ikonotv/ascribe_detail/ikonotv_piece_container.js
+++ b/js/components/whitelabel/wallet/components/ikonotv/ascribe_detail/ikonotv_piece_container.js
@@ -97,12 +97,11 @@ let IkonotvPieceContainer = React.createClass({
                     </CollapsibleParagraph>
                     <CollapsibleParagraph
                         title={getLangText('Notes')}
-                        show={(!!this.state.currentUser.username) ||
-                                (this.state.piece.public_note)}>
+                        show={!!(this.state.currentUser.username || this.state.piece.public_note)}>
                         <Note
                             id={() => {return {'id': this.state.piece.id}; }}
                             label={getLangText('Personal note (private)')}
-                            defaultValue={this.state.piece.private_note ? this.state.piece.private_note : null}
+                            defaultValue={this.state.piece.private_note || null}
                             placeholder={getLangText('Enter your comments ...')}
                             editable={true}
                             successMessage={getLangText('Private note saved')}
diff --git a/js/components/whitelabel/wallet/components/ikonotv/ikonotv_contract_notifications.js b/js/components/whitelabel/wallet/components/ikonotv/ikonotv_contract_notifications.js
index 6ba8ca95..0f631758 100644
--- a/js/components/whitelabel/wallet/components/ikonotv/ikonotv_contract_notifications.js
+++ b/js/components/whitelabel/wallet/components/ikonotv/ikonotv_contract_notifications.js
@@ -12,6 +12,8 @@ import NotificationStore from '../../../../../stores/notification_store';
 import UserActions from '../../../../../actions/user_actions';
 import UserStore from '../../../../../stores/user_store';
 
+import OwnershipFetcher from '../../../../../fetchers/ownership_fetcher';
+
 import WhitelabelStore from '../../../../../stores/whitelabel_store';
 import WhitelabelActions from '../../../../../actions/whitelabel_actions';
 
@@ -21,9 +23,7 @@ import GlobalNotificationActions from '../../../../../actions/global_notificatio
 import CopyrightAssociationForm from '../../../../ascribe_forms/form_copyright_association';
 
 import AppConstants from '../../../../../constants/application_constants';
-import ApiUrls from '../../../../../constants/api_urls';
 
-import requests from '../../../../../utils/requests';
 import { getLangText } from '../../../../../utils/lang_utils';
 import { mergeOptions } from '../../../../../utils/general_utils';
 
@@ -110,7 +110,7 @@ let IkonotvContractNotifications = React.createClass({
 
     handleConfirm() {
         let contractAgreement = this.state.contractAgreementListNotifications[0].contract_agreement;
-        requests.put(ApiUrls.ownership_contract_agreements_confirm, {contract_agreement_id: contractAgreement.id}).then(
+        OwnershipFetcher.confirmContractAgreement(contractAgreement).then(
             () => this.handleConfirmSuccess()
         );
     },
@@ -123,7 +123,7 @@ let IkonotvContractNotifications = React.createClass({
 
     handleDeny() {
         let contractAgreement = this.state.contractAgreementListNotifications[0].contract_agreement;
-        requests.put(ApiUrls.ownership_contract_agreements_deny, {contract_agreement_id: contractAgreement.id}).then(
+        OwnershipFetcher.denyContractAgreement(contractAgreement).then(
             () => this.handleDenySuccess()
         );
     },
@@ -135,11 +135,9 @@ let IkonotvContractNotifications = React.createClass({
     },
 
     getCopyrightAssociationForm(){
-        if (this.state.currentUser && this.state.currentUser.profile
-            && this.state.currentUser.profile.copyright_association){
-            return null;
-        }
-        if (this.state.currentUser && this.state.currentUser.profile) {
+        let c = this.state.currentUser;
+
+        if (c && c.profile && !c.profile.copyright_association) {
             return (
                 <div className='notification-contract-footer'>
                     <h1>{getLangText('Are you a member of any copyright societies?')}</h1>
diff --git a/js/components/whitelabel/wallet/components/ikonotv/ikonotv_landing.js b/js/components/whitelabel/wallet/components/ikonotv/ikonotv_landing.js
index 7abfd62c..3fee2e50 100644
--- a/js/components/whitelabel/wallet/components/ikonotv/ikonotv_landing.js
+++ b/js/components/whitelabel/wallet/components/ikonotv/ikonotv_landing.js
@@ -31,7 +31,7 @@ let IkonotvLanding = React.createClass({
     },
 
     getEnterButton() {
-        let redirect = 'signup';
+        let redirect = 'login';
 
         if(this.state.currentUser && this.state.currentUser.email) {
             redirect = 'pieces';
@@ -39,7 +39,6 @@ let IkonotvLanding = React.createClass({
         else if (this.getQuery() && this.getQuery().redirect) {
             redirect = this.getQuery().redirect;
         }
-        console.log(redirect);
         return (
             <ButtonLink to={redirect} query={this.getQuery()}>
                 {getLangText('ENTER')}
diff --git a/js/components/whitelabel/wallet/components/ikonotv/ikonotv_register_piece.js b/js/components/whitelabel/wallet/components/ikonotv/ikonotv_register_piece.js
index ecb594c5..700b5678 100644
--- a/js/components/whitelabel/wallet/components/ikonotv/ikonotv_register_piece.js
+++ b/js/components/whitelabel/wallet/components/ikonotv/ikonotv_register_piece.js
@@ -165,14 +165,8 @@ let IkonotvRegisterPiece = React.createClass({
     },
 
     canSubmit() {
-        if (this.state.currentUser && this.state.currentUser.acl) {
-            return (
-                this.state.currentUser
-                && this.state.currentUser.acl
-                && !!this.state.currentUser.acl.acl_submit
-            );
-        }
-        return false;
+        let c = this.state.currentUser;
+        return c && c.acl && c.acl.acl_submit;
     },
 
     getSlideArtistDetails() {
diff --git a/js/fetchers/ownership_fetcher.js b/js/fetchers/ownership_fetcher.js
index f9ce5f86..b0d88927 100644
--- a/js/fetchers/ownership_fetcher.js
+++ b/js/fetchers/ownership_fetcher.js
@@ -44,6 +44,14 @@ let OwnershipFetcher = {
         return requests.get(ApiUrls.ownership_contract_agreements, queryParams);
     },
 
+    confirmContractAgreement(contractAgreement){
+        return requests.put(ApiUrls.ownership_contract_agreements_confirm, {contract_agreement_id: contractAgreement.id});
+    },
+
+    denyContractAgreement(contractAgreement){
+        return requests.put(ApiUrls.ownership_contract_agreements_deny, {contract_agreement_id: contractAgreement.id});
+    },
+
     fetchLoanPieceRequestList(){
         return requests.get(ApiUrls.ownership_loans_pieces_request);
     },