Refaccionats els components, funcions i control·ladors de rutes relat…

…ius a les acceptacions Eliminades les restes de funcions marcades com a obsoletes
albertms10 · May 31, 2020 · 2a6a8d4 · 2a6a8d4
1 parent 551892a
commit 2a6a8d4
Show file tree

Hide file tree

Showing 14 changed files with 59 additions and 320 deletions.
diff --git a/client/src/components/avis-acceptacio/avis-acceptacio.js b/client/src/components/avis-acceptacio/avis-acceptacio.js
@@ -7,11 +7,12 @@ import { CheckboxAcceptacioIndependent } from "./components/checkbox-acceptacio-
 import { SeccioAvis } from "./components/seccio-avis";
 import { useAvisAcceptacio } from "./hooks";
 
-const AvisAcceptacio = ({ idAvis, acceptacionsSoci, isForm }) => {
-  const [textAvisAcceptacio, loading] = useAvisAcceptacio(idAvis);
+const AvisAcceptacio = ({ nameAvis, acceptacionsSoci, isForm }) => {
+  const [textAvisAcceptacio, loading] = useAvisAcceptacio(nameAvis);
 
   return (
     <SettingCard title={textAvisAcceptacio.titol} loading={loading}>
+      <SeccioAvis descripcio={textAvisAcceptacio.descripcio} />
       {textAvisAcceptacio.hasOwnProperty("seccions") &&
         textAvisAcceptacio.seccions.map(({ id, titol, descripcio }) => (
           <SeccioAvis key={id} titol={titol} descripcio={descripcio} />
@@ -38,13 +39,12 @@ const AvisAcceptacio = ({ idAvis, acceptacionsSoci, isForm }) => {
 };
 
 AvisAcceptacio.propTypes = {
-  idAvis: PropTypes.number,
+  nameAvis: PropTypes.string.isRequired,
   acceptacionsSoci: AcceptacionsSociPropTypes,
   isForm: PropTypes.bool,
 };
 
 AvisAcceptacio.defaultProps = {
-  idAvis: 1,
   acceptacionsSoci: {},
   isForm: false,
 };

diff --git a/client/src/components/avis-acceptacio/hooks/use-avis-acceptacio.js b/client/src/components/avis-acceptacio/hooks/use-avis-acceptacio.js
@@ -1,22 +1,18 @@
 import { useEffect, useState } from "react";
 
-/**
- * @param {number} idAvis
- * @returns {[Object, boolean]}
- */
-export default (idAvis) => {
+export default (nameAvis) => {
   const [textAvisAcceptacio, setTextAvisAcceptacio] = useState({});
   const [loading, setLoading] = useState(false);
 
   useEffect(() => {
     setLoading(true);
-    fetch(`/api/associacio/avisos/${idAvis}`)
+    fetch(`/api/associacio/avisos/${nameAvis}`)
       .then((res) => res.json())
       .then((data) => {
         setTextAvisAcceptacio(data);
         setLoading(false);
       });
-  }, [idAvis]);
+  }, [nameAvis]);
 
   return [textAvisAcceptacio, loading];
 };
diff --git a/client/src/components/steps-afegir-soci/components/resum-afegir-soci/resum-afegir-soci.js b/client/src/components/steps-afegir-soci/components/resum-afegir-soci/resum-afegir-soci.js
@@ -2,16 +2,8 @@ import { Descriptions, Space, Spin } from "antd";
 import PropTypes from "prop-types";
 import React from "react";
 import { SettingCard } from "../../../../standalone/setting-card";
-import { StatusIcon } from "../../../../standalone/status-icon";
 
-const ResumAfegirSoci = ({
-  form,
-  selfCreation,
-  username,
-  loadingUsername,
-  acceptaProteccioDades,
-  acceptaDretsImatge,
-}) => {
+const ResumAfegirSoci = ({ form, username, loadingUsername }) => {
   const data = form.getFieldsValue();
 
   return (
@@ -31,94 +23,59 @@ const ResumAfegirSoci = ({
           </Descriptions>
         }
       />
-      {data.experiencia_musical || data.estudis_musicals || data.data_alta ? (
-        <SettingCard
-          title="Informació musical"
-          info={
-            <Descriptions size="small">
-              {data.experiencia_musical ? (
-                <Descriptions.Item label="Experiència musical" span={3}>
-                  {data.experiencia_musical}
-                </Descriptions.Item>
-              ) : (
-                ""
-              )}
-              {data.estudis_musicals ? (
-                <Descriptions.Item label="Estudis musicals" span={3}>
-                  {data.estudis_musicals}
-                </Descriptions.Item>
-              ) : (
-                ""
-              )}
-              {data.data_alta ? (
-                <Descriptions.Item label="Data d’alta">
-                  {data.data_alta.format("LL")}
-                </Descriptions.Item>
-              ) : (
-                ""
-              )}
-            </Descriptions>
-          }
-        />
-      ) : (
-        ""
-      )}
+      {data.experiencia_musical ||
+        data.estudis_musicals ||
+        (data.data_alta && (
+          <SettingCard
+            title="Informació musical"
+            info={
+              <Descriptions size="small">
+                {data.experiencia_musical && (
+                  <Descriptions.Item label="Experiència musical" span={3}>
+                    {data.experiencia_musical}
+                  </Descriptions.Item>
+                )}
+                {data.estudis_musicals && (
+                  <Descriptions.Item label="Estudis musicals" span={3}>
+                    {data.estudis_musicals}
+                  </Descriptions.Item>
+                )}
+                {data.data_alta && (
+                  <Descriptions.Item label="Data d’alta">
+                    {data.data_alta.format("LL")}
+                  </Descriptions.Item>
+                )}
+              </Descriptions>
+            }
+          />
+        ))}
       <SettingCard
         title="Dades de contacte"
         info={
           <Descriptions size="small">
             <Descriptions.Item label="Adreça electrònica" span={2}>
               {data.email}
             </Descriptions.Item>
-            {data.telefon ? (
+            {data.telefon && (
               <Descriptions.Item label="Telèfon">
                 {data.telefon}
               </Descriptions.Item>
-            ) : (
-              ""
             )}
           </Descriptions>
         }
       />
-      {selfCreation && (
-        <SettingCard
-          title="Acceptacions"
-          info={
-            <Descriptions size="small">
-              <Descriptions.Item label="Protecció de dades">
-                <StatusIcon
-                  statusId={acceptaProteccioDades ? 1 : 3}
-                  label={acceptaProteccioDades ? "Accepto" : "No accepto"}
-                />
-              </Descriptions.Item>
-              <Descriptions.Item label="Drets d’imatge">
-                <StatusIcon
-                  statusId={acceptaDretsImatge ? 1 : 3}
-                  label={acceptaDretsImatge ? "Accepto" : "No accepto"}
-                />
-              </Descriptions.Item>
-            </Descriptions>
-          }
-        />
-      )}
     </Space>
   );
 };
 
 ResumAfegirSoci.propTypes = {
   form: PropTypes.any.isRequired,
-  selfCreation: PropTypes.bool,
   username: PropTypes.string.isRequired,
   loadingUsername: PropTypes.bool,
-  acceptaProteccioDades: PropTypes.bool,
-  acceptaDretsImatge: PropTypes.bool,
 };
 
 ResumAfegirSoci.defaultProps = {
-  selfCreation: false,
   loadingUsername: false,
-  acceptaProteccioDades: false,
-  acceptaDretsImatge: false,
 };
 
 export default ResumAfegirSoci;
diff --git a/client/src/components/steps-afegir-soci/hooks/use-steps-afegir-soci.js b/client/src/components/steps-afegir-soci/hooks/use-steps-afegir-soci.js
@@ -10,13 +10,11 @@ import {
   Row,
   Select,
   Space,
-  Typography,
 } from "antd";
 import moment from "moment";
 import React, { useCallback, useState } from "react";
 import { useDispatch } from "react-redux";
 import { fetchAPI } from "../../../helpers";
-import { SettingCard } from "../../../standalone/setting-card";
 import { SubHeader } from "../../../standalone/sub-header";
 import { upperCaseFirst } from "../../../utils";
 import { AvisAcceptacio } from "../../avis-acceptacio";
@@ -25,23 +23,6 @@ import { usePaisos, useUsername } from "./index";
 
 const { Option } = Select;
 const { TextArea } = Input;
-const { Paragraph } = Typography;
-
-export const textDretsImatge = (
-  <Paragraph>
-    Atès que el dret a la imatge es troba regulat per l’article 18.1 de la
-    Constitució, per la Llei Orgànica 1/1982 sobre el dret a l’honor, a la
-    intimitat personal i familiar, i per la Llei Orgànica 15/1999 de Protecció
-    de Dades de Caràcter Personal, sol·licitem el seu consentiment per publicar
-    la seva imatge o veu, de forma clarament identificable, en fotografies i
-    gravacions corresponents a les activitats pròpies de l'associació, i que
-    s’exposin públicament a la pàgina web, revistes, YouTube o altres
-    publicacions internes o de tercers, així com a reproduir-la públicament per
-    a la promoció de les activitats i serveis de les entitats. El present
-    consentiment i autorització s’atorga de forma gratuïta i amb renúncia formal
-    a qualsevol contraprestació econòmica.
-  </Paragraph>
-);
 
 export default (
   onSuccessCallback,
@@ -52,7 +33,6 @@ export default (
 
   const [confirmLoading, setConfirmLoading] = useState(false);
   const [currentPageIndex, setCurrentPageIndex] = useState(0);
-  const [acceptaDretsImatge, setAcceptaDretsImatge] = useState(false);
   const [username, loadingUsername, getUsername] = useUsername();
 
   const [dniValidation, setDniValidation] = useState("");
@@ -91,7 +71,7 @@ export default (
       key: "proteccio",
       title: "Protecció de dades",
       selfCreationOnly: true,
-      content: <AvisAcceptacio isForm />,
+      content: <AvisAcceptacio nameAvis="proteccio_dades" isForm />,
     },
     {
       key: "dades",
@@ -149,7 +129,9 @@ export default (
                     }
                   >
                     {paisos.map((pais) => (
-                      <Option key={pais.id_pais}>{pais.nom}</Option>
+                      <Option key={pais.id_pais} value={pais.id_pais}>
+                        {pais.nom}
+                      </Option>
                     ))}
                   </Select>
                 </Form.Item>
@@ -249,7 +231,7 @@ export default (
                   <TextArea autoSize={{ minRows: 2, maxRows: 4 }} />
                 </Form.Item>
               </Col>
-              {!selfCreation ? (
+              {!selfCreation && (
                 <Col>
                   <Form.Item name="data_alta" label="Data d’alta">
                     <DatePicker
@@ -259,8 +241,6 @@ export default (
                     />
                   </Form.Item>
                 </Col>
-              ) : (
-                ""
               )}
             </Row>
           </Card>
@@ -271,7 +251,7 @@ export default (
       key: "imatge",
       title: "Drets d’imatge",
       selfCreationOnly: true,
-      content: <SettingCard title="Drets d’imatge" info={textDretsImatge} />,
+      content: <AvisAcceptacio nameAvis="drets_imatge" isForm />,
     },
     {
       key: "resum",
@@ -280,11 +260,8 @@ export default (
       content: (
         <ResumAfegirSoci
           form={form}
-          selfCreation={selfCreation}
           username={username}
           loadingUsername={loadingUsername}
-          acceptaProteccioDades={selfCreation}
-          acceptaDretsImatge={acceptaDretsImatge}
         />
       ),
     },
@@ -301,13 +278,6 @@ export default (
       .then((values) => {
         setConfirmLoading(true);
 
-        /** @deprecated */
-        values.accepta_proteccio_dades = selfCreation;
-        /** @deprecated */
-        values.accepta_drets_imatge = selfCreation ? acceptaDretsImatge : false;
-
-        if (selfCreation) values.acceptacions.drets_imatge = acceptaDretsImatge;
-
         values.username = username;
         values.nom = upperCaseFirst(values.nom);
         values.cognoms = upperCaseFirst(values.cognoms);
@@ -359,24 +329,13 @@ export default (
         )}
       </div>
       <Space>
-        {selfCreation && stepsRef[currentPageIndex].key === "imatge" && (
-          <Button
-            key="next"
-            onClick={() => next().then(() => setAcceptaDretsImatge(false))}
-          >
-            No accepto
-          </Button>
-        )}
         {currentPageIndex < stepsRef.length - 1 ? (
           <Button
             key="next"
             type="primary"
-            onClick={() =>
-              next().then(() => selfCreation && setAcceptaDretsImatge(true))
-            }
+            onClick={next}
           >
-            {stepsRef[currentPageIndex].key === "proteccio" ||
-            stepsRef[currentPageIndex].key === "imatge"
+            {stepsRef[currentPageIndex].key === "proteccio"
               ? "Ho he llegit i dono el meu consentiment"
               : "Següent"}
           </Button>

diff --git a/client/src/pages-home/inicia-sessio/components/avisos-inici/avisos-inici.js b/client/src/pages-home/inicia-sessio/components/avisos-inici/avisos-inici.js
@@ -37,7 +37,7 @@ export default () => {
             className="signin-logo"
             style={{ color: "var(--primary-color)" }}
           />
-          <AvisAcceptacio isForm idAvis={currentUser.avisos[0]} />
+          <AvisAcceptacio nameAvis={currentUser.avisos[0]} isForm />
           <div className="signin-footer-actions">
             <Space>
               <Button onClick={() => dispatch(logoutRemoveUser())}>

diff --git a/client/src/pages-tauler/perfil-soci/components/soci-tab-associacio/hooks/index.js b/client/src/pages-tauler/perfil-soci/components/soci-tab-associacio/hooks/index.js
@@ -1,2 +1 @@
 export { default as useAcceptacions } from "./use-acceptacions";
-export { default as useAcceptaDretsImatge } from "./use-accepta-drets-imatge";
diff --git a/...pages-tauler/perfil-soci/components/soci-tab-associacio/hooks/use-accepta-drets-imatge.js b/...pages-tauler/perfil-soci/components/soci-tab-associacio/hooks/use-accepta-drets-imatge.js
Original file line number	Diff line number	Diff line change
		@@ -1,2 +1 @@
		export { default as useAcceptacions } from "./use-acceptacions";
		export { default as useAcceptaDretsImatge } from "./use-accepta-drets-imatge";