ثبت افزونه در مخزن وردپرس

aliyaghobi

کاربر عضو
سلام دوستان
یک افزونه کوتاه کننده لینک را که سایت سرویس دهیش از دسترس خارج شده را از مخزن وردپرس دانلود کردم. فارسی سازیش کردم و از سرویس سایت استفاده کردم.

و افزونه ای که من فارسی سازیش کردم

برای ثبت در مخزن برای وردپرس ارسال کردم و همچین ایمیلی برام ارسال کردند. میشه لطف کنید کمکم کنید تا مشکل را رفع کنم تا تو مخزن منتشر بشه

There are issues with your plugin code.​

Please read this ENTIRE email, address all listed issues, and reply to this email with your corrected code attached. It is required for you to read and reply to these emails, and failure to do so will result in significant delays with your plugin being accepted.​

Also please remember in addition to code quality, security and functionality, we require all plugins adhere to our guidelines. If you have not yet, please read them:​

*

## Calling files remotely​

Offloading images, js, css, cgi, and other scripts to Google (or jquery.com or anywhere else frankly) is disallowed because you're introducing an unnecessary dependency on another site.  If the file you're trying to use isn't a part of WordPress Core, then you should include it -locally- in your plugin, not remotely. If the file IS included in WordPress core, please call that instead.​

The one exception to this rule is if your plugin is performing a service. We will permit this on a case by case basis, however since this can be confusing, we have some examples of what are not permitted:​

* Offloading jquery CSS files to Google - You should include the CSS in your plugin.
* Inserting an iframe with a help doc - A link, or including the docs in your plugin is preferred.
* Calling images from your own domain - They should be included in your plugin.​

Here are some examples of what we would permit:​

* Calling font families from Google or their approved CDN (if GPL compatible)
* API calls back to your server to process possible spam comments (like Akismet)
* Offloading comments to your own servers (like Disqus)
* oEmbed calls to a service provider (like Twitter or YouTube)​

Please remove this dependency from your plugin and, if possible, include all files within the plugin (that is not called remotely). If instead you feel you ARE providing a service, please re-write your readme.txt in a manner that explains the service, the servers being called, and if any account is needed to connect.​

Examples:​

        wp_enqueue_script('plink-table','//cdn.datatables.net/1.10.4/js/jquery.dataTables.min.js',array('jquery'));
        wp_enqueue_style('plink-table','//cdn.datatables.net/1.10.4/css/jquery.dataTables.min.css');​

## Generic function (and/or define) names​

All plugins must have unique function names, defines, and classnames. This prevents your plugin from conflicting with other plugins or themes.​

For example, if your plugin is called "Easy Custom Post Types", then you might prefix your functions with ecpt_{your function name here}. Similarly a define of LICENSE would be better done as ECPT_LICENSE. You can use namespaces instead, however make sure that those also are unique. A namespace or class of 'MyPlugin' is NOT actually all that unique.​

This extends to anything in a define. For example, if you were to use this, it would be a bad idea:​

define( 'PLUGIN_PATH', plugins_url( __FILE__ ) );​

That define is a global, so PLUGIN_PATH could conflict with a number of other things.​

Don't try to use two letter slugs anymore. As of 2016, all the good ones are taken. Instead consider easy_cpts_ (from the first example).​

Similarly, don't use __ (double underscores), wp_ , or _ (single underscore) as a prefix. Those are reserved for WordPress itself. You can use them inside your classes, but not as stand-alone function.​

Please update your plugin to use more unique function and class names.​

Some examples from your plugin:​

function shortlink_create_menu()
function register_mysettings()
function  short_link_settings_page()
function wan_load_textdomain()​

If those are intended to be in shared libraries, please detect IF the code is already included and not re-include it, as doing so will cause conflicts if two people call the same defines and functions.
Please make sure you've addressed ALL issues brought up in this email. When you've corrected your code, reply to this email with the updated code attached as a zip, or provide a link to the new code for us to review. If you have questions, concerns, or need clarification, please reply to this email and just ask us.​

(While we have tried to make this review as exhaustive as possible we, like you, are humans and may have missed things. As such, we will re-review the ENTIRE plugin when you send it back to us. We appreciate your patience and understanding in this.)​

منتظر پاسخ شما هستم
سپاس

 

mostafa98

کاربر عضو
There are issues with your plugin code.​

Please read this ENTIRE email, address all listed issues, and reply to this email with your corrected code attached. It is required for you to read and reply to these emails, and failure to do so will result in significant delays with your plugin being accepted.​

Also please remember in addition to code quality, security and functionality, we require all plugins adhere to our guidelines. If you have not yet, please read them​
مسائلی در پلاگین شما وجود داره.

لطفا ایمیل رو بخونید و به کل مسائل ذکر شده رسیدگی  و کدهای اصلاح شده رو در پاسخ به این ایمیل ضمیمه کنید

لازم به ذکر است هر گونه تاخیر وعدم انجام کار بعد از خواندن این ایمیل رو خودتون پذیرفته اید.

همچنین به یاد داشته باشید علاوه بر کیفیت کد وامنیت و قابلیت پلاگین شما ملزم به پایبندی به دستورالعمل های ما هستین.

اگر هنوز افزونه شما رتبه دهی نشده لطفا دستور العمل ها را بخوانید

بقیه رو سعی میکنم زودتر ترجمه کنم

 
آخرین ویرایش توسط مدیر:

aliyaghobi

کاربر عضو
مسائلی در پلاگین شما وجود داره.

لطفا ایمیل رو بخونید و به کل مسائل ذکر شده رسیدگی  و کدهای اصلاح شده رو در پاسخ به این ایمیل ضمیمه کنید

لازم به ذکر است هر گونه تاخیر وعدم انجام کار بعد از خواندن این ایمیل رو خودتون پذیرفته اید.

همچنین به یاد داشته باشید علاوه بر کیفیت کد وامنیت و قابلیت پلاگین شما مازم به پایبندی به دستورالعمل های ما هستین.

اگر هنوز افزونه شما رتبه دهی نشده لطفا دستور العمل ها را بخوانید

بقیه رو سعی میکنم زودتر ترجمه کنم
با ترجمه اش مشکلی ندارم منظورشو فهمیدم. 

اشکال هایی که از افزونه گرفته رو میخوام بدونم چجوری باید رفع کنم

 
آخرین ویرایش توسط مدیر:

aliyaghobi

کاربر عضو
این قسمت رو درست کن
دوست عزیز میدونم به چی گیر داده گفتم که;

میخوام بدونم چجوری باید رفع کنم
آخه من دقیقن همون افزونه که تو مخزن هستش رو برداشتم و فقط ترجمه اش کردم و از API سایت Plink.ir استفاده کردم. چرا گیر داده نمیدونم!

 

mostafa98

کاربر عضو
آخه من دقیقن همون افزونه که تو مخزن هستش رو برداشتم و فقط ترجمه اش کردم و از API سایت Plink.ir استفاده کردم. چرا گیر داده نمیدونم!
    افزونه شما باید بر اساس مجوز GPLv2 باشد.
    افزونه نباید شامل موارد غیرقانونی باشد (این مورد توسط تیم بررسی کننده افزونه‌ها بررسی خواهد شد).
    نباید در افزونه لینک‌های خارجی که به یک سایت پیوند داده می‌شوند وجود داشته باشد. مانند: “ساخته شده توسط X”

در اولین مرحله شما باید یک فایل با نام readme.txt برای افزونه خود بسازید و افزونه را از طریق این لینک ارسال کنید تا پس از مدتی (بعضی از افزونه‌ها پس از چند ساعت و بعضی‌ها هم پس از چند روز تایید می‌شوند) تایید شود.

توضیحات کاملتر رو از اینجا بخون

کد:
http://mandegarweb.com/%D8%A2%D9%85%D9%88%D8%B2%D8%B4-%D8%AB%D8%A8%D8%AA-%D8%A7%D9%81%D8%B2%D9%88%D9%86%D9%87-%D9%BE%D9%84%D8%A7%DA%AF%DB%8C%D9%86-%D8%AF%D8%B1-%D9%85%D8%AE%D8%B2%D9%86-%D9%88%D8%B1%D8%AF%D9%BE%D8%B1%D8%B3/
 

MahdiY

راهبر انجمن
سلام

گفتید که دقیقا همون افزونه را کپی کردید!

داخل ایمیل دارن دقیقا به همین مورد گیر میدن

میگه که باید از اسم توابع خاص برای فانکشن هاتون استفاده کنید (برای عدم تداخل افزونه های مخزن)

اول نام همه فانکشن هاتون plink بذارید. هم یکدست میشه کدتون هم مشکل رفع میشه

 

aliyaghobi

کاربر عضو
سلام

گفتید که دقیقا همون افزونه را کپی کردید!

داخل ایمیل دارن دقیقا به همین مورد گیر میدن

میگه که باید از اسم توابع خاص برای فانکشن هاتون استفاده کنید (برای عدم تداخل افزونه های مخزن)

اول نام همه فانکشن هاتون plink بذارید. هم یکدست میشه کدتون هم مشکل رفع میشه


مهدی جان همین کاری که گفتید رو انجام دادم و برای وردپرس فرستادم. ولی باز هم به یه چیز دیگه اش ایراد گرفته!!!

کد:
This is a bad define:

define('DEFAULT_API_URL', 'http://plink.ir/api?api='.$var_Apikey.'&format=text&url=%s');

It's too generic

## Don’t use ‘if exists’ like a silver bullet

Using if (!function_exists(‘NAME ‘)) { around all your functions and classes sounds like a great idea until you realize the fatal flaw. What happens IF someone else has a function with the same name and their plugin or theme loads first?

Your plugin breaks. And that would suck. Please only use if-exists checks for SHARED libraries.

Example:

if ( ! function_exists( 'curl_get_url' ) ){
  function curl_get_url($url) {

WordPress comes with a CURL function. You should use that.
 

MahdiY

راهبر انجمن
قبل از تعریف ثابت با استفاده از تابع defined و قبل از تعریف فانکشن با استفاده از تابع function_exists بررسی کن که موجود نباشن

 

aliyaghobi

کاربر عضو
قبل از تعریف ثابت با استفاده از تابع defined و قبل از تعریف فانکشن با استفاده از تابع function_exists بررسی کن که موجود نباشن


if ( ! function_exists( 'curl_get_url' ) ){
function curl_get_url($url) {



اول نام همه فانکشن هاتون plink بذارید. هم یکدست میشه کدتون هم مشکل رفع میشه


باید به اول curl_get_url هم plink رو اضافه کنم ؟!

مهدی جان میشه لطف کنی یه نگاهی به سورسش بندازی و مشکلشو حل کنی، ممنون میشم.

You must be registered for see images attach


 

MahdiY

راهبر انجمن
باید به اول curl_get_url هم plink رو اضافه کنم ؟!

مهدی جان میشه لطف کنی یه نگاهی به سورسش بندازی و مشکلشو حل کنی، ممنون میشم.

You must be registered for see images attach
به تمام فانکشن هایی که خودت تو پلاگین تعریف کردی plink اضافه کن

 
بالا