#7028: Better raw_id_fields feedback in newform-admins branch
---------------------------------------------------+------------------------
          Reporter:  Marcob <marcob...@gmail.com>  |         Owner:  nobody     
              
            Status:  new                           |     Milestone:  1.2        
              
         Component:  django.contrib.admin          |       Version:  SVN        
              
        Resolution:                                |      Keywords:  
raw-id-fields nfa-someday
             Stage:  Accepted                      |     Has_patch:  1          
              
        Needs_docs:  0                             |   Needs_tests:  0          
              
Needs_better_patch:  0                             |  
---------------------------------------------------+------------------------
Changes (by mrts):

  * needs_better_patch:  1 => 0
  * version:  1.0 => SVN
  * needs_tests:  1 => 0

Comment:

 Tried my best to create a clean representation of the intent:
 {{{
             yield mark_safe(u'<%s%s><a href="%s"%s>%s</a></%s>' %
                 (table_tag, row_class, url,
                     (cl.is_popup
                         and ' onclick='
                             '"opener.dismissRelatedLookupPopup(window, %s,
 '
                             "'<a href=&quot;%s&quot; "
                             'onclick=&quot;return
 showRelatedObjectPopup(this);&quot;'
                             '>%s</a>\'); return false;"' %
                             (result_id, result_url, result_name)
                         or ''),
                     conditional_escape(result_repr), table_tag))
 }}}

 Commit:
 http://github.com/mrts/django/commit/e95f27ad4b74eef3a9d4e9553e5cfb58f159c10a

 Tests pass, also tested to be working in a real site.

 Patch upcoming.

-- 
Ticket URL: <http://code.djangoproject.com/ticket/7028#comment:20>
Django <http://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 post to this group, send email to django-updates@googlegroups.com
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to