)]}'
{"freezer_ui/api/api.py":[{"author":{"_account_id":39044,"name":"Alicja Filipek","display_name":"alaf01","email":"alicja.filipek@cleura.com","username":"alaf01"},"change_message_id":"8ba6c9decff0821199d51e30bc2264d254782747","unresolved":true,"context_lines":[{"line_number":520,"context_line":"        return [utils.BackupObject("},{"line_number":521,"context_line":"            backup_id\u003db.get(\u0027backup_id\u0027),"},{"line_number":522,"context_line":"            status\u003db.get(\u0027status\u0027) or b.get("},{"line_number":523,"context_line":"                \u0027backup_metadata\u0027, {}).get(\u0027status\u0027),"},{"line_number":524,"context_line":"            action\u003db.get(\u0027backup_metadata\u0027, {}).get(\u0027action\u0027),"},{"line_number":525,"context_line":"            time_stamp\u003db.get(\u0027backup_metadata\u0027, {}).get(\u0027time_stamp\u0027),"},{"line_number":526,"context_line":"            backup_name\u003db.get(\u0027backup_metadata\u0027, {}).get(\u0027backup_name\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"db3ea4f9_35a5f590","line":523,"updated":"2026-07-27 11:34:58.000000000","message":"What if we have different data in \u0027backup_metadata\u0027? Also if we have no data in \u0027status\u0027 field and some data in \u0027backup_metadata\u0027 I would treat it as a bug and at least log it as a warning.","commit_id":"a0b8887c07d843f1cdd4e14bf70ad694ace0737f"}],"freezer_ui/backups/views.py":[{"author":{"_account_id":39044,"name":"Alicja Filipek","display_name":"alaf01","email":"alicja.filipek@cleura.com","username":"alaf01"},"change_message_id":"8ba6c9decff0821199d51e30bc2264d254782747","unresolved":true,"context_lines":[{"line_number":48,"context_line":"        backup \u003d backup_api.get(kwargs[\u0027backup_id\u0027], json\u003dTrue)"},{"line_number":49,"context_line":"        metadata \u003d backup.get(\u0027backup_metadata\u0027, {})"},{"line_number":50,"context_line":"        if \u0027status\u0027 not in backup:"},{"line_number":51,"context_line":"            backup[\u0027status\u0027] \u003d metadata.get(\u0027status\u0027, \u0027available\u0027)"},{"line_number":52,"context_line":"        if \u0027time_stamp\u0027 in metadata and metadata[\u0027time_stamp\u0027]:"},{"line_number":53,"context_line":"            try:"},{"line_number":54,"context_line":"                metadata[\u0027time_stamp_formatted\u0027] \u003d ("}],"source_content_type":"text/x-python","patch_set":1,"id":"2f4ae3f2_a0a75ccc","line":51,"updated":"2026-07-27 11:34:58.000000000","message":"here also we should suspect something weird if the \u0027status\u0027 is not in the backup but is in metadata. I suggest log warning in here.","commit_id":"a0b8887c07d843f1cdd4e14bf70ad694ace0737f"}],"freezer_ui/tests/test_utils.py":[{"author":{"_account_id":39044,"name":"Alicja Filipek","display_name":"alaf01","email":"alicja.filipek@cleura.com","username":"alaf01"},"change_message_id":"8ba6c9decff0821199d51e30bc2264d254782747","unresolved":true,"context_lines":[{"line_number":27,"context_line":"        self.assertEqual(datetime_to_iso_string(dt), \u00272026-06-18T12:34:56\u0027)"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    def test_backup_object_status(self):"},{"line_number":30,"context_line":"        from freezer_ui.utils import BackupObject"},{"line_number":31,"context_line":"        obj1 \u003d BackupObject(backup_id\u003d\u0027b1\u0027)"},{"line_number":32,"context_line":"        self.assertEqual(obj1.status, \u0027available\u0027)"},{"line_number":33,"context_line":"        obj2 \u003d BackupObject(backup_id\u003d\u0027b2\u0027, status\u003d\u0027available\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"6acfe62f_dc685cfd","line":30,"updated":"2026-07-27 11:34:58.000000000","message":"why not add import on top? We already import from freezer_ui.utils","commit_id":"a0b8887c07d843f1cdd4e14bf70ad694ace0737f"},{"author":{"_account_id":39044,"name":"Alicja Filipek","display_name":"alaf01","email":"alicja.filipek@cleura.com","username":"alaf01"},"change_message_id":"8ba6c9decff0821199d51e30bc2264d254782747","unresolved":true,"context_lines":[{"line_number":30,"context_line":"        from freezer_ui.utils import BackupObject"},{"line_number":31,"context_line":"        obj1 \u003d BackupObject(backup_id\u003d\u0027b1\u0027)"},{"line_number":32,"context_line":"        self.assertEqual(obj1.status, \u0027available\u0027)"},{"line_number":33,"context_line":"        obj2 \u003d BackupObject(backup_id\u003d\u0027b2\u0027, status\u003d\u0027available\u0027)"},{"line_number":34,"context_line":"        self.assertEqual(obj2.status, \u0027available\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"94239f87_0fc86800","line":33,"updated":"2026-07-27 11:34:58.000000000","message":"If the available is the default, lets test some other case (\u0027created\u0027?) to be sure we not just passing default in here.","commit_id":"a0b8887c07d843f1cdd4e14bf70ad694ace0737f"}]}
