Update private/admin/admin.js

private/admin/admin.js

@@ -2103,30 +2103,23 @@ class AdminPanel {
         return `<span class="math-inline math-fallback">${this.escapeHtmlDisplay(fallback)}</span>`;
     }
 
-    /**
-     * Parse markdown tables into HTML
-     * Optimized for performance with array building
-     * @private
-     */
     _parseMarkdownTables(content, placeholders) {
         if (!content) return content;
         
         const lines = content.split('\n');
         const result = [];
-        const lineCount = lines.length;
+        let i = 0;
         
-        for (let i = 0; i < lineCount; i++) {
+        while (i < lines.length) {
             const line = lines[i];
-            const trimmed = line.trim();
             
-            if (trimmed.startsWith('|') && trimmed.endsWith('|')) {
+            if (line.trim().startsWith('|') && line.trim().endsWith('|')) {
                 const nextLine = lines[i + 1];
-                
                 if (nextLine && /^\|[\s:|-]+\|$/.test(nextLine.trim())) {
                     const tableLines = [line];
                     let j = i + 1;
                     
-                    while (j < lineCount && lines[j].trim().startsWith('|')) {
+                    while (j < lines.length && lines[j].trim().startsWith('|')) {
                         tableLines.push(lines[j]);
                         j++;
                     }
@@ -2136,76 +2129,52 @@ class AdminPanel {
                         const placeholder = `__TABLE_BLOCK_${placeholders.length}__`;
                         placeholders.push(tableHtml);
                         result.push(placeholder);
-                        i = j - 1; // -1 because loop will increment
+                        i = j;
                         continue;
                     }
                 }
             }
             
             result.push(line);
+            i++;
         }
         
         return result.join('\n');
     }
 
-    /**
-     * Convert table lines to HTML
-     * Optimized with array building instead of string concatenation
-     * @private
-     */
     _convertTableToHtml(lines) {
         if (lines.length < 2) return null;
         
-        // Extract cells (skip first and last empty cells from split)
-        const extractCells = (line) => {
-            const parts = line.split('|');
-            return parts.slice(1, -1).map(c => c.trim());
-        };
-        
-        const headerCells = extractCells(lines[0].trim());
-        if (headerCells.length === 0) return null;
-        
-        const headerCount = headerCells.length;
+        const headerLine = lines[0].trim();
+        const headerCells = headerLine.split('|').filter(c => c.trim()).map(c => c.trim());
         
-        // Parse alignments from separator
-        const sepCells = extractCells(lines[1].trim());
-        const aligns = sepCells.map(cell => {
+        const separatorLine = lines[1].trim();
+        const aligns = separatorLine.split('|').filter(c => c.trim()).map(c => {
+            const cell = c.trim();
             if (cell.startsWith(':') && cell.endsWith(':')) return 'center';
             if (cell.endsWith(':')) return 'right';
             return 'left';
         });
         
-        // Build HTML using array
-        const htmlParts = ['<div class="table-wrapper"><table><thead><tr>'];
-        
-        // Header row
-        for (let i = 0; i < headerCount; i++) {
-            const align = aligns[i] || 'left';
-            const content = this._formatInlineContent(headerCells[i]);
-            htmlParts.push(`<th style="text-align:${align}">${content}</th>`);
-        }
-        
-        htmlParts.push('</tr></thead><tbody>');
+        let html = '<div class="table-wrapper"><table><thead><tr>';
+        headerCells.forEach((h, i) => {
+            html += `<th style="text-align:${aligns[i] || 'left'}">${this._formatInlineContent(h)}</th>`;
+        });
+        html += '</tr></thead><tbody>';
         
-        // Data rows (skip header and separator)
         for (let i = 2; i < lines.length; i++) {
-            const cells = extractCells(lines[i]);
-            if (cells.length === 0) continue;
-            
-            htmlParts.push('<tr>');
-            
-            for (let j = 0; j < headerCount; j++) {
-                const align = aligns[j] || 'left';
-                const cellContent = cells[j] || '';
-                const content = this._formatInlineContent(cellContent);
-                htmlParts.push(`<td style="text-align:${align}">${content}</td>`);
-            }
-            
-            htmlParts.push('</tr>');
+            const cells = lines[i].split('|').filter(c => c !== '').map(c => c.trim());
+            html += '<tr>';
+            cells.forEach((c, j) => {
+                if (c !== undefined) {
+                    html += `<td style="text-align:${aligns[j] || 'left'}">${this._formatInlineContent(c)}</td>`;
+                }
+            });
+            html += '</tr>';
         }
         
-        htmlParts.push('</tbody></table></div>');
-        return htmlParts.join('');
+        html += '</tbody></table></div>';
+        return html;
     }
 
     _initCodeCopyButtons() {
@@ -3155,110 +3124,64 @@ class AdminPanel {
         return `<span style="padding:2px 6px;background:rgba(102, 126, 234, 0.1);border-radius:4px;font-family:'Times New Roman',Georgia,serif;font-style:italic;">${this.escapeHtml(rendered)}</span>`;
     }
 
-    /**
-     * Parse markdown tables for PDF
-     * Optimized with array building and cleaner logic
-     * @private
-     */
     _parseMarkdownTablesForPdf(content) {
         if (!content) return content;
-        
         const lines = content.split('\n');
         const result = [];
-        const lineCount = lines.length;
-        const separatorRegex = /^\|[\s:|-]+\|$/;
+        let i = 0;
         
-        for (let i = 0; i < lineCount; i++) {
+        while (i < lines.length) {
             const line = lines[i];
-            const trimmed = line.trim();
-            
-            if (trimmed.startsWith('|') && trimmed.endsWith('|')) {
+            if (line.trim().startsWith('|') && line.trim().endsWith('|')) {
                 const nextLine = lines[i + 1];
-                
-                if (nextLine && separatorRegex.test(nextLine.trim())) {
+                if (nextLine && /^\|[\s:|-]+\|$/.test(nextLine.trim())) {
                     const tableLines = [line];
                     let j = i + 1;
-                    
-                    while (j < lineCount && lines[j].trim().startsWith('|')) {
+                    while (j < lines.length && lines[j].trim().startsWith('|')) {
                         tableLines.push(lines[j]);
                         j++;
                     }
-                    
                     result.push(this._convertTableToHtmlForPdf(tableLines));
-                    i = j - 1; // -1 because loop will increment
+                    i = j;
                     continue;
                 }
             }
-            
             result.push(line);
+            i++;
         }
-        
         return result.join('\n');
     }
 
-    /**
-     * Convert table lines to HTML for PDF
-     * Optimized with array building instead of string concatenation
-     * @private
-     */
     _convertTableToHtmlForPdf(lines) {
         if (lines.length < 2) return lines.join('\n');
         
-        // Extract cells (skip first and last empty cells from split)
-        const extractCells = (line) => {
-            const parts = line.split('|');
-            return parts.slice(1, -1).map(c => c.trim());
-        };
-        
-        // Parse alignments from separator
-        const sepCells = extractCells(lines[1]);
-        const aligns = sepCells.map(cell => {
-            if (cell.startsWith(':') && cell.endsWith(':')) return 'center';
-            if (cell.endsWith(':')) return 'right';
+        const sep = lines[1];
+        const aligns = sep.split('|').filter(c => c.trim()).map(c => {
+            const t = c.trim();
+            if (t.startsWith(':') && t.endsWith(':')) return 'center';
+            if (t.endsWith(':')) return 'right';
             return 'left';
         });
         
-        // Parse header
-        const headerCells = extractCells(lines[0]);
-        const headerCount = headerCells.length;
-        
-        // Build HTML using array
-        const htmlParts = [
-            '<table style="width:100%;border-collapse:collapse;margin:20px 0;border-radius:12px;overflow:hidden;box-shadow:0 4px 12px rgba(0,0,0,0.1);">',
-            '<thead><tr style="background:linear-gradient(135deg, #667eea 0%, #764ba2 100%);">'
-        ];
-        
-        // Header row
-        for (let i = 0; i < headerCount; i++) {
-            const align = aligns[i] || 'left';
-            const content = this._formatInlineContentForPdf(headerCells[i]);
-            htmlParts.push(
-                `<th style="text-align:${align};padding:14px 16px;color:#ffffff;font-weight:600;font-size:12px;text-transform:uppercase;letter-spacing:0.5px;border:none;">${content}</th>`
-            );
-        }
-        
-        htmlParts.push('</tr></thead><tbody>');
-        
-        // Data rows (skip header and separator)
+        const headerCells = lines[0].split('|').filter(c => c.trim()).map(c => c.trim());
+        let html = '<table style="width:100%;border-collapse:collapse;margin:20px 0;border-radius:12px;overflow:hidden;box-shadow:0 4px 12px rgba(0,0,0,0.1);"><thead><tr style="background:linear-gradient(135deg, #667eea 0%, #764ba2 100%);">';
+        headerCells.forEach((h, i) => {
+            html += `<th style="text-align:${aligns[i] || 'left'};padding:14px 16px;color:#ffffff;font-weight:600;font-size:12px;text-transform:uppercase;letter-spacing:0.5px;border:none;">${this._formatInlineContentForPdf(h)}</th>`;
+        });
+        html += '</tr></thead><tbody>';
         for (let i = 2; i < lines.length; i++) {
-            const cells = extractCells(lines[i]);
+            const cells = lines[i].match(/\|([^|]*)/g)?.map(c => c.slice(1).trim()) || [];
             const rowBg = (i - 2) % 2 === 0 ? '#ffffff' : '#f8fafc';
-            
-            htmlParts.push(`<tr style="background:${rowBg};">`);
-            
-            for (let j = 0; j < headerCount && j < cells.length; j++) {
-                const align = aligns[j] || 'left';
-                const content = this._formatInlineContentForPdf(cells[j]);
-                htmlParts.push(
-                    `<td style="text-align:${align};padding:14px 16px;border-bottom:1px solid #e2e8f0;color:#2d3748;font-size:14px;">${content}</td>`
-                );
-            }
-            
-            htmlParts.push('</tr>');
+            html += `<tr style="background:${rowBg};">`;
+            cells.forEach((c, j) => {
+                if (c !== undefined) {
+                    html += `<td style="text-align:${aligns[j] || 'left'};padding:14px 16px;border-bottom:1px solid #e2e8f0;color:#2d3748;font-size:14px;">${this._formatInlineContentForPdf(c)}</td>`;
+                }
+            });
+            html += '</tr>';
         }
-        
-        htmlParts.push('</tbody></table>');
-        return htmlParts.join('');
+        html += '</tbody></table>';
+        return html;
     }
 
     // Format inline content for PDF (bold, italic, code, underline)