Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2021-02-18 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  David
 |  Smith
 Type:  Bug  |   Status:  closed
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-

Comment (by GitHub ):

 In [changeset:"96a50934007f4a9a14c76a08be5bbd3800b4b66c" 96a50934]:
 {{{
 #!CommitTicketReference repository=""
 revision="96a50934007f4a9a14c76a08be5bbd3800b4b66c"
 Refs #30231 -- Fixed SeleniumTests.test_inlines_verbose_name with headless
 mode.

 Horizontal scrollbar doesn't appear with the headless mode on small
 windows, that's why window.scrollTo() is not an option for these tests
 even after fixing #32459.
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.17baf9ab4ca4d33a6a85d45c3b6900cd%40djangoproject.com.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2021-02-16 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  David
 |  Smith
 Type:  Bug  |   Status:  closed
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-
Changes (by Carlton Gibson ):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"45bef6706ae7436b5c2c503018b21ff70b2e6728" 45bef670]:
 {{{
 #!CommitTicketReference repository=""
 revision="45bef6706ae7436b5c2c503018b21ff70b2e6728"
 Fixed #30231 -- Fixed admin filter horizontal/vertical verbose_name
 generation.

 Co-authored-by: David Smith 
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.ee502ce00a5723caa0a0922ece19adc1%40djangoproject.com.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2021-02-11 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  David
 |  Smith
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-
Changes (by Carlton Gibson):

 * stage:  Accepted => Ready for checkin


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.8f1dae6e72d2142a568e8c106b49f550%40djangoproject.com.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2020-09-23 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+---
 Reporter:  Florian Schuler  |Owner:  David Smith
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+---
Changes (by David Smith):

 * owner:  Teresa Partida => David Smith


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.4596a885a4afe4b86263900fa616fa41%40djangoproject.com.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2020-09-15 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  Teresa
 |  Partida
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-
Changes (by felixxm):

 * cc: Carlton Gibson (added)
 * needs_tests:  1 => 0


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.32b968f201210af31eca1b8b6da1fa75%40djangoproject.com.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-04-25 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  Teresa
 |  Partida
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-

Comment (by Teresa Partida):

 Hi! I submitted a new patch with the changes Florian made and the Selenium
 test
 https://github.com/django/django/pull/11282

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.7413a812f7b3510999c7381515708195%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-04-22 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:
 |  TeresaPartidaS
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-
Changes (by TeresaPartidaS):

 * status:  new => assigned
 * owner:  (none) => TeresaPartidaS


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.dd8f04c56a16259a0dbb52e31bc3aee9%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-03-18 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+
 Reporter:  Florian Schuler  |Owner:  (none)
 Type:  Bug  |   Status:  new
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+

Comment (by Florian Schuler):

 Replying to [comment:14 Tim Graham]:
 > Okay, I'll deassign the ticket then. Even trivial bug fixes require
 tests to prevent regressions.

 Hey Tim, Okay!
 Just noticed that the owner field of this ticket is now empty. Shouldn't
 it be "nobody" like the other new tickets so that other users can query
 for unassigned tickets by searching for owner "nobody"?

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.ffc8a1d23725f6acc73991d70c8e322e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-03-17 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+
 Reporter:  Florian Schuler  |Owner:  (none)
 Type:  Bug  |   Status:  new
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+
Changes (by Tim Graham):

 * status:  assigned => new
 * owner:  Florian Schuler => (none)


Comment:

 Okay, I'll deassign the ticket then. Even trivial bug fixes require tests
 to prevent regressions.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.897e72dda2fae3218bcd75ec76b74856%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-03-11 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  Florian
 |  Schuler
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-

Comment (by Florian Schuler):

 Replying to [comment:12 Tim Graham]:
 > Tests (probably using selenium in this case) are also required.
 Hey Tim, I don't know what tests could be useful in this case of UI bug
 where only a wrong name was passed to display (it's a very trivial bugfix,
 simply fetching the name from fieldName attribute instead). I don't have
 any experience in writing tests using selenium for Django UI and also
 don't have any time for reading into it and setting up a testing
 environment at the moment. So if there really should be tests needed,
 someone else needs to take care of it.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.995ebc1eb8f1f678c63ec05c9fa28b5f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-03-11 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  Florian
 |  Schuler
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-
Changes (by Tim Graham):

 * needs_tests:  0 => 1


Comment:

 Tests (probably using selenium in this case) are also required.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.0727dfc963f224cf7838d0f8606832ef%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-03-11 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  Florian
 |  Schuler
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-
Changes (by Florian Schuler):

 * has_patch:  0 => 1


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.b64acb94353fe799c42a097dfc07366f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-03-11 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  Florian
 |  Schuler
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-

Comment (by Florian Schuler):

 Just submitted a patch with the changes i already mentioned above and made
 a pull request.
 https://github.com/django/django/pull/11076

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.39023be16ae21ab5cf6f5ae2c66ce8dc%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-03-11 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  Florian
 |  Schuler
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-
Changes (by Florian Schuler):

 * owner:  Ahisahar Pretel => Florian Schuler


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.dad86ba1ba19a5a252ba0e91b2d1f003%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-03-05 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  Ahisahar
 |  Pretel
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-

Comment (by Florian Schuler):

 Replying to [comment:7 Ahisahar Pretel]:
 > I should probably let it to someone who has more experience right
 @Florian Schuler @Tim Graham? This is my first contribution

 Hey @Ahisahar Pretel and @Tim Graham, I just found the origin of this
 wrong behaviour and fixed it in my local Django system (2.1.7)!
 Like already noted in my bug report it seems like a small logic fault in a
 javascript file. In the file which is responsible for initializing the
 FilteredSelectMultiple widget at page load
 (django/contrib/admin/static/admin/js/SelectFilter2.js) there is the
 correct usage of calling the SelectFilter.init() method at the very end
 (line 244 to 250):

 {{{
 window.addEventListener('load', function(e) {
 $('select.selectfilter,
 select.selectfilterstacked').each(function() {
 var $el = $(this),
 data = $el.data();
 SelectFilter.init($el.attr('id'), data.fieldName,
 parseInt(data.isStacked, 10));
 });
 });
 }}}

 So second parameter has to be the data attribute fieldName of the HTML
 element, which represents the verbose_name if one is given for the field.

 In the js file which is responsible for triggering the initialization of
 newly added FilteredSelectMultiple Widgets after clicking the "add
 another" button (django/contrib/admin/static/admin/js/inlines.js) this
 parameter isn't set correctly.

 **For tabular inlines:**

 Current function:
 {{{
 var updateSelectFilter = function() {
 // If any SelectFilter widgets are a part of the new form,
 // instantiate a new SelectFilter instance for it.
 if (typeof SelectFilter !== 'undefined') {
 $('.selectfilter').each(function(index, value) {
 var namearr = value.name.split('-');
 SelectFilter.init(value.id, namearr[namearr.length -
 1], false);
 });
 $('.selectfilterstacked').each(function(index, value) {
 var namearr = value.name.split('-');
 SelectFilter.init(value.id, namearr[namearr.length -
 1], true);
 });
 }
 };
 }}}

 So it gets the field name out of the "name" attribute which is the real
 field name, not the verbose name which could be easily taken with
 "$(this).data('fieldName')"

 Corrected function should be:
 {{{
 var updateSelectFilter = function() {
 // If any SelectFilter widgets are a part of the new form,
 // instantiate a new SelectFilter instance for it.
 if (typeof SelectFilter !== 'undefined') {
 $('.selectfilter').each(function(index, value) {
 SelectFilter.init(value.id, $(this).data('fieldName'),
 false);
 });
 $('.selectfilterstacked').each(function(index, value) {
 SelectFilter.init(value.id, $(this).data('fieldName'),
 true);
 });
 }
 };
 }}}

 **The same for Stacked inline:**

 Current function:
 {{{
  var updateSelectFilter = function() {
 // If any SelectFilter widgets were added, instantiate a new
 instance.
 if (typeof SelectFilter !== "undefined") {
 $(".selectfilter").each(function(index, value) {
 var namearr = value.name.split('-');
 SelectFilter.init(value.id, namearr[namearr.length -
 1], false);
 });
 $(".selectfilterstacked").each(function(index, value) {
 var namearr = value.name.split('-');
 SelectFilter.init(value.id, namearr[namearr.length -
 1], true);
 });
 }
 };
 }}}

 Corrected function should be:

 {{{
 var updateSelectFilter = function() {
 // If any SelectFilter widgets were added, instantiate a new
 instance.
 if (typeof SelectFilter !== "undefined") {
 $(".selectfilter").each(function(index, value) {
 SelectFilter.init(value.id, 

Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-03-05 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  Ahisahar
 |  Pretel
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-

Comment (by Ahisahar Pretel):

 I should probably let it to someone who has more experience right @Florian
 Schuler @Tim Graham? This is my first contribution

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.e9df02a648a439bb06bd95d49c4d2f92%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button

2019-03-04 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  Ahisahar
 |  Pretel
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-

Comment (by Florian Schuler):

 Replying to [comment:4 Ahisahar Pretel]:
 > Hello @Florian Schuler, actually not. It happens in admin.TabularInline
 view or in the admin modelview once that we have a form with
 FilteredSelectMultiple?
 >
 > Thank you

 Like Tim Graham just confirmed it happens in admin.TabularInline view,
 when adding a field to "filter_horizontal". In addition I already tried to
 overwrite get_formset method of the regarding admin.TabularInline and
 added FilteredSelectMultiple widget manually with verbose_name redefined
 on the widget instead, but with the same result. So the bug occurs with
 manually added FilteredSelectMultiple widgets too.

 @Tim Graham thank you for the confirmation of the bug and clarifying the
 title of the ticket! Of course it is important that this only happens to
 inlines added via the "add another" button.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.ea389c6dc29be3cfed64e0c0c4260711%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for inlines created with "Add another" button (was: "verbose_name" is ignored in FilteredSelectMultiple widget for

2019-03-04 Thread Django
#30231: Field's verbose_name is ignored in FilteredSelectMultiple widget for
inlines created with "Add another" button
-+-
 Reporter:  Florian Schuler  |Owner:  Ahisahar
 |  Pretel
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  1
-+-
Changes (by Tim Graham):

 * stage:  Unreviewed => Accepted
 * easy:  1 => 0


Comment:

 I reproduced at 9681e968ebdcd58cac99c1e60f0a6932abd4e5c9 with this patch
 on the tutorial:
 {{{ #!diff
 diff --git a/polls/admin.py b/polls/admin.py
 index f1bf573..0a184ab 100644
 --- a/polls/admin.py
 +++ b/polls/admin.py
 @@ -6,6 +6,7 @@ from .models import Choice, Question
  class ChoiceInline(admin.TabularInline):
  model = Choice
  extra = 3
 +filter_horizontal = ['groups']


  class QuestionAdmin(admin.ModelAdmin):
 diff --git a/polls/models.py b/polls/models.py
 index 38ae549..b0756e3 100644
 --- a/polls/models.py
 +++ b/polls/models.py
 @@ -4,6 +4,7 @@ import datetime

  from six import python_2_unicode_compatible

 +from django.contrib.auth.models import Group
  from django.db import models
  from django.utils import timezone

 @@ -29,6 +30,7 @@ class Choice(models.Model):
  question = models.ForeignKey(Question, on_delete=models.CASCADE)
  choice_text = models.CharField(max_length=200)
  votes = models.IntegerField(default=0)
 +groups = models.ForeignKey(Group, blank=True, verbose_name='test')

  def __str__(self):
  return self.choice_text
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/072.3179fc57d80490e88ad7bfffaf2ca2db%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.