)]}'
{"glance/api/v2/images.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"0a50bf36472747ee10de3de93112fb8cf124c1c4","unresolved":false,"context_lines":[{"line_number":865,"context_line":"                if locations:"},{"line_number":866,"context_line":"                    # Choose best location configured strategy"},{"line_number":867,"context_line":"                    l \u003d location_strategy.choose_best_location(locations)"},{"line_number":868,"context_line":"                    image_view[\u0027direct_url\u0027] \u003d l[\u0027url\u0027] if l else None"},{"line_number":869,"context_line":"                else:"},{"line_number":870,"context_line":"                    LOG.debug(\"The \u0027locations\u0027 list of image %s is empty, \""},{"line_number":871,"context_line":"                              \"not including \u0027direct_url\u0027 in response\","}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa0c359_a7136566","line":868,"updated":"2018-02-06 04:47:32.000000000","message":"I don\u0027t think this is necessary.  We get the locations list from the image at line 864, and if it\u0027s None or empty we won\u0027t be in this branch, we\u0027ll be processing the else.  (Also, you can see from the unit tests for the location strategies that if they\u0027re passed a non-empty location list, they do return a location.)","commit_id":"90e8e7e22df75d977c0ded57ac512dcaed47fad7"}]}