)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"480495aecebfd738c77c6662d6f15e81c8c70af4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"733ee86f_e6774396","updated":"2022-05-02 09:10:21.000000000","message":"Thank you, Francesco 😊\n\nAdded some comments below. Please add some tests so we know it works ^^\n\nMaybe it would make sense to implement is_domain parameter as well, documented e.g. here:\nhttps://docs.openstack.org/api-ref/identity/v3/index.html?expanded\u003dcreate-project-detail#create-project-detail","commit_id":"d07361941e5fed84940dee394f4004d8c82fc334"}],"plugins/modules/project.py":[{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"480495aecebfd738c77c6662d6f15e81c8c70af4","unresolved":true,"context_lines":[{"line_number":30,"context_line":"     type: str"},{"line_number":31,"context_line":"   parent_id:"},{"line_number":32,"context_line":"     description:"},{"line_number":33,"context_line":"        - Parent project of this project."},{"line_number":34,"context_line":"     aliases: [\u0027parent\u0027]"},{"line_number":35,"context_line":"     type: str"},{"line_number":36,"context_line":"   enabled:"}],"source_content_type":"text/x-python","patch_set":1,"id":"ef4e25f6_f0507bcc","line":33,"updated":"2022-05-02 09:10:21.000000000","message":"- Name or ID of the parent project","commit_id":"d07361941e5fed84940dee394f4004d8c82fc334"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"480495aecebfd738c77c6662d6f15e81c8c70af4","unresolved":true,"context_lines":[{"line_number":157,"context_line":"        name \u003d self.params[\u0027name\u0027]"},{"line_number":158,"context_line":"        description \u003d self.params[\u0027description\u0027]"},{"line_number":159,"context_line":"        domain \u003d self.params[\u0027domain_id\u0027]"},{"line_number":160,"context_line":"        creation_kwargs \u003d dict()"},{"line_number":161,"context_line":"        if \u0027parent_id\u0027 in self.params and self.params[\u0027parent_id\u0027] is not None:"},{"line_number":162,"context_line":"            creation_kwargs[\u0027parent_id\u0027] \u003d self.params[\u0027parent_id\u0027]"},{"line_number":163,"context_line":"        enabled \u003d self.params[\u0027enabled\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"d6c45024_03abe8e1","line":160,"updated":"2022-05-02 09:10:21.000000000","message":"parent \u003d and self.params[\u0027parent_id\u0027]\n\n(move kwargs dict downwards, see below)","commit_id":"d07361941e5fed84940dee394f4004d8c82fc334"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"480495aecebfd738c77c6662d6f15e81c8c70af4","unresolved":true,"context_lines":[{"line_number":188,"context_line":"            self.exit_json(changed\u003dself._system_state_change(project))"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"        if state \u003d\u003d \u0027present\u0027:"},{"line_number":191,"context_line":"            if project is None:"},{"line_number":192,"context_line":"                if creation_kwargs is {}:"},{"line_number":193,"context_line":"                    project \u003d self.conn.create_project("},{"line_number":194,"context_line":"                        name\u003dname, description\u003ddescription,"}],"source_content_type":"text/x-python","patch_set":1,"id":"c9b17b10_617ecc66","line":191,"updated":"2022-05-02 09:10:21.000000000","message":"Get id of parent project here because parent_id could be a name or an id. Compare to domain parameter for an analogy.","commit_id":"d07361941e5fed84940dee394f4004d8c82fc334"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"480495aecebfd738c77c6662d6f15e81c8c70af4","unresolved":true,"context_lines":[{"line_number":189,"context_line":""},{"line_number":190,"context_line":"        if state \u003d\u003d \u0027present\u0027:"},{"line_number":191,"context_line":"            if project is None:"},{"line_number":192,"context_line":"                if creation_kwargs is {}:"},{"line_number":193,"context_line":"                    project \u003d self.conn.create_project("},{"line_number":194,"context_line":"                        name\u003dname, description\u003ddescription,"},{"line_number":195,"context_line":"                        domain_id\u003ddomain,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fce3be9f_f9926d7a","line":192,"updated":"2022-05-02 09:10:21.000000000","message":"No need for this check here, one can simply pass an empty dict as kwargs to create_project(... **kwargs)","commit_id":"d07361941e5fed84940dee394f4004d8c82fc334"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"480495aecebfd738c77c6662d6f15e81c8c70af4","unresolved":true,"context_lines":[{"line_number":196,"context_line":"                        enabled\u003denabled)"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"                else:"},{"line_number":199,"context_line":"                    project \u003d self.conn.create_project("},{"line_number":200,"context_line":"                        name\u003dname, description\u003ddescription,"},{"line_number":201,"context_line":"                        domain_id\u003ddomain,"},{"line_number":202,"context_line":"                        enabled\u003denabled,"}],"source_content_type":"text/x-python","patch_set":1,"id":"07cdf334_0f87bb91","line":199,"updated":"2022-05-02 09:10:21.000000000","message":"Create kwargs here and add parent to it if it is not None. Compare to code which handles domain(_id) parameter.","commit_id":"d07361941e5fed84940dee394f4004d8c82fc334"}]}
