Opened 4 years ago

Closed 4 years ago

Last modified 3 years ago

#16897 closed Bug (fixed)

makemessages option_list default value for "extensions" is missing .txt

Reported by: DrMeers Owned by: nobody
Component: Core (Management commands) Version: 1.3
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

Looks like r16437 updated everything except the option_list. Simple patch attached. Of course we might also consider listing the conditional '.js' switch here as well, but maybe that's too much detail for the option_list.

Attachments (2)

16897.diff (1010 bytes) - added by DrMeers 4 years ago.
16897_v2.diff (1.0 KB) - added by DrMeers 4 years ago.

Download all attachments as: .zip

Change History (6)

Changed 4 years ago by DrMeers

comment:1 Changed 4 years ago by julien

  • Has patch set
  • Needs documentation unset
  • Needs tests set
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted

I got bitten by this too. This wouldn't have been missed if there were tests, so it'd be good to add some :-)

comment:2 Changed 4 years ago by DrMeers

Seriously, tests for help strings?

comment:3 Changed 4 years ago by julien

  • Needs tests unset
  • Patch needs improvement set

Woops, sorry I got confused there! :P

While we're fixing this I think it's worth adding a note that the default is '.js' if the domain is 'djangojs'.

Changed 4 years ago by DrMeers

comment:4 Changed 4 years ago by julien

  • Resolution set to fixed
  • Status changed from new to closed

In [16864]:

Fixed #16897 -- Fixed some docstrings and help texts for the makemessages management command. Thanks, Simon Meers.

Note: See TracTickets for help on using tickets.
Back to Top