From 6e939451395eb9d38bb17728388c34b7af9889bf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Diego=20Rodr=C3=ADguez?= <diego.rodriguez@rootstrap.com>
Date: Tue, 11 Feb 2025 14:45:27 -0300
Subject: [PATCH] Removed many comments

---
 .../src/app/layout/grupos/grupos.component.ts |  18 +-
 .../app/layout/matefun/matefun.component.ts   |  13 --
 .../manage-group-users-modal.component.html   |  38 ----
 .../manage-group-users-modal.component.ts     |   5 -
 .../request-feedback-modal.component.ts       |   2 -
 .../return-feedback-modal.component.ts        |   2 -
 .../shared/services/group-document.service.ts | 212 ------------------
 .../src/app/shared/services/group.service.ts  |  12 -
 8 files changed, 1 insertion(+), 301 deletions(-)

diff --git a/Frontend Angular 4/src/app/layout/grupos/grupos.component.ts b/Frontend Angular 4/src/app/layout/grupos/grupos.component.ts
index 0b0fe03..45e72b6 100755
--- a/Frontend Angular 4/src/app/layout/grupos/grupos.component.ts	
+++ b/Frontend Angular 4/src/app/layout/grupos/grupos.component.ts	
@@ -402,31 +402,15 @@ export class GruposComponent {
         this.directorioActual.id
       );
     }
-    // let selectedFile = this.selectedUserDirectorioActual?.archivos?.find(
-    //   (archivo) => archivo.documentId === this.selectedFile.id
-    // );
-    // if (!!selectedFile) {
-    //   this.seleccionarUser(this.selectedUser);
-    //   selectedFile.feedbackRequested = false;
-    // } else {
-    //   selectedFile = this.directorioActual.archivos.find(
-    //     (archivo) => archivo.documentId === this.selectedFile.id
-    //   );
-    //   if (!!selectedFile) {
-    //     selectedFile.feedbackRequested = false;
-    //   }
-    // }
   }
 
   seleccionarArchivo(archivo) {
     this.selectedFile = archivo;
     if (archivo.directorio) {
       this.directorioActual = archivo;
-      // this.loadFilesAndFolders(this.grupoSeleccionado.id, archivo.id);
       this.archivoSeleccionado = undefined;
       this.selectedFile = undefined; // Probando
       this.archivoSeleccionado = archivo;
-      // this.selectedUser = undefined;
       this.currentPath += `${archivo.nombre}/`;
       this.preview = "";
     } else {
@@ -460,7 +444,7 @@ export class GruposComponent {
       archivo.feedbackRequested = this.selectedFile.feedbackRequested;
     }
     this.archivoSeleccionado = archivo;
-    this.selectedFile = archivo; //Aquí el problema, se necesita para los que pidieron feedback, pero jode para los archivos propios
+    this.selectedFile = archivo;
     this.preview = !!archivo?.contenido ? archivo.contenido : "";
   }
 
diff --git a/Frontend Angular 4/src/app/layout/matefun/matefun.component.ts b/Frontend Angular 4/src/app/layout/matefun/matefun.component.ts
index dde9109..1906dd1 100755
--- a/Frontend Angular 4/src/app/layout/matefun/matefun.component.ts	
+++ b/Frontend Angular 4/src/app/layout/matefun/matefun.component.ts	
@@ -742,19 +742,6 @@ export class MateFunComponent implements OnInit, OnChanges {
     } else {
       this.configCodeMirror.mode.name = "matefun-ES";
     }
-
-    // this.fileService.getFiles().subscribe((files) => {
-    //   let matefun_files = files["files"];
-    //   matefun_files = this.fileService.completeFiles(matefun_files);
-    //   this.sessionService.setUserFiles(matefun_files);
-    //   let fileArchivos = this.fileService.fileToArchivo(matefun_files);
-
-    //   this.archivosTree = fileArchivos;
-    //   if (!!this.editor) {
-    //     (this.editor as any).options.files = this.archivosTree;
-    //   }
-    //   this.sessionService.setArchivosTree(this.archivosTree);
-    // });
   }
 
   importFilesData(data) {
diff --git a/Frontend Angular 4/src/app/shared/components/manage-group-users-modal/manage-group-users-modal.component.html b/Frontend Angular 4/src/app/shared/components/manage-group-users-modal/manage-group-users-modal.component.html
index 7608fa9..891497b 100644
--- a/Frontend Angular 4/src/app/shared/components/manage-group-users-modal/manage-group-users-modal.component.html	
+++ b/Frontend Angular 4/src/app/shared/components/manage-group-users-modal/manage-group-users-modal.component.html	
@@ -9,41 +9,3 @@
   [requiredRole]="'Owner'"
   [save]="save"
 ></app-add-users-modal>
-
-<!-- <div class="modal-header">
-  <span class="modal-title font-bold" id="modal-title">{{ modalTitle }}</span>
-  <button
-    aria-label="Cerrar diálogo"
-    (click)="modal.dismiss('Close share file modal')"
-    part="close-button"
-    class="close-button fa fa-close"
-    type="button"
-  ></button>
-</div>
-<div class="modal-body">
-  <div class="flex flex-col justify-between w-full">
-    <label for="new-users">New Users</label>
-    <div class="flex justify-between w-full items-center">
-      <app-chip-input
-        [(chips)]="chips"
-        [placeholder]="'Add a new user'"
-        [id]="'new-users'"
-        class="w-full"
-      ></app-chip-input>
-      <app-select
-        [options]="newUsersPermissionsEnum"
-        [(selectedOption)]="newUsersPermission"
-      ></app-select>
-    </div>
-  </div>
-  <div class="flex justify-end">
-    <button
-      (click)="confirmFileCreation(documentTitle, currentFolder.id)"
-      [disabled]="!documentTitle || !currentFolder"
-      class="btn btn-primary"
-      slot="primary"
-    >
-      Guardar
-    </button>
-  </div>
-</div> -->
diff --git a/Frontend Angular 4/src/app/shared/components/manage-group-users-modal/manage-group-users-modal.component.ts b/Frontend Angular 4/src/app/shared/components/manage-group-users-modal/manage-group-users-modal.component.ts
index 22303ff..a8dfef4 100644
--- a/Frontend Angular 4/src/app/shared/components/manage-group-users-modal/manage-group-users-modal.component.ts	
+++ b/Frontend Angular 4/src/app/shared/components/manage-group-users-modal/manage-group-users-modal.component.ts	
@@ -37,11 +37,6 @@ export class ManageGroupUsersModal implements OnInit {
    */
   @Input() confirmGroupUpdated: () => void;
 
-  // Hacer get del grupo al inicializar el modal
-  // Capaz mostrar un loader mientras ocurre
-  // Con eso mostrar los usuarios que tienen acceso al grupo y sus permisos
-  // Y usar el group_id para hacer el share
-
   chips: string[] = [];
 
   constructor(
diff --git a/Frontend Angular 4/src/app/shared/components/request-feedback-modal/request-feedback-modal.component.ts b/Frontend Angular 4/src/app/shared/components/request-feedback-modal/request-feedback-modal.component.ts
index ee43f09..968d5e1 100644
--- a/Frontend Angular 4/src/app/shared/components/request-feedback-modal/request-feedback-modal.component.ts	
+++ b/Frontend Angular 4/src/app/shared/components/request-feedback-modal/request-feedback-modal.component.ts	
@@ -15,8 +15,6 @@ import { GroupFileService } from "../../services/group-file.service";
   changeDetection: ChangeDetectionStrategy.OnPush,
 })
 export class RequestFeedbackModal implements OnInit {
-  // TODO: Hacer todo este modal basándome en lo que ya está que viene del DestroyGroupModal,
-  //       y luego terminar de hacer la llamada y todo en el GroupComponent
   @Input() fileName: string;
   @Input() groupName: string | null;
   @Input() groupId: number;
diff --git a/Frontend Angular 4/src/app/shared/components/return-feedback-modal/return-feedback-modal.component.ts b/Frontend Angular 4/src/app/shared/components/return-feedback-modal/return-feedback-modal.component.ts
index 550dbe3..422ebad 100644
--- a/Frontend Angular 4/src/app/shared/components/return-feedback-modal/return-feedback-modal.component.ts	
+++ b/Frontend Angular 4/src/app/shared/components/return-feedback-modal/return-feedback-modal.component.ts	
@@ -15,8 +15,6 @@ import { GroupFileService } from "../../services/group-file.service";
   changeDetection: ChangeDetectionStrategy.OnPush,
 })
 export class ReturnFeedbackModal implements OnInit {
-  // TODO: Hacer todo este modal basándome en lo que ya está que viene del DestroyGroupModal,
-  //       y luego terminar de hacer la llamada y todo en el GroupComponent
   @Input() fileName: string;
   @Input() groupName: string | null;
   @Input() groupId: number;
diff --git a/Frontend Angular 4/src/app/shared/services/group-document.service.ts b/Frontend Angular 4/src/app/shared/services/group-document.service.ts
index b1df8dd..a76638c 100644
--- a/Frontend Angular 4/src/app/shared/services/group-document.service.ts	
+++ b/Frontend Angular 4/src/app/shared/services/group-document.service.ts	
@@ -54,18 +54,6 @@ export class GroupDocumentService {
     });
   }
 
-  // getDocuments(ids: number[]): Observable<{ documents: MDocument[] }> {
-  //   const params = new HttpParams({
-  //     fromObject: { "document_ids[]": ids },
-  //   });
-  //   return this.http
-  //     .get<{ documents: MDocument[] }>(GET_DOCUMENTS, {
-  //       headers: this.getHeaders(),
-  //       params: params,
-  //     })
-  //     .pipe(catchError(this.handleError));
-  // }
-
   getGroupDocument(
     groupId: number,
     id: number,
@@ -89,206 +77,6 @@ export class GroupDocumentService {
       .pipe(catchError(this.handleError));
   }
 
-  // documentToArchivo(document: MDocument): Archivo {
-  //   return {
-  //     id: document.id,
-  //     documentId: document.id,
-  //     nombre: document.title,
-  //     contenido: document.text_doc,
-  //     fechaCreacion: document.created_at,
-  //     cedulaCreador: document.users[0]?.email,
-  //     editable: true,
-  //     padreId: document.parent_id ? document.parent_id : -1,
-  //     fileId: document.file_id,
-  //     archivoOrigenId: null,
-  //     archivos: [],
-  //     directorio: false,
-  //     estado: "activo",
-  //     eliminado: false,
-  //     evaluacion: null,
-  //     users: document.users,
-  //     role: document.role,
-  //   };
-  // }
-
-  // archivoToDocument(archivo: Archivo): MDocument {
-  //   return {
-  //     id: archivo.id,
-  //     title: archivo.nombre,
-  //     text_doc: archivo.contenido,
-  //     created_at: archivo.fechaCreacion,
-  //     parent_id: archivo.padreId,
-  //     role: archivo.role,
-  //     users: [],
-  //   };
-  // }
-
-  // createDocument(
-  //   document: MDocument,
-  //   groupId?: number
-  // ): Observable<HttpResponse<MDocument>> {
-  //   const params = { ...document };
-  //   if (groupId) {
-  //     params["group_id"] = groupId;
-  //   }
-  //   return this.http.post<MDocument>(
-  //     CREATE_DOCUMENT,
-  //     { document: params },
-  //     {
-  //       observe: "response",
-  //       headers: this.postHeaders(),
-  //     }
-  //   );
-  // }
-
-  // // Legacy method, need to check if it's still used
-  // getArchivos(cedula: string): Observable<Archivo[]> {
-  //   let headers = this.getHeaders();
-  //   let params: HttpParams = new HttpParams();
-  //   params = params.set("cedula", cedula);
-  //   let httpOptions = { headers: headers, params: params };
-
-  //   return this.http
-  //     .get<Archivo[]>(SERVER + "/servicios/archivo", httpOptions)
-  //     .pipe(catchError(this.handleError));
-  // }
-
-  // // Legacy method, need to check if it's still used
-  // getArchivosCompartidosAlumno(cedula: string): Observable<Archivo[]> {
-  //   let headers = this.getHeaders();
-  //   let params: HttpParams = new HttpParams();
-  //   params = params.set("cedula", cedula);
-  //   params = params.set("compartidos", "true");
-  //   let httpOptions = { headers: headers, params: params };
-
-  //   return this.http
-  //     .get<Archivo[]>(SERVER + "/servicios/archivo", httpOptions)
-  //     .pipe(catchError(this.handleError));
-  // }
-
-  // createFile(
-  //   file_title,
-  //   file_binary_doc_text
-  // ): Observable<HttpResponse<Object>> {
-  //   return this.http.post<Object>(
-  //     CREATE_DOCUMENT,
-  //     {
-  //       document: { title: file_title, text_doc: file_binary_doc_text },
-  //     },
-  //     {
-  //       observe: "response",
-  //       headers: this.postHeaders(),
-  //     }
-  //   );
-  // }
-
-  // updateDocument(
-  //   file_id,
-  //   file_title,
-  //   save_document_content = false
-  // ): Observable<HttpResponse<Object>> {
-  //   return this.http.patch<Object>(
-  //     `${UPDATE_DOCUMENT}/${file_id}`,
-  //     {
-  //       document: {
-  //         title: file_title,
-  //         save_document_content: save_document_content,
-  //       },
-  //     },
-  //     {
-  //       observe: "response",
-  //       headers: this.postHeaders(),
-  //     }
-  //   );
-  // }
-
-  // destroyAuxDocument(): Observable<HttpResponse<Object>> {
-  //   return this.http.delete<Object>(DESTROY_AUX_DOCUMENT, {
-  //     observe: "response",
-  //     headers: this.postHeaders(),
-  //   });
-  // }
-
-  // // Legacy method, need to check if it's still used
-  // eliminarArchivo(archivoId): Observable<Response> {
-  //   let headers = this.getHeaders();
-  //   let httpOptions = { headers: headers };
-
-  //   return this.http
-  //     .delete<Response>(SERVER + "/servicios/archivo/" + archivoId, httpOptions)
-  //     .pipe(catchError(this.handleError));
-  // }
-
-  // // Legacy method, need to check if it's still used
-  // getCopiaArchivoCompartidoGrupo(cedula, archivoId): Observable<Archivo> {
-  //   let headers = this.getHeaders();
-  //   let params: HttpParams = new HttpParams();
-  //   params = params.set("cedula", cedula);
-  //   let httpOptions = { headers: headers, params: params };
-
-  //   return this.http
-  //     .get<Archivo>(
-  //       SERVER + "/servicios/archivo/compartido/" + archivoId,
-  //       httpOptions
-  //     )
-  //     .pipe(catchError(this.handleError));
-  // }
-
-  // // Legacy method, need to check if it's still used
-  // compartirArchivoGrupo(grupo, archivoId): Observable<Archivo> {
-  //   let headers = this.getHeaders();
-  //   let httpOptions = { headers: headers };
-  //   var archId = {
-  //     id: archivoId,
-  //   };
-  //   return this.http
-  //     .post<Archivo>(
-  //       SERVER +
-  //         "/servicios/grupo/" +
-  //         grupo.liceoId +
-  //         "/" +
-  //         grupo.anio +
-  //         "/" +
-  //         grupo.grado +
-  //         "/" +
-  //         grupo.grupo +
-  //         "/archivo",
-  //       archId,
-  //       httpOptions
-  //     )
-  //     .pipe(catchError(this.handleError));
-  // }
-
-  // // Legacy method, need to check if it's still used
-  // calificarArchivo(archivoId, estado, evaluacion): Observable<Evaluacion> {
-  //   let headers = this.getHeaders();
-  //   let httpOptions = { headers: headers };
-  //   return this.http
-  //     .post<Evaluacion>(
-  //       SERVER +
-  //         "/servicios/archivo/" +
-  //         archivoId +
-  //         "/" +
-  //         estado +
-  //         "/evaluacion",
-  //       evaluacion,
-  //       httpOptions
-  //     )
-  //     .pipe(catchError(this.handleError));
-  // }
-
-  // // Legacy method, need to check if it's still used
-  // getGrupos(cedula: string): Observable<Group[]> {
-  //   let headers = this.getHeaders();
-  //   let params: HttpParams = new HttpParams();
-  //   params = params.set("cedula", cedula);
-  //   let httpOptions = { headers: headers, params: params };
-
-  //   return this.http
-  //     .get<Group[]>(SERVER + "/servicios/grupo", httpOptions)
-  //     .pipe(catchError(this.handleError));
-  // }
-
   /**
    * Handle HTTP error
    */
diff --git a/Frontend Angular 4/src/app/shared/services/group.service.ts b/Frontend Angular 4/src/app/shared/services/group.service.ts
index 8d42f40..94080d5 100644
--- a/Frontend Angular 4/src/app/shared/services/group.service.ts	
+++ b/Frontend Angular 4/src/app/shared/services/group.service.ts	
@@ -72,18 +72,6 @@ export class GroupService {
       .pipe(catchError(this.handleError));
   }
 
-  // getFilesList(options = {}): Observable<{ files: MFile[] }> {
-  //   const siblingsOfId = options["siblingsOfId"];
-  //   return this.http
-  //     .get<{ files: MFile[] }>(GET_FILES_LISTS, {
-  //       headers: this.getHeaders(),
-  //       params: siblingsOfId
-  //         ? new HttpParams().set("siblings_of_id", siblingsOfId)
-  //         : null,
-  //     })
-  //     .pipe(catchError(this.handleError));
-  // }
-
   fileToArchivo(file: MFile, parent: Archivo = null): Archivo {
     let archivo = new Archivo();
     archivo;
-- 
GitLab