FAQ
#22648: Overridden get_lookup/get_transform not chainable
-------------------------------------+-------------------------------------
                Reporter: mjtamlyn | Owner: nobody
                    Type: Bug | Status: new
               Component: Database | Version: master
   layer (models, ORM) | Keywords:
                Severity: Normal | Has patch: 0
            Triage Stage: Accepted | Needs tests: 0
     Needs documentation: 0 | Easy pickings: 0
Patch needs improvement: 0 |
                   UI/UX: 0 |
-------------------------------------+-------------------------------------
  If a transform has an `output_type` which has a customised `get_transform`
  or `get_lookup` function, then it is not properly called.

--
Ticket URL: <https://code.djangoproject.com/ticket/22648>
Django <https://code.djangoproject.com/>
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+unsubscribe@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/051.b3e99113f720f3cea2bf822b1671ec24%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Search Discussions

  • Django at May 17, 2014 at 10:15 am
    #22648: Overridden get_lookup/get_transform not chainable
    -------------------------------------+-------------------------------------
          Reporter: mjtamlyn | Owner: nobody
              Type: Bug | Status: new
         Component: Database layer | Version: master
       (models, ORM) | Resolution:
          Severity: Normal | Triage Stage: Accepted
          Keywords: | Needs documentation: 0
         Has patch: 1 | Patch needs improvement: 0
       Needs tests: 0 | UI/UX: 0
    Easy pickings: 0 |
    -------------------------------------+-------------------------------------
    Changes (by mjtamlyn):

      * has_patch: 0 => 1


    Comment:

      Fixed in
      https://github.com/mjtamlyn/django/commit/253ef84bcf3c10660941301f81e92d02e43886c6

    --
    Ticket URL: <https://code.djangoproject.com/ticket/22648#comment:1>
    Django <https://code.djangoproject.com/>
    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+unsubscribe@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/066.5a52cdddd932995f309cb43f3d9c47c5%40djangoproject.com.
    For more options, visit https://groups.google.com/d/optout.
  • Django at May 17, 2014 at 10:26 am
    #22648: Overridden get_lookup/get_transform not chainable
    -------------------------------------+-------------------------------------
          Reporter: mjtamlyn | Owner: nobody
              Type: Bug | Status: closed
         Component: Database layer | Version: master
       (models, ORM) | Resolution: fixed
          Severity: Normal | Triage Stage: Accepted
          Keywords: | Needs documentation: 0
         Has patch: 1 | Patch needs improvement: 0
       Needs tests: 0 | UI/UX: 0
    Easy pickings: 0 |
    -------------------------------------+-------------------------------------
    Changes (by Marc Tamlyn <marc.tamlyn@…>):

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


    Comment:

      In [changeset:"a2dd618e3b4a7472fab852da450ca5eef92a922f"]:
      {{{
      #!CommitTicketReference repository=""
      revision="a2dd618e3b4a7472fab852da450ca5eef92a922f"
      Fixed #22648 -- Transform.output_type should respect overridden
      custom_lookup and custom_transform.

      Previously, class lookups from the output_type would be used, but any
      changes to custom_lookup or custom_transform would be ignored.
      }}}

    --
    Ticket URL: <https://code.djangoproject.com/ticket/22648#comment:2>
    Django <https://code.djangoproject.com/>
    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+unsubscribe@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/066.ae5ffe983d43d0396fb86c6a1a20e532%40djangoproject.com.
    For more options, visit https://groups.google.com/d/optout.
  • Django at May 18, 2014 at 6:39 pm
    #22648: Overridden get_lookup/get_transform not chainable
    -------------------------------------+-------------------------------------
          Reporter: mjtamlyn | Owner: nobody
              Type: Bug | Status: closed
         Component: Database layer | Version: master
       (models, ORM) | Resolution: fixed
          Severity: Normal | Triage Stage: Accepted
          Keywords: | Needs documentation: 0
         Has patch: 1 | Patch needs improvement: 0
       Needs tests: 0 | UI/UX: 0
    Easy pickings: 0 |
    -------------------------------------+-------------------------------------

    Comment (by apollo13):

      Needs backport to 1.7, also see the fix in
      https://github.com/django/django/commit/5a2556afb9b1b3ef6e0622552970c67ac84ecd28
      -- not sure if field_type should ever be None, please fix in the process
      of backporting.

    --
    Ticket URL: <https://code.djangoproject.com/ticket/22648#comment:3>
    Django <https://code.djangoproject.com/>
    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+unsubscribe@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/066.51a76a7e031c9f152da6eaf5390d7c25%40djangoproject.com.
    For more options, visit https://groups.google.com/d/optout.

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupdjango-updates @
categoriesdjango, python
postedMay 17, '14 at 10:11a
activeMay 18, '14 at 6:39p
posts4
users1
websitedjangoproject.com

1 user in discussion

Django: 4 posts

People

Translate

site design / logo © 2018 Grokbase