Update server.js

server.js

@@ -448,125 +448,15 @@ const userSchema = new mongoose.Schema({
     otpExpires: Date,
     refreshTokens: [{ token: String, createdAt: { type: Date, default: Date.now } }],
     notifications: [{ type: String }],
-    profile: {
-        nickname: { type: String, sparse: true },
-        avatar: String,
-        status: { type: String, default: 'Available', enum: ['Available', 'Busy', 'Open to Work'] },
-        jobTitle: String,
-        pdfFormat: { type: String, enum: ['jspdf', 'canva', 'template1', 'template2'], default: 'jspdf' },
-        bio: String,
-        phone: { type: String, default: '' },
-        socialLinks: {
-            linkedin: { type: String, default: '' },
-            behance: { type: String, default: '' },
-            github: { type: String, default: '' },
-            whatsapp: { type: String, default: '' }
-        },
-        education: [{ institution: String, degree: String, year: String }],
-        experience: [{ company: String, role: String, duration: String }],
-        certificates: [{ name: String, issuer: String, year: String }],
-        skills: [{ name: String, percentage: Number }],
-        projects: [
-            {
-                isPrivate: { type: Boolean, default: false },
-                title: String,
-                description: String,
-                image: String,
-                rating: String,
-                stars: { type: Number, min: 0, max: 5 },
-                isPublic: { type: Boolean, default: true },
-                links: [{ option: String, value: String }]
-            }
-        ],
-        githubRepos: [
-            {
-                id: String,
-                name: String,
-                description: String,
-                url: String,
-                image: String
-            }
-        ],
-
-        theme: {
-            id: { type: String, default: 'default' },
-            primaryColor: { type: String, default: '#3b82f6' },
-            secondaryColor: { type: String, default: '#8b5cf6' },
-            fontFamily: { type: String, default: 'Inter' },
-            borderRadius: { type: String, default: '0.5rem' },
-        },
-
-        // ✅ إضافة إعدادات التخطيط (Layout)
-        layout: {
-            type: { type: String, enum: ['grid', 'list', 'masonry'], default: 'grid' },
-            columns: { type: Number, default: 3 },
-            showProjectImages: { type: Boolean, default: true },
-            showProjectDescriptions: { type: Boolean, default: true },
-            showProjectRatings: { type: Boolean, default: true },
-            showProjectLinks: { type: Boolean, default: true },
-        },
-
-        // ✅ إضافة إعدادات الهيدر
-        header: {
-            showAvatar: { type: Boolean, default: true },
-            showJobTitle: { type: Boolean, default: true },
-            showBio: { type: Boolean, default: true },
-            showContactInfo: { type: Boolean, default: true },
-            showSocialLinks: { type: Boolean, default: true },
-            layout: { type: String, enum: ['centered', 'left-aligned'], default: 'centered' },
-        },
-
-        // ✅ إضافة إعدادات الفوتر
-        footer: {
-            showCopyright: { type: Boolean, default: true },
-            customText: { type: String, default: '' },
-        },
-
-        // ✅ إضافة إعدادات SEO
-        seo: {
-            title: { type: String, default: '' },
-            description: { type: String, default: '' },
-            keywords: { type: String, default: '' },
-            ogImage: { type: String, default: '' },
-            ogTitle: { type: String, default: '' },
-            ogDescription: { type: String, default: '' },
-            twitterCard: { type: String, enum: ['summary', 'summary_large_image', 'app', 'player'], default: 'summary_large_image' },
-            twitterSite: { type: String, default: '' },
-            canonicalUrl: { type: String, default: '' },
-            noindex: { type: Boolean, default: false },
-            nofollow: { type: Boolean, default: false },
-        },
-
-        // ✅ إضافة إعدادات Schema.org
-        schema: {
-            type: { type: String, enum: ['Person', 'Organization', 'ProfessionalService', 'LocalBusiness'], default: 'Person' },
-            name: { type: String, default: '' },
-            description: { type: String, default: '' },
-            image: { type: String, default: '' },
-            sameAs: [{ type: String }],
-            jobTitle: { type: String, default: '' },
-            worksFor: { type: String, default: '' },
-            alumniOf: [{ type: String }],
-            knowsAbout: [{ type: String }],
-        },
-        //     canvaAccessToken: String,
-        // canvaRefreshToken: String,
-
-
-
-        customFields: [{ name: String, value: String }],
-        interests: [String],
-        isPublic: { type: Boolean, default: true },
-        avatarDisplayType: { type: String, enum: ['svg', 'normal'], default: 'normal' },
-        svgColor: { type: String, default: '#000000' },
-        portfolioName: { type: String, default: 'Portfolio' },
-        pushNotifications: { type: Boolean, default: false }
+   profile: {
+        type: mongoose.Schema.Types.Mixed,
+        default: {}
     }
     },
     {
     minimize: false,
-    toJSON: { getters: false, virtuals: false },
-    toObject: { getters: false, virtuals: false }
+    // toJSON: { getters: false, virtuals: false },
+    // toObject: { getters: false, virtuals: false }
 
 });
 
@@ -1512,15 +1402,31 @@ async function authenticateToken(req, res, next) {
 
 
 app.get('/api/verify-token', authenticateToken, async (req, res) => {
-    const user = await User.findById(req.user.userId);
-    if (!user) return res.status(404).json({ error: 'User not found' });
-    res.json({
-        valid: true,
-        userId: req.user.userId,
-        isAdmin: req.user.isAdmin,
-        username: user.username,
-        profile: user.profile
-    });
+    try {
+        // ✅ استخدم lean() وحدد الحقول اللي عايزها بس
+        const user = await User.findById(req.user.userId)
+            .select('username profile.nickname profile.portfolioName isAdmin')
+            .lean()  // ⭐ هذا هو الحل السحري
+            .exec();
+        
+        if (!user) {
+            return res.status(404).json({ error: 'User not found' });
+        }
+        
+        res.json({
+            valid: true,
+            userId: req.user.userId,
+            isAdmin: req.user.isAdmin || user.isAdmin,
+            username: user.username,
+            profile: {
+                nickname: user.profile?.nickname,
+                portfolioName: user.profile?.portfolioName || 'Portfolio'
+            }
+        });
+    } catch (error) {
+        logger.error(`Error in verify-token: ${error.message}`);
+        res.status(500).json({ error: 'Internal server error' });
+    }
 });
 
 app.post('/api/logout', authenticateToken, async (req, res) => {
@@ -2341,7 +2247,7 @@ app.delete('/api/skills/:skillId', authenticateToken, isAdmin, async (req, res)
 // تحديث endpoint /api/profile/me
 app.get('/api/profile/me', authenticateToken, async (req, res) => {
     try {
-        // ✅ استخدم lean() لتجنب مشاكل Mongoose getters
+        // ✅ استخدم lean()
         const user = await User.findById(req.user.userId)
             .select('username profile')
             .lean();
@@ -2350,39 +2256,34 @@ app.get('/api/profile/me', authenticateToken, async (req, res) => {
             return res.status(404).json({ error: 'المستخدم غير موجود' });
         }
 
-        // ✅ تأكد من وجود profile
-        if (!user.profile) {
-            user.profile = {};
-        }
-
-        // ✅ القيم الافتراضية للـ profile
-        const profile = {
-            portfolioName: user.profile.portfolioName || 'Portfolio',
-            nickname: user.profile.nickname || '',
-            jobTitle: user.profile.jobTitle || '',
-            bio: user.profile.bio || '',
-            phone: user.profile.phone || '',
-            isPublic: user.profile.isPublic ?? true,
-            status: user.profile.status || 'Available',
-            socialLinks: user.profile.socialLinks || { linkedin: '', behance: '', github: '', whatsapp: '' },
-            avatar: user.profile.avatar || '',
-            avatarDisplayType: user.profile.avatarDisplayType || 'normal',
-            svgColor: user.profile.svgColor || '#000000',
-            pdfFormat: user.profile.pdfFormat || 'jspdf',
-            education: user.profile.education || [],
-            experience: user.profile.experience || [],
-            certificates: user.profile.certificates || [],
-            skills: user.profile.skills || [],
-            projects: user.profile.projects || [],
-            interests: user.profile.interests || [],
-            theme: user.profile.theme || {
+        // ✅ القيم الافتراضية للـ profile (نسخة نظيفة)
+        const defaultProfile = {
+            portfolioName: 'Portfolio',
+            nickname: '',
+            jobTitle: '',
+            bio: '',
+            phone: '',
+            isPublic: true,
+            status: 'Available',
+            socialLinks: { linkedin: '', behance: '', github: '', whatsapp: '' },
+            avatar: '',
+            avatarDisplayType: 'normal',
+            svgColor: '#000000',
+            pdfFormat: 'jspdf',
+            education: [],
+            experience: [],
+            certificates: [],
+            skills: [],
+            projects: [],
+            interests: [],
+            theme: {
                 id: 'default',
                 primaryColor: '#3b82f6',
                 secondaryColor: '#8b5cf6',
                 fontFamily: 'Inter',
                 borderRadius: '0.5rem',
             },
-            layout: user.profile.layout || {
+            layout: {
                 type: 'grid',
                 columns: 3,
                 showProjectImages: true,
@@ -2390,7 +2291,7 @@ app.get('/api/profile/me', authenticateToken, async (req, res) => {
                 showProjectRatings: true,
                 showProjectLinks: true,
             },
-            header: user.profile.header || {
+            header: {
                 showAvatar: true,
                 showJobTitle: true,
                 showBio: true,
@@ -2398,11 +2299,11 @@ app.get('/api/profile/me', authenticateToken, async (req, res) => {
                 showSocialLinks: true,
                 layout: 'centered',
             },
-            footer: user.profile.footer || {
+            footer: {
                 showCopyright: true,
                 customText: '',
             },
-            seo: user.profile.seo || {
+            seo: {
                 title: '',
                 description: '',
                 keywords: '',
@@ -2415,7 +2316,7 @@ app.get('/api/profile/me', authenticateToken, async (req, res) => {
                 noindex: false,
                 nofollow: false,
             },
-            schema: user.profile.schema || {
+            schema: {
                 type: 'Person',
                 name: '',
                 description: '',
@@ -2426,15 +2327,19 @@ app.get('/api/profile/me', authenticateToken, async (req, res) => {
                 alumniOf: [],
                 knowsAbout: [],
             },
-            pushNotifications: user.profile.pushNotifications || false
+            pushNotifications: false
         };
 
+        // ✅ دمج default مع اللي موجود
+        const profile = { ...defaultProfile, ...(user.profile || {}) };
+
         res.json({
             username: user.username,
             profile: profile
         });
+        
     } catch (error) {
-        logger.error(`Error fetching profile: ${error.message}`);
+        logger.error(`Error fetching profile/me: ${error.message}`);
         res.status(500).json({ error: 'خطأ في استرجاع الملف الشخصي' });
     }
 });
@@ -2477,21 +2382,28 @@ app.get('/api/profile/:nickname', async (req, res) => {
     try {
         const decodedNickname = decodeURIComponent(req.params.nickname);
 
-        // 1. جلب المستخدم مع البروفايل
+        // 1. جلب المستخدم مع البروفايل - ✅ أضفنا .lean()
         const user = await User.findOne({
             $or: [
                 { 'profile.nickname': { $regex: `^${decodedNickname}$`, $options: 'i' } },
                 { username: { $regex: `^${decodedNickname}$`, $options: 'i' } },
             ],
-        }).select('username profile notifications');
+        })
+        .select('username profile notifications')
+        .lean();  // ⭐ هذا هو المطلوب
 
         if (!user) {
             logger.warn(`Profile not found for nickname: ${decodedNickname}`);
             return res.status(404).json({ error: `Profile not found for ${decodedNickname}` });
         }
 
-        // 2. التحقق من الخصوصية
-        const isOwner = req.user && req.user.userId === user._id.toString();
+        // 2. التحقق من الخصوصية - ✅ user._id بقى string
+        const isOwner = req.user && req.user.userId === user._id;  // مش محتاج toString()
+
+        // ✅ التحقق من وجود profile
+        if (!user.profile) {
+            user.profile = {};
+        }
 
         if (!user.profile.isPublic && !isOwner) {
             logger.warn(`Unauthorized access attempt to private profile: ${decodedNickname}`);
@@ -2501,7 +2413,7 @@ app.get('/api/profile/:nickname', async (req, res) => {
         // 3. جلب المشاريع من Project collection
         const projectsQuery = { userId: user._id };
         if (!isOwner) {
-            projectsQuery.isPublic = true; // غير المالك يشوف العامة فقط
+            projectsQuery.isPublic = true;
         }
 
         const projects = await Project.find(projectsQuery)
@@ -2509,11 +2421,11 @@ app.get('/api/profile/:nickname', async (req, res) => {
             .sort({ createdAt: -1 })
             .lean();
 
-        // 4. تجهيز الرد مع كل البيانات بما في ذلك المظهر و SEO
+        // 4. تجهيز الرد مع default values
         const response = {
             username: user.username,
             profile: {
-                // المعلومات الأساسية (موجودة)
+                // المعلومات الأساسية
                 nickname: user.profile.nickname || user.username,
                 portfolioName: user.profile.portfolioName || 'Portfolio',
                 avatar: user.profile.avatar || '/assets/img/default-avatar.png',
@@ -2544,7 +2456,7 @@ app.get('/api/profile/:nickname', async (req, res) => {
                 skills: user.profile.skills || [],
                 interests: user.profile.interests || [],
 
-                // ✅ إعدادات المظهر (الجديدة)
+                // إعدادات المظهر
                 theme: user.profile.theme || {
                     id: 'default',
                     primaryColor: '#3b82f6',
@@ -2553,7 +2465,7 @@ app.get('/api/profile/:nickname', async (req, res) => {
                     borderRadius: '0.5rem',
                 },
 
-                // ✅ إعدادات التخطيط (الجديدة)
+                // إعدادات التخطيط
                 layout: user.profile.layout || {
                     type: 'grid',
                     columns: 3,
@@ -2563,7 +2475,7 @@ app.get('/api/profile/:nickname', async (req, res) => {
                     showProjectLinks: true,
                 },
 
-                // ✅ إعدادات الهيدر (الجديدة)
+                // إعدادات الهيدر
                 header: user.profile.header || {
                     showAvatar: true,
                     showJobTitle: true,
@@ -2573,13 +2485,13 @@ app.get('/api/profile/:nickname', async (req, res) => {
                     layout: 'centered',
                 },
 
-                // ✅ إعدادات الفوتر (الجديدة)
+                // إعدادات الفوتر
                 footer: user.profile.footer || {
                     showCopyright: true,
                     customText: '',
                 },
 
-                // ✅ إعدادات SEO (الجديدة)
+                // إعدادات SEO
                 seo: user.profile.seo || {
                     title: user.profile.portfolioName || 'My Portfolio',
                     description: user.profile.bio || '',
@@ -2594,7 +2506,7 @@ app.get('/api/profile/:nickname', async (req, res) => {
                     nofollow: false,
                 },
 
-                // ✅ إعدادات Schema (الجديدة)
+                // إعدادات Schema
                 schema: user.profile.schema || {
                     type: 'Person',
                     name: user.profile.nickname || user.username,
@@ -2609,7 +2521,7 @@ app.get('/api/profile/:nickname', async (req, res) => {
             },
         };
 
-        // 5. تسجيل المشاهدة (اختياري)
+        // 5. تسجيل المشاهدة (اختياري) - ✅ تأكد من وجود user.notifications
         if (!isOwner) {
             try {
                 // Google Analytics
@@ -2628,10 +2540,10 @@ app.get('/api/profile/:nickname', async (req, res) => {
                     }, { timeout: 5000 });
                 }
 
-                // إشعار push لصاحب الملف الشخصي
-                if (user.notifications?.length > 0) {
+                // إشعار push لصاحب الملف الشخصي - ✅ تأكد من وجود notifications
+                if (user.notifications && user.notifications.length > 0) {
                     const subscription = user.notifications[0];
-                    if (subscription.endpoint && subscription.keys?.p256dh && subscription.keys?.auth) {
+                    if (subscription && subscription.endpoint && subscription.keys?.p256dh && subscription.keys?.auth) {
                         const payload = JSON.stringify({
                             title: '👀 Profile Viewed',
                             body: `Your profile was viewed by ${req.user?.username || 'someone'}`,
@@ -2640,7 +2552,6 @@ app.get('/api/profile/:nickname', async (req, res) => {
                     }
                 }
             } catch (analyticsError) {
-                // لا نوقف التنفيذ إذا فشلت التحليلات
                 logger.error(`Analytics error: ${analyticsError.message}`);
             }
         }