Page MenuHomePhabricator
Create Task
Maniphest T150090

Warnings thrown up on creating a new newsletter via Special:CreateNewsletter
Closed, ResolvedPublic
Actions

Description

Errors thrown up on creating a new newsletter via Special:CreateNewsletter:

Warning: Missing argument 1 for NewsletterValidator::validate(), called in /var/www/core/core/extensions/Newsletter/includes/specials/SpecialNewsletterCreate.php on line 82 and defined in /var/www/core/core/extensions/Newsletter/includes/NewsletterValidator.php on line 34

Notice: Undefined variable: new in /var/www/core/core/extensions/Newsletter/includes/NewsletterValidator.php on line 35

Notice: Undefined variable: new in /var/www/core/core/extensions/Newsletter/includes/NewsletterValidator.php on line 43

Mentor: @01tonythomas

Details

SubjectRepoBranchLines +/-
Fix "Warnings thrown up on creating a new newsletter via Special:CreateNewsletter"mediawiki/extensions/Newslettermaster+1 -1
Customize query in gerrit

Related Objects
Search...

StatusSubtypeAssignedTask
 DuplicateQgilT125545 Phabricator Q&A session for Community Liaisons
 ResolvedQgilT116025 Goal: Align Community Liaison and Developer Relations project management practices
 ResolvedQgilT119387 Community Liaison and Developer Relation quarterly goals for January - March 2016
 DeclinedNoneT104131 Exporting existing newsletter to the Newsletter extension
 ResolvedAddshoreT110170 Goal: Deploy Newsletter extension in Wikimedia
 DuplicateNoneT115098 Deploy Newsletter extension in beta cluster
 ResolvedoriT127297 Add the Newsletter extension to the Beta Cluster
 ResolvedBawolffT115095 Security review of Newsletter extension
 ResolvedGlaisherT116382 Raw html should be escaped before output in Newsletter pages
 ResolvedTinaj1234T110491 Newsletter extension should have validation
 DeclinedNoneT131616 Cap the number of active newsletters per user
 ResolvedGlaisherT132016 Add CheckUser integration to Extension:Newsletter
 Resolved01tonythomasT132018 Add newsletter description to log messages
 ResolvedGlaisherT132017 Throttle for newsletter announcements
 Resolved01tonythomasT132019 Add table prefix to sub-queries in NewsletterTablePager
 ResolvedPpperyT132022 Add AbuseFilter integration to Extension:Newsletter
 ResolvedDaimonaT135668 Need documentation on How to integrate AbuseFilter with xyz extension
 ResolvedDaimonaT239348 Custom actions should always be included in the list of available actions
 Resolved01tonythomasT119058 Special:Newsletter fatals if the newsletter main page is deleted
 ResolvedQgilT154229 Agree first user group that will be able to create newsletters in Wikimedia
 ResolvedFilipT154534 Restrict 'newsletter-{create,manage,destroy}' only to 'sysop' by default
 ResolvedxSavitarT159075 Fix character escaping throughout the extension files
ResolvedNoneT159084 [Security] Fix inconsistent Newsletter user rights
Resolved01tonythomasT159085 [Security] Improve data attribute naming to avoid forge
 Resolved01tonythomasT160854 Newsletter strange behaviour on an API create/edit
Resolved01tonythomasT140402 Namespace is repeated in notification link
 Resolved01tonythomasT138462 Convert Newsletter to use ContentHandler
 ResolvedFilipT150090 Warnings thrown up on creating a new newsletter via Special:CreateNewsletter

Event Timeline

01tonythomas created this task.Nov 5 2016, 3:28 PM
01tonythomas moved this task from Backlog to Google Code In 2016 on the MediaWiki-extensions-Newsletter board.Nov 5 2016, 4:51 PM
Aklapper renamed this task from [Beginner] Warnings thrown up on creating a new newsletter via Special:CreateNewsletter to Warnings thrown up on creating a new newsletter via Special:CreateNewsletter.Nov 5 2016, 5:02 PM
01tonythomas updated the task description. (Show Details)Nov 5 2016, 5:02 PM
Aklapper moved this task from Proposed tasks to Imported in GCI Site on the Google-Code-In-2016 board.Nov 28 2016, 12:11 AM

Imported as https://codein.withgoogle.com/tasks/5186061135249408/

gerritbot added a comment.Dec 4 2016, 11:18 AM

Change 325158 had a related patch set uploaded (by Filip):
Fix "Warnings thrown up on creating a new newsletter via Special:CreateNewsletter"

https://gerrit.wikimedia.org/r/325158

gerritbot added a project: Patch-For-Review.Dec 4 2016, 11:18 AM
gerritbot added a comment.Dec 4 2016, 12:28 PM

Change 325158 merged by jenkins-bot:
Fix "Warnings thrown up on creating a new newsletter via Special:CreateNewsletter"

https://gerrit.wikimedia.org/r/325158

01tonythomas closed this task as Resolved.Dec 4 2016, 11:36 PM
01tonythomas assigned this task to Filip.

Thank you!

Qgil awarded a token.Dec 6 2016, 8:56 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL