)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"d254b69b755b466e769ab3e7059cf9376bfa245a","unresolved":false,"context_lines":[{"line_number":18,"context_line":"  @hybrid_property"},{"line_number":19,"context_line":"  def foo(..)"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"If referencing Wozzle.foo, then you need a hybrid property, otherwise"},{"line_number":22,"context_line":"a regular @property will work just fine."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Change-Id: Ifd3ff9a888919e09b80e87282af829bc14693369"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a3c09a3_9ae36a33","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":25},"updated":"2017-01-18 22:13:12.000000000","message":"this kind of references are to build queries, right?","commit_id":"0f759fde8d5f46d548e56efd332cd2ff3753692f"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"5d4942a03ad2ff92766f38aa7fb096ad72b4083e","unresolved":false,"context_lines":[{"line_number":18,"context_line":"  @hybrid_property"},{"line_number":19,"context_line":"  def foo(..)"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"If referencing Wozzle.foo, then you need a hybrid property, otherwise"},{"line_number":22,"context_line":"a regular @property will work just fine."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Change-Id: Ifd3ff9a888919e09b80e87282af829bc14693369"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a3c09a3_dab972cd","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":25},"in_reply_to":"7a3c09a3_9ae36a33","updated":"2017-01-18 22:26:32.000000000","message":"yep!","commit_id":"0f759fde8d5f46d548e56efd332cd2ff3753692f"}],"keystone/identity/backends/sql_model.py":[{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"68c70c28c607e3710d1b816f22a30a3249e29d31","unresolved":false,"context_lines":[{"line_number":170,"context_line":"    # enabled property"},{"line_number":171,"context_line":"    @hybrid_property"},{"line_number":172,"context_line":"    def enabled(self):"},{"line_number":173,"context_line":"        \"\"\"Return whether user is created or not.\"\"\""},{"line_number":174,"context_line":"        if self._enabled:"},{"line_number":175,"context_line":"            max_days \u003d ("},{"line_number":176,"context_line":"                CONF.security_compliance.disable_user_account_days_inactive)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_bfadc39c","line":173,"range":{"start_line":173,"start_character":34,"end_line":173,"end_character":41},"updated":"2017-01-17 20:52:42.000000000","message":"enabled","commit_id":"8cc17be1cf8455b86f9e9a6269bbb46c82e1f6fe"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"34064ac6fe615092f74af4fa5f6f636ba41f3df7","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        else:"},{"line_number":84,"context_line":"            return None"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    @hybrid_property"},{"line_number":87,"context_line":"    def password(self):"},{"line_number":88,"context_line":"        \"\"\"Return the current password.\"\"\""},{"line_number":89,"context_line":"        if self.password_ref:"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a3c09a3_da1436a1","line":86,"range":{"start_line":86,"start_character":5,"end_line":86,"end_character":20},"updated":"2017-01-18 13:30:21.000000000","message":"why is this still \"hybrid\"?","commit_id":"4e69e9ed667d30f039acb99dd71e043b3b2c0d35"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"8905cb7afec8a4af766639431fde2e640c36b798","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        else:"},{"line_number":84,"context_line":"            return None"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    @hybrid_property"},{"line_number":87,"context_line":"    def password(self):"},{"line_number":88,"context_line":"        \"\"\"Return the current password.\"\"\""},{"line_number":89,"context_line":"        if self.password_ref:"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a3c09a3_9748bcb4","line":86,"range":{"start_line":86,"start_character":5,"end_line":86,"end_character":20},"in_reply_to":"7a3c09a3_da1436a1","updated":"2017-01-18 15:55:27.000000000","message":"If it still should be hybrid, the note describing why should be above it.","commit_id":"4e69e9ed667d30f039acb99dd71e043b3b2c0d35"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"d254b69b755b466e769ab3e7059cf9376bfa245a","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    # NOTE(stevemar): we use a hybrid property here because we leverage the"},{"line_number":88,"context_line":"    # expression method, see `@password.expression` and `Password.password`"},{"line_number":89,"context_line":"    # below."},{"line_number":90,"context_line":"    @hybrid_property"},{"line_number":91,"context_line":"    def password(self):"},{"line_number":92,"context_line":"        \"\"\"Return the current password.\"\"\""},{"line_number":93,"context_line":"        if self.password_ref:"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a3c09a3_fa0d4ee9","line":90,"range":{"start_line":90,"start_character":5,"end_line":90,"end_character":20},"updated":"2017-01-18 22:13:12.000000000","message":"if accessed via Password.password, I think it will need the password_ref to return an expression, so I think we still need \"hybrid_property\" in line 79. Please correct me if I\u0027m confusing stuff here :)","commit_id":"0f759fde8d5f46d548e56efd332cd2ff3753692f"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"5d4942a03ad2ff92766f38aa7fb096ad72b4083e","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    # NOTE(stevemar): we use a hybrid property here because we leverage the"},{"line_number":88,"context_line":"    # expression method, see `@password.expression` and `Password.password`"},{"line_number":89,"context_line":"    # below."},{"line_number":90,"context_line":"    @hybrid_property"},{"line_number":91,"context_line":"    def password(self):"},{"line_number":92,"context_line":"        \"\"\"Return the current password.\"\"\""},{"line_number":93,"context_line":"        if self.password_ref:"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a3c09a3_7a2ebea6","line":90,"range":{"start_line":90,"start_character":5,"end_line":90,"end_character":20},"in_reply_to":"7a3c09a3_fa0d4ee9","updated":"2017-01-18 22:26:32.000000000","message":"last time i changed a hybrid_property -\u003e property it blew up in the tests. if L79 needed to be hybrid the tests wouldn\u0027t pass (or it\u0027s untested)","commit_id":"0f759fde8d5f46d548e56efd332cd2ff3753692f"}]}
