الأتمتة الدورية المُفعَّلة شهريًا لا تعمل كما هو متوقع

Lilly · 12 مايو 2026، 8:04ص

حسناً، لقد تحدثتُ اليوم مع موين حول هذه القضية، وقد أشارت إلى أنه ربما يكون هناك خطأ برمجي عند استخدام الخيار الشهري، لأن طول الشهر يختلف من شهر لآخر؟ بالطبع كانت محقة إلى حد ما…

على أي حال، قمتُ ببعض البحث الإضافي

لم أستطع فعلياً العثور على سكريبت الأتمتة الخاص بـ schedule pm with data-explorer results في مجلد plugins/automation/lib/discourse_automation/scripts الأساسي، لذا فهو مختبئ في مكان آخر.

لكنني كنتُ أراجع المنطق هنا الخاص بالتكرار (من المفترض أن السكريبت يستخدمه) في الملف plugins/automation/lib/discourse_automation/triggers/recurring.rb:

github.com/discourse/discourse

plugins/automation/lib/discourse_automation/triggers/recurring.rb

4c15bb09c


      
          when "month"
            count = 0
            (start_date.beginning_of_month.to_date..start_date.end_of_month.to_date).each do |date|
              count += 1 if date.strftime("%A") == start_date.strftime("%A")
              break if date.day == start_date.day
            end
            RRule::Rule
              .new("FREQ=MONTHLY;INTERVAL=#{interval};BYDAY=#{count}#{byday}", dtstart: start_date)
              .between(Time.now, interval_end.months.from_now)
              .find { |date| date > Time.zone.now }

أعتقد أن الكود الأساسي للأتمتة الشهرية المتكررة يحسب اليوم الأسبوعي رقمي (مثل اليوم الثالث) بدلاً من تاريخ التقويم الفعلي، وهناك عدم تطابق في المنطق مع Calendar day - فبإدراج BYDAY=#{count}#{byday} في قاعدة RRule، تجبر الأتمتة على محاذاة يوم الأسبوع، بينما يجب استخدام BYMONTHDAY بدلاً من BYDAY.

لذلك، إذا قمتَ بضبط الأتمتة لتبدأ في 15 أبريل (الذي يقع في اليوم الثالث من الأربعاء في أبريل)، فسيفسر النظام القاعدة على النحو التالي: تشغيل في الأربعاء الثالث من كل شهر.

وعندما يأتي شهر مايو، يكون الأربعاء الثالث هو 20 مايو، مما يتسبب في انحراف الجدول الزمني بمقدار 5 أيام.

علاوة على ذلك، إذا قمتَ بضبط الأتمتة في اليوم 30 أو 31 من شهر ما (مثل “الثلاثاء الخامس”)، فإن النظام سيبحث عن الثلاثاء الخامس في الشهر التالي. وإذا كان ذلك الشهر يحتوي فقط على 4 أيام ثلاثاء، فإن الأتمتة ستفشل تماماً في العثور على تاريخ صالح وستتخطى الشهر بأكمله بصمت.

لدي إصلاح تم اختباره وأنا أقوم بإنشاء طلب دمج (PR) إذا أراد الفريق الاطلاع عليه. يغير هذا الإصلاح المنطق من اليوم الأسبوعي رقمي إلى تاريخ التقويم الدقيق.

إليك بعض لقطات الشاشة من اختباراتي وتصحيح الأخطاء:

حالتان، واحدة بدون إصلاح والأخرى مع إصلاح المنطق الخاص بي

قم بإنشاء نفس الأتمتة تماماً في كلتا الحالتين (النسخة المحسنة على اليمين):

انتبه إلى الفروقات في اللافتات التي تقول “سيتم تشغيل الأتمتة التالية في:” مع انحراف 5 أيام - النسخة غير المصححة تنحرف إلى الأربعاء الثالث (20 مايو)، بدلاً من 15 مايو.

على سبيل المثال، قد يتم تخطي شهور كاملة إذا كان النظام يبحث عن الخميس الرابع بدلاً من تاريخ فعلي.

يمكنك أيضاً اختبار النتيجة نفسها في وحدة تحكم Rails

غير مصحح:

مع الإصلاح:

طلب الدمج هنا:

github.com/discourse/discourse

FIX: Automation Plugin - Monthly recurring triggers shifting dates and skipping months (BYDAY to BYMONTHDAY) (#39917)

main ← Lillinator:automation-recurring-month-fix

opened 07:59AM - 12 May 26 UTC

Lillinator

+4 -9

**Description:** **Context / Bug:** Currently, when an admin sets a monthly …recurring automation, the script attempts to calculate the "Nth Weekday" (e.g., "The 3rd Tuesday") using `BYDAY` instead of the actual calendar date. This causes two major issues for administrators: 1. **Shifting Dates:** Users expect a monthly report to run on the exact same calendar date (e.g., the 15th). Instead, the date shifts back and forth depending on when the "3rd Tuesday" or "2nd Friday" falls in the following month. 2. **Silent Failures:** If an admin sets a start date on the 30th or 31st of a month (e.g., the 5th Tuesday), the automation generates a rule like `BYDAY=5TU`. Months that only have 4 Tuesdays will completely fail to find a valid execution date and silently skip the entire month. **Changes:** * Replaced the `BYDAY` string calculation with `BYMONTHDAY=#{start_date.day}`. Monthly schedules will now reliably trigger on the exact calendar date matching user expectations. * Removed the now-unused `count` while-loop that was calculating the weekday occurrence. * *Minor cleanup:* Swapped `Time.now` for `Time.zone.now` in the RRule bounds check for standard Rails timezone consistency. **Test Updates:** * Updated two existing assertions in `recurring_spec.rb`. The original tests were explicitly (and incorrectly) expecting the buggy Nth-weekday drift. I corrected the assertion dates to expect the proper exact calendar day. **Files Touched:** * `plugins/discourse-automation/lib/discourse_automation/triggers/recurring.rb` * `plugins/discourse-automation/spec/lib/discourse_automation/triggers/recurring_spec.rb`

الموضوع		الردود	مرات العرض
Automation script Schedule PM with Data Explorer results not sending PMs Bug automation	12	1270	6 يوليو 2023
Discourse Automation Plugin official , automation , included-in-core	76	24460	19 مارس 2026
Expand automation scheduling options Feature data-explorer , automation	3	103	1 مايو 2025
Calendar and Event Plugin: recurring event option: [n]th [week]day of the month Feature calendar-and-event	4	183	23 فبراير 2026
Policy Plugin: Monthly policy recurrence Feature policy	3	619	14 سبتمبر 2020

الأتمتة الدورية المُفعَّلة شهريًا لا تعمل كما هو متوقع

الموضوعات ذات الصلة