add optional chaining, remove conditional to fetch on component mount

src/shared/components/community/communities.tsx

@@ -79,10 +79,8 @@ export class Communities extends Component<any, CommunitiesState> {
   }
 
   async componentDidMount() {
-    if (!this.state.isIsomorphic) {
     await this.refetch();
   }
-  }
 
   get documentTitle(): string {
     return `${I18NextService.i18n.t("communities")} - ${
@@ -91,7 +89,7 @@ export class Communities extends Component<any, CommunitiesState> {
   }
 
   renderListings() {
-    switch (this.state.listCommunitiesResponse.state) {
+    switch (this.state.listCommunitiesResponse?.state) {
       case "loading":
         return (
           <h5>

src/shared/components/community/community.tsx

@@ -221,9 +221,7 @@ export class Community extends Component<
   }
 
   async componentDidMount() {
-    if (!this.state.isIsomorphic) {
     await Promise.all([this.fetchCommunity(), this.fetchData()]);
-    }
 
     setupTippy();
   }

src/shared/components/home/admin-settings.tsx

@@ -101,10 +101,8 @@ export class AdminSettings extends Component<any, AdminSettingsState> {
   }
 
   async componentDidMount() {
-    if (!this.state.isIsomorphic) {
     await this.fetchData();
   }
-  }
 
   get documentTitle(): string {
     return `${I18NextService.i18n.t("admin_settings")} - ${
@@ -114,7 +112,7 @@ export class AdminSettings extends Component<any, AdminSettingsState> {
 
   render() {
     const federationData =
-      this.state.instancesRes.state === "success"
+      this.state.instancesRes?.state === "success"
         ? this.state.instancesRes.data.federated_instances
         : undefined;
 
@@ -281,7 +279,7 @@ export class AdminSettings extends Component<any, AdminSettingsState> {
   }
 
   bannedUsers() {
-    switch (this.state.bannedRes.state) {
+    switch (this.state.bannedRes?.state) {
       case "loading":
         return (
           <h5>
@@ -289,7 +287,7 @@ export class AdminSettings extends Component<any, AdminSettingsState> {
           </h5>
         );
       case "success": {
-        const bans = this.state.bannedRes.data.banned;
+        const bans = this.state.bannedRes?.data.banned;
         return (
           <>
             <h5>{I18NextService.i18n.t("banned_users")}</h5>

src/shared/components/home/instances.tsx

@@ -45,10 +45,8 @@ export class Instances extends Component<any, InstancesState> {
   }
 
   async componentDidMount() {
-    if (!this.state.isIsomorphic) {
     await this.fetchInstances();
   }
-  }
 
   async fetchInstances() {
     this.setState({

src/shared/components/modlog.tsx

@@ -837,7 +837,7 @@ export class Modlog extends Component<
   }
 
   renderModlogTable() {
-    switch (this.state.res.state) {
+    switch (this.state.res?.state) {
       case "loading":
         return (
           <h5>

src/shared/components/person/inbox.tsx

@@ -177,10 +177,8 @@ export class Inbox extends Component<any, InboxState> {
   }
 
   async componentDidMount() {
-    if (!this.state.isIsomorphic) {
     await this.refetch();
   }
-  }
 
   get documentTitle(): string {
     const mui = UserService.Instance.myUserInfo;
@@ -195,11 +193,11 @@ export class Inbox extends Component<any, InboxState> {
     if (this.state.unreadOrAll == UnreadOrAll.Unread) {
       const { repliesRes, mentionsRes, messagesRes } = this.state;
       const replyCount =
-        repliesRes.state == "success" ? repliesRes.data.replies.length : 0;
+        repliesRes?.state == "success" ? repliesRes.data.replies.length : 0;
       const mentionCount =
-        mentionsRes.state == "success" ? mentionsRes.data.mentions.length : 0;
+        mentionsRes?.state == "success" ? mentionsRes.data.mentions.length : 0;
       const messageCount =
-        messagesRes.state == "success"
+        messagesRes?.state == "success"
           ? messagesRes.data.private_messages.length
           : 0;
 

src/shared/components/person/profile.tsx

@@ -10,7 +10,6 @@ import {
   setIsoData,
   updatePersonBlock,
 } from "@utils/app";
-import { restoreScrollPosition, saveScrollPosition } from "@utils/browser";
 import {
   capitalizeFirstLetter,
   futureDaysToUnixTime,
@@ -216,16 +215,10 @@ export class Profile extends Component<
   }
 
   async componentDidMount() {
-    if (!this.state.isIsomorphic) {
     await this.fetchUserData();
-    }
     setupTippy();
   }
 
-  componentWillUnmount() {
-    saveScrollPosition(this.context);
-  }
-
   async fetchUserData() {
     const { page, sort, view } = getProfileQueryParams();
 
@@ -240,12 +233,12 @@ export class Profile extends Component<
         auth: myAuth(),
       }),
     });
-    restoreScrollPosition(this.context);
+
     this.setPersonBlock();
   }
 
   get amCurrentUser() {
-    if (this.state.personRes.state === "success") {
+    if (this.state.personRes?.state === "success") {
       return (
         UserService.Instance.myUserInfo?.local_user_view.person.id ===
         this.state.personRes.data.person_view.person.id
@@ -302,7 +295,7 @@ export class Profile extends Component<
   }
 
   renderPersonRes() {
-    switch (this.state.personRes.state) {
+    switch (this.state.personRes?.state) {
       case "loading":
         return (
           <h5>

src/shared/components/person/registration-applications.tsx

@@ -68,9 +68,7 @@ export class RegistrationApplications extends Component<
   }
 
   async componentDidMount() {
-    if (!this.state.isIsomorphic) {
     await this.refetch();
-    }
     setupTippy();
   }
 
@@ -84,7 +82,7 @@ export class RegistrationApplications extends Component<
   }
 
   renderApps() {
-    switch (this.state.appsRes.state) {
+    switch (this.state.appsRes?.state) {
       case "loading":
         return (
           <h5>
@@ -92,7 +90,7 @@ export class RegistrationApplications extends Component<
           </h5>
         );
       case "success": {
-        const apps = this.state.appsRes.data.registration_applications;
+        const apps = this.state.appsRes?.data.registration_applications;
         return (
           <div className="row">
             <div className="col-12">

src/shared/components/person/reports.tsx

@@ -129,10 +129,8 @@ export class Reports extends Component<any, ReportsState> {
   }
 
   async componentDidMount() {
-    if (!this.state.isIsomorphic) {
     await this.refetch();
   }
-  }
 
   get documentTitle(): string {
     const mui = UserService.Instance.myUserInfo;
@@ -329,6 +327,11 @@ export class Reports extends Component<any, ReportsState> {
 
   get buildCombined(): ItemType[] {
     const commentRes = this.state.commentReportsRes;
+
+    if (!commentRes) {
+      return [];
+    }
+
     const comments =
       commentRes.state == "success"
         ? commentRes.data.comment_reports.map(this.commentReportToItemType)

src/shared/components/person/settings.tsx

@@ -228,8 +228,8 @@ export class Settings extends Component<any, SettingsState> {
   }
 
   async componentDidMount() {
-    setupTippy();
     this.setState({ themeList: await fetchThemeList() });
+    setupTippy();
   }
 
   get documentTitle(): string {

src/shared/components/post/create-post.tsx

@@ -103,7 +103,7 @@ export class CreatePost extends Component<
         id: communityId,
         auth,
       });
-      if (res.state === "success") {
+      if (res?.state === "success") {
         this.setState({
           selectedCommunityChoice: {
             label: res.data.community_view.community.title,
@@ -116,8 +116,6 @@ export class CreatePost extends Component<
   }
 
   async componentDidMount() {
-    // TODO test this
-    if (!this.state.isIsomorphic) {
     const { communityId } = getCreatePostQueryParams();
 
     const initialCommunitiesRes = await fetchCommunitiesForOptions(
@@ -128,9 +126,7 @@ export class CreatePost extends Component<
       initialCommunitiesRes,
     });
 
-      if (
+    if (communityId?.toString() !== this.state.selectedCommunityChoice?.value) {
-        communityId?.toString() !== this.state.selectedCommunityChoice?.value
-      ) {
       await this.fetchCommunity();
     } else if (!communityId) {
       this.setState({
@@ -139,7 +135,6 @@ export class CreatePost extends Component<
       });
     }
   }
-  }
 
   get documentTitle(): string {
     return `${I18NextService.i18n.t("create_post")} - ${
@@ -181,8 +176,8 @@ export class CreatePost extends Component<
                 selectedCommunityChoice={selectedCommunityChoice}
                 onSelectCommunity={this.handleSelectedCommunityChange}
                 initialCommunities={
-                  this.state.initialCommunitiesRes.state === "success"
+                  this.state.initialCommunitiesRes?.state === "success"
-                    ? this.state.initialCommunitiesRes.data.communities
+                    ? this.state.initialCommunitiesRes?.data.communities
                     : []
                 }
               />
@@ -224,7 +219,7 @@ export class CreatePost extends Component<
   async handlePostCreate(form: CreatePostI) {
     const res = await HttpService.client.createPost(form);
 
-    if (res.state === "success") {
+    if (res?.state === "success") {
       const postId = res.data.post_view.post.id;
       this.props.history.replace(`/post/${postId}`);
     } else {

src/shared/components/post/post.tsx

@@ -270,9 +270,7 @@ export class Post extends Component<any, PostState> {
   }
 
   async componentDidMount() {
-    if (!this.state.isIsomorphic) {
     await this.fetchPost();
-    }
 
     autosize(document.querySelectorAll("textarea"));
 

src/shared/components/private_message/create-private-message.tsx

@@ -54,10 +54,8 @@ export class CreatePrivateMessage extends Component<
   }
 
   async componentDidMount() {
-    if (!this.state.isIsomorphic) {
     await this.fetchPersonDetails();
   }
-  }
 
   static async fetchInitialData({
     client,

src/shared/components/search.tsx

@@ -332,7 +332,6 @@ export class Search extends Component<any, SearchState> {
   }
 
   async componentDidMount() {
-    if (!this.state.isIsomorphic) {
     const promises = [this.fetchCommunities()];
     if (this.state.searchText) {
       promises.push(this.search());
@@ -340,7 +339,6 @@ export class Search extends Component<any, SearchState> {
 
     await Promise.all(promises);
   }
-  }
 
   async fetchCommunities() {
     this.setState({ communitiesRes: { state: "loading" } });