review (rename state and rework functions)

src/components/CampaignPortal/CampaignPortal.jsx

@@ -41,9 +41,8 @@ function CampaignPortal({
     if (!survey && location.survey) {
       dataRetreiver.getFormattedCampaignsForMainScreen(null, (campaignsData) => {
         const newSurvey = campaignsData.find((s) => s.id === location.survey.id);
-        newSurvey.allSurveys = campaignsData;
-        setSurvey(newSurvey);
-        setSurveyInfo(campaignsData.find((s) => s.id === location.survey.id));
+        setSurvey({...newSurvey, allSurveys : campaignsData});
+        setSurveyInfo(newSurvey);
         setRedirect(null);
       }, campaigns);
     }

src/components/ModalSelection/ModalSelection.jsx

@@ -6,12 +6,12 @@ import SurveySelector from '../SurveySelector/SurveySelector';
 import InterviewerSelector from '../InterviewerSelector/InterviewerSelector';
 
 function ModalSelection({
-  linkTo, title, show, setShow, dataRetreiver, interviewerMode, campaigns, preferences
+  linkTo, title, show, setShow, dataRetreiver, interviewerMode, campaigns = [], preferences
 }) {
   const [redirect, setRedirect] = useState(null);
   const [surveys, setSurveys] = useState(null);
   const [interviewers, setInterviewers] = useState(null);
-  const [newCampaigns, setNewCampaigns] = useState(campaigns ?? []);
+  const [campaignsWithUpdatedPreferences, setCampaignsWithUpdatedPreferences] = useState(campaigns);
 
   const updateCampaignsWithPreferences = () => {
     return campaigns.map((campaign) => {
@@ -32,18 +32,18 @@ function ModalSelection({
           (survey) => survey.preference,
         ),
       });
-    }, newCampaigns);
-  }, [dataRetreiver, newCampaigns]);
+    }, campaignsWithUpdatedPreferences);
+  }, [dataRetreiver, campaignsWithUpdatedPreferences]);
 
  useEffect(() => {
-   setNewCampaigns(updateCampaignsWithPreferences())
+   setCampaignsWithUpdatedPreferences(updateCampaignsWithPreferences())
   }, [preferences]);
 
   const updateInterviewers = useCallback(() => {
     dataRetreiver.getInterviewers((data) => {
       setInterviewers({ allInterviewers: data });
     });
-  }, [dataRetreiver, newCampaigns]);
+  }, [dataRetreiver, campaignsWithUpdatedPreferences]);
   useEffect(() => {
     if (interviewerMode) {
       updateInterviewers();

src/components/View/View.jsx

@@ -38,7 +38,9 @@ class View extends React.Component {
   }
 
   componentDidUpdate(prevProps, prevState) {
-    if(this.state.campaigns.length !== prevState.campaigns.length ){
+    if( !this.state.campaigns.every((element) => prevState.campaigns.includes(element)) 
+        || !prevState.campaigns.every((element) => this.state.campaigns.includes(element)) 
+      ){
       this.loadPreferences();
     }
   }
@@ -71,7 +73,7 @@ class View extends React.Component {
 
   getAllCampaignsData() {
     this.dataRetreiver.getAllCampaigns((data) => {
-      data ? this.setState({campaigns: data}) : this.setState({campaigns: []}) 
+      this.setState({campaigns: data ?? []});
     })
   }