שיחת משתמש:1Or/clearsandbox.js

קיפודנחש, שיפצתי את מה שהיה כתוב ב‏‏משתמש:Yonidebest/monobook.js/clearSandBox.js על מנת שזה יתאים לAPI ויהיה יותר קל לשימוש. אפשר לשנות את הדף ‏‏משתמש:Yonidebest/monobook.js/clearSandBox.js למה שכתבתי פה? בברכה, אור • (ש|ת|מ) • 11:20, 29 באוקטובר 2016 (IDT)

Yonidebest אור • שיחה • 16:50, 30 באוקטובר 2016 (IST)

כמובן שאי אפשר לשנות סקריפט במרחב המשתמש של מישהו אחר. אתה יכול להוסיף את הסקריפט שלך לוק:ס, ואם תבדוק אותו לעומק אולי אפילו להחליף את הקישור הקיים משם לסקריפט של יוני בקישור לסקריפט שלך. הסקריפט הזה לא רע, אבל יש לו כמה בעיות:

1. הוא מוסיף את הפונקציה clearsandbox למרחב הגלובלי (window). לא נחוץ, ומקובל להימנע מכך.
2. הוא בודק ישירות את המשתנה הגלובלי wgPageName . המשתנים הללו הם deprecated ובקרוב ייעלמו. יש להשתמש ב-mw.config.get
3. הוא משתמש בפונקציה deprecated‏ (mw.util.jsMessage) ראה תיעוד בmw:ResourceLoader/Modules
4. הוא משתמש בפונקציות משלושה מודולים שעלולים להיות טעונים, אבל גם עלולים לא להיות טעונים. עדיף לטעון אותם בפירוש, בעזרת mw.loader.using (המודולים הם mediawiki.user, mediawiki.util, ו-mediawiki.api).
5. הצורה שהוא מדביק את הסעיף בתפריט לא מקובלת ומוזרה: אם רוצים להדביק עוד אפשרויות ל-contentSub (יש סקריפטים שעושים זאת), הדרך המקובלת ביא לא בעזרת addPortletLink - זה יוצר מראה משונה ולא קונסיסטנטי.

קיפודנחש 20:59, 30 באוקטובר 2016 (IST)

היי קיפודנחש, תיקנתי הכל לפי הערותיך. האם עכשיו הסקריפט עומד בכל הסטנדרטים? תודה, אור • שיחה • 11:33, 12 במרץ 2017 (IST)

נראה לי שכן, אבל בכל זאת....

1. למה אתה דורש את mediawiki.util ? לא נראה שהסקריפט משתמש בו.
2. אם במקום לבקש mediawiki.api תבקש mediawiki.api.edit, תוכל לחסוך את הטיפול הידני ב-token (וכך לחסוך את הדרישה ל-mediawiki.user), ולפשט את הקוד עוד. זה נראה בערך כך:

new mw.Api()
    .edit( 'ויקיפדיה:ארגז חול',  function(){ return '{{' + 'ס:ריקון' + '}}'; } )
    .done( function() {
        // מה שאתה עושה היום
    } );

קיפודנחש 17:20, 14 במרץ 2017 (IST)

תודה על ההערה קיפודנחש. החלפתי את הAPI והסרתי את הדרישה לuser ולutil. בברכה, אור • שיחה • למדו את ויקיפדיה לזהות השחתות בעצמה! • 17:34, 14 במרץ 2017 (IST)