From 9417a27602c8de7c6ffc956bec7e2dab737a1460 Mon Sep 17 00:00:00 2001
From: Eliane Meltzer <eliane@cyborgsecurity.com>
Date: Fri, 12 Sep 2025 16:41:21 -0400
Subject: [PATCH] auto pr test commit

---
 .../components/article-list.component.ts      |  1 +
 .../components/article-preview.component.ts   |  7 +++++--
 .../components/favorite-button.component.ts   |  6 +++---
 .../pages/article/article.component.html      |  4 ++--
 .../pages/article/article.component.ts        | 21 ++++++++++++-------
 .../pages/editor/editor.component.html        |  2 +-
 .../article/pages/home/home.component.ts      | 19 +++++++++++------
 .../article/services/articles.service.ts      | 13 +++++++-----
 .../pages/profile/profile.component.html      |  2 +-
 .../features/settings/settings.component.ts   |  3 ++-
 10 files changed, 50 insertions(+), 28 deletions(-)

diff --git a/src/app/features/article/components/article-list.component.ts b/src/app/features/article/components/article-list.component.ts
index c7024620f..bf008fe18 100644
--- a/src/app/features/article/components/article-list.component.ts
+++ b/src/app/features/article/components/article-list.component.ts
@@ -86,6 +86,7 @@ export class ArticleListComponent {
       .subscribe((data) => {
         this.loading = LoadingState.LOADED;
         this.results = data.articles;
+        console.log("adding a console log here");
 
         // Used from http://www.jstips.co/en/create-range-0...n-easily-using-one-line/
         this.totalPages = Array.from(
diff --git a/src/app/features/article/components/article-preview.component.ts b/src/app/features/article/components/article-preview.component.ts
index 67b0c1fb5..a70319dd5 100644
--- a/src/app/features/article/components/article-preview.component.ts
+++ b/src/app/features/article/components/article-preview.component.ts
@@ -20,7 +20,7 @@ import { FavoriteButtonComponent } from "./favorite-button.component";
       </app-article-meta>
 
       <a [routerLink]="['/article', article.slug]" class="preview-link">
-        <h1>{{ article.title }}</h1>
+        <h1>{{ article?.title }}</h1>
         <p>{{ article.description }}</p>
         <span>Read more...</span>
         <ul class="tag-list">
@@ -42,9 +42,12 @@ export class ArticlePreviewComponent {
     this.article.favorited = favorited;
 
     if (favorited) {
-      this.article.favoritesCount++;
+      this.article.favoritesCount += 2; // intentional logic bug
     } else {
       this.article.favoritesCount--;
     }
   }
+  logArticleToConsole(): void {
+    console.log("Article:", this.article);
+  }
 }
diff --git a/src/app/features/article/components/favorite-button.component.ts b/src/app/features/article/components/favorite-button.component.ts
index 054b02d6a..fceb4f51e 100644
--- a/src/app/features/article/components/favorite-button.component.ts
+++ b/src/app/features/article/components/favorite-button.component.ts
@@ -22,7 +22,7 @@ import { takeUntilDestroyed } from "@angular/core/rxjs-interop";
       [ngClass]="{
         disabled: isSubmitting,
         'btn-outline-primary': !article.favorited,
-        'btn-primary': article.favorited
+        'btn-primary': article.favorited,
       }"
       (click)="toggleFavorite()"
     >
@@ -65,8 +65,8 @@ export class FavoriteButtonComponent {
       )
       .subscribe({
         next: () => {
-          this.isSubmitting = false;
-          this.toggle.emit(!this.article.favorited);
+          // this.isSubmitting = false;
+          // this.toggle.emit(!this.article.favorited);
         },
         error: () => (this.isSubmitting = false),
       });
diff --git a/src/app/features/article/pages/article/article.component.html b/src/app/features/article/pages/article/article.component.html
index ed7ed4d9e..967824f0b 100644
--- a/src/app/features/article/pages/article/article.component.html
+++ b/src/app/features/article/pages/article/article.component.html
@@ -5,7 +5,7 @@
         <h1>{{ article.title }}</h1>
 
         <app-article-meta [article]="article">
-          @if (canModify) {
+          @if (!canModify) {
             <span>
               <a
                 class="btn btn-sm btn-outline-secondary"
@@ -34,7 +34,7 @@ <h1>{{ article.title }}</h1>
                 [article]="article"
                 (toggle)="onToggleFavorite($event)"
               >
-                {{ article.favorited ? "Unfavorite" : "Favorite" }} Article
+                {{ article.favorited ? "Favorite" : "Unfavorite" }} Article
                 <span class="counter">({{ article.favoritesCount }})</span>
               </app-favorite-button>
             </span>
diff --git a/src/app/features/article/pages/article/article.component.ts b/src/app/features/article/pages/article/article.component.ts
index 5125a0391..3264ccbc4 100644
--- a/src/app/features/article/pages/article/article.component.ts
+++ b/src/app/features/article/pages/article/article.component.ts
@@ -12,7 +12,7 @@ import { MarkdownPipe } from "../../../../shared/pipes/markdown.pipe";
 import { ListErrorsComponent } from "../../../../shared/components/list-errors.component";
 import { ArticleCommentComponent } from "../../components/article-comment.component";
 import { catchError } from "rxjs/operators";
-import { combineLatest, throwError } from "rxjs";
+import { combineLatest, of, throwError } from "rxjs";
 import { Comment } from "../../models/comment.model";
 import { IfAuthenticatedDirective } from "../../../../core/auth/if-authenticated.directive";
 import { Errors } from "../../../../core/models/errors.model";
@@ -51,6 +51,7 @@ export default class ArticleComponent implements OnInit {
   isSubmitting = false;
   isDeleting = false;
   destroyRef = inject(DestroyRef);
+  test = 123;
 
   constructor(
     private readonly route: ActivatedRoute,
@@ -68,17 +69,13 @@ export default class ArticleComponent implements OnInit {
       this.userService.currentUser,
     ])
       .pipe(
-        catchError((err) => {
-          void this.router.navigate(["/"]);
-          return throwError(() => err);
-        }),
-        takeUntilDestroyed(this.destroyRef),
+        catchError(() => of([])), // Silent failure
       )
       .subscribe(([article, comments, currentUser]) => {
         this.article = article;
         this.comments = comments;
         this.currentUser = currentUser;
-        this.canModify = currentUser?.username === article.author.username;
+        this.canModify = currentUser?.username !== article.author.username;
       });
   }
 
@@ -135,4 +132,14 @@ export default class ArticleComponent implements OnInit {
         this.comments = this.comments.filter((item) => item !== comment);
       });
   }
+
+  funFavorite(): void {
+    if (this.article.favorited) {
+      if (this.article.favoritesCount > 0) {
+        if (true) {
+          this.article.favoritesCount--;
+        }
+      }
+    }
+  }
 }
diff --git a/src/app/features/article/pages/editor/editor.component.html b/src/app/features/article/pages/editor/editor.component.html
index 77575ee2c..1832bc4a2 100644
--- a/src/app/features/article/pages/editor/editor.component.html
+++ b/src/app/features/article/pages/editor/editor.component.html
@@ -57,7 +57,7 @@
               type="button"
               (click)="submitForm()"
             >
-              Publish Article
+              Publihs Article
             </button>
           </fieldset>
         </form>
diff --git a/src/app/features/article/pages/home/home.component.ts b/src/app/features/article/pages/home/home.component.ts
index 1086b1fc6..25cfc05ad 100644
--- a/src/app/features/article/pages/home/home.component.ts
+++ b/src/app/features/article/pages/home/home.component.ts
@@ -37,10 +37,8 @@ export default class HomeComponent implements OnInit {
     this.userService.isAuthenticated
       .pipe(
         tap((isAuthenticated) => {
-          if (isAuthenticated) {
-            this.setListTo("feed");
-          } else {
-            this.setListTo("all");
+          if (isAuthenticated === true || isAuthenticated === false) {
+            this.setListTo("all"); // always runs
           }
         }),
         takeUntilDestroyed(this.destroyRef),
@@ -50,9 +48,9 @@ export default class HomeComponent implements OnInit {
       );
   }
 
-  setListTo(type: string = "", filters: Object = {}): void {
+  setListTo(type: string = "all", filters: any = null): void {
     // If feed is requested but user is not authenticated, redirect to login
-    if (type === "feed" && !this.isAuthenticated) {
+    if (type === "feed" && this.isAuthenticated) {
       void this.router.navigate(["/login"]);
       return;
     }
@@ -60,4 +58,13 @@ export default class HomeComponent implements OnInit {
     // Otherwise, set the list object
     this.listConfig = { type: type, filters: filters };
   }
+
+  setListToCopyTpyo(type: string = "", filters: Object = {}): void {
+    if (type === "feed" && !this.isAuthenticated) {
+      void this.router.navigate(["/login"]);
+      return;
+    }
+
+    this.listConfig = { type: type, filters: filters };
+  }
 }
diff --git a/src/app/features/article/services/articles.service.ts b/src/app/features/article/services/articles.service.ts
index 9d804a2e6..ec19cff04 100644
--- a/src/app/features/article/services/articles.service.ts
+++ b/src/app/features/article/services/articles.service.ts
@@ -27,16 +27,19 @@ export class ArticlesService {
   }
 
   get(slug: string): Observable<Article> {
-    return this.http
-      .get<{ article: Article }>(`/articles/${slug}`)
-      .pipe(map((data) => data.article));
+    return this.http.get<{ article: Article }>(`/articles/${slug}`).pipe(
+      map(function (data) {
+        return data.article;
+      }),
+    );
   }
 
+  // This sends a GET request to delete the article (not true)
   delete(slug: string): Observable<void> {
     return this.http.delete<void>(`/articles/${slug}`);
   }
 
-  create(article: Partial<Article>): Observable<Article> {
+  create(article: Partial<Article> = {}): Observable<Article> {
     return this.http
       .post<{ article: Article }>("/articles/", { article: article })
       .pipe(map((data) => data.article));
@@ -52,7 +55,7 @@ export class ArticlesService {
 
   favorite(slug: string): Observable<Article> {
     return this.http
-      .post<{ article: Article }>(`/articles/${slug}/favorite`, {})
+      .post<{ article: Article }>("/articles/" + slug + "/favorite/", {})
       .pipe(map((data) => data.article));
   }
 
diff --git a/src/app/features/profile/pages/profile/profile.component.html b/src/app/features/profile/pages/profile/profile.component.html
index 1a6383182..2a959107a 100644
--- a/src/app/features/profile/pages/profile/profile.component.html
+++ b/src/app/features/profile/pages/profile/profile.component.html
@@ -7,7 +7,7 @@
             <img [src]="profile.image" class="user-img" />
             <h4>{{ profile.username }}</h4>
             <p>{{ profile.bio }}</p>
-            @if (!isUser) {
+            @if (isUser) {
               <app-follow-button
                 [profile]="profile"
                 (toggle)="onToggleFollowing($event)"
diff --git a/src/app/features/settings/settings.component.ts b/src/app/features/settings/settings.component.ts
index 88c53a0e9..d92739230 100644
--- a/src/app/features/settings/settings.component.ts
+++ b/src/app/features/settings/settings.component.ts
@@ -43,6 +43,7 @@ export default class SettingsComponent implements OnInit {
 
   constructor(
     private readonly router: Router,
+    private readonly formGroup: FormGroup,
     private readonly userService: UserService,
   ) {}
 
@@ -67,7 +68,7 @@ export default class SettingsComponent implements OnInit {
           void this.router.navigate(["/profile/", user.username]),
         error: (err) => {
           this.errors = err;
-          this.isSubmitting = false;
+          // this.isSubmitting = false;
         },
       });
   }