orthanc: OrthancServer/Sources/OrthancRestApi/OrthancRestAnonymizeModify.cpp annotate

annotate OrthancServer/Sources/OrthancRestApi/OrthancRestAnonymizeModify.cpp @ 4879:c728d8f25e5a

fix doc

author	Alain Mazy <am@osimis.io>
date	Wed, 19 Jan 2022 17:05:24 +0100
parents	7053502fbf97
children	2e71a08eea15 43e613a7756b

rev	line source
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	1 /**
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	2 * Orthanc - A Lightweight, RESTful DICOM Store
1900 b1291df2f780 2016 Sebastien Jodogne <s.jodogne@gmail.com> parents: 1805 diff changeset	3 * Copyright (C) 2012-2016 Sebastien Jodogne, Medical Physics
1288 6e7e5ed91c2d upgrade to year 2015 Sebastien Jodogne <s.jodogne@gmail.com> parents: 1279 diff changeset	4 * Department, University Hospital of Liege, Belgium
4437 d9473bd5ed43 upgrade to year 2021 Sebastien Jodogne <s.jodogne@gmail.com> parents: 4435 diff changeset	5 * Copyright (C) 2017-2021 Osimis S.A., Belgium
4831 7053502fbf97 added copyright UCLouvain Sebastien Jodogne <s.jodogne@gmail.com> parents: 4796 diff changeset	6 * Copyright (C) 2021-2021 Sebastien Jodogne, ICTEAM UCLouvain, Belgium
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	7 *
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	8 * This program is free software: you can redistribute it and/or
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	9 * modify it under the terms of the GNU General Public License as
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	10 * published by the Free Software Foundation, either version 3 of the
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	11 * License, or (at your option) any later version.
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	12 *
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	13 * In addition, as a special exception, the copyright holders of this
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	14 * program give permission to link the code of its release with the
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	15 * OpenSSL project's "OpenSSL" library (or with modified versions of it
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	16 * that use the same license as the "OpenSSL" library), and distribute
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	17 * the linked executables. You must obey the GNU General Public License
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	18 * in all respects for all of the code used other than "OpenSSL". If you
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	19 * modify file(s) with this exception, you may extend this exception to
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	20 * your version of the file(s), but you are not obligated to do so. If
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	21 * you do not wish to do so, delete this exception statement from your
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	22 * version. If you delete this exception statement from all source files
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	23 * in the program, then also delete it here.
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	24 *
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	25 * This program is distributed in the hope that it will be useful, but
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	26 * WITHOUT ANY WARRANTY; without even the implied warranty of
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	27 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	28 * General Public License for more details.
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	29 *
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	30 * You should have received a copy of the GNU General Public License
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	31 * along with this program. If not, see <http://www.gnu.org/licenses/>.
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	32 **/
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	33
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	34
831 84513f2ee1f3 pch for unit tests and server Sebastien Jodogne <s.jodogne@gmail.com> parents: 824 diff changeset	35 #include "../PrecompiledHeadersServer.h"
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	36 #include "OrthancRestApi.h"
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	37
4045 05b8fd21089c fix path Sebastien Jodogne <s.jodogne@gmail.com> parents: 4044 diff changeset	38 #include "../../../OrthancFramework/Sources/DicomParsing/FromDcmtkBridge.h"
05b8fd21089c fix path Sebastien Jodogne <s.jodogne@gmail.com> parents: 4044 diff changeset	39 #include "../../../OrthancFramework/Sources/Logging.h"
05b8fd21089c fix path Sebastien Jodogne <s.jodogne@gmail.com> parents: 4044 diff changeset	40 #include "../../../OrthancFramework/Sources/SerializationToolbox.h"
3709 1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	41 #include "../OrthancConfiguration.h"
1437 02f5a3f5c0a0 access to the REST API from Lua Sebastien Jodogne <s.jodogne@gmail.com> parents: 1288 diff changeset	42 #include "../ServerContext.h"
2853 52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	43 #include "../ServerJobs/MergeStudyJob.h"
2642 ccc470091ea6 reorganization Sebastien Jodogne <s.jodogne@gmail.com> parents: 2640 diff changeset	44 #include "../ServerJobs/ResourceModificationJob.h"
2844 99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	45 #include "../ServerJobs/SplitStudyJob.h"
787 ac18946afa74 refactoring of anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 782 diff changeset	46
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	47 #include <boost/lexical_cast.hpp>
1641 4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	48 #include <boost/algorithm/string/predicate.hpp>
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	49
4687 fcd2dc7c8f31 "Replace", "Keep" and "Remove" in "/modify" and "/anonymize" accept paths to subsequences Sebastien Jodogne <s.jodogne@gmail.com> parents: 4607 diff changeset	50 #define INFO_SUBSEQUENCES \
fcd2dc7c8f31 "Replace", "Keep" and "Remove" in "/modify" and "/anonymize" accept paths to subsequences Sebastien Jodogne <s.jodogne@gmail.com> parents: 4607 diff changeset	51 "Starting with Orthanc 1.9.4, paths to subsequences can be provided using the "\
fcd2dc7c8f31 "Replace", "Keep" and "Remove" in "/modify" and "/anonymize" accept paths to subsequences Sebastien Jodogne <s.jodogne@gmail.com> parents: 4607 diff changeset	52 "same syntax as the `dcmodify` command-line tool (wildcards are supported as well)."
fcd2dc7c8f31 "Replace", "Keep" and "Remove" in "/modify" and "/anonymize" accept paths to subsequences Sebastien Jodogne <s.jodogne@gmail.com> parents: 4607 diff changeset	53
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	54
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	55 static const char* const CONTENT = "Content";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	56 static const char* const FORCE = "Force";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	57 static const char* const INSTANCES = "Instances";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	58 static const char* const INTERPRET_BINARY_TAGS = "InterpretBinaryTags";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	59 static const char* const KEEP = "Keep";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	60 static const char* const KEEP_PRIVATE_TAGS = "KeepPrivateTags";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	61 static const char* const KEEP_SOURCE = "KeepSource";
4766 388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	62 static const char* const LEVEL = "Level";
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	63 static const char* const PARENT = "Parent";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	64 static const char* const PRIVATE_CREATOR = "PrivateCreator";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	65 static const char* const REMOVE = "Remove";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	66 static const char* const REPLACE = "Replace";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	67 static const char* const RESOURCES = "Resources";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	68 static const char* const SERIES = "Series";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	69 static const char* const TAGS = "Tags";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	70 static const char* const TRANSCODE = "Transcode";
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	71
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	72
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	73 namespace Orthanc
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	74 {
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	75 // Modification of DICOM instances ------------------------------------------
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	76
2519 2e6b7862ccf2 ParseAnonymizationRequest/ParseModifyRequest now in DicomModification Sebastien Jodogne <s.jodogne@gmail.com> parents: 2518 diff changeset	77
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	78 static std::string GeneratePatientName(ServerContext& context)
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	79 {
4607 f75c63aa9de0 differentiating between shared and private global properties Sebastien Jodogne <s.jodogne@gmail.com> parents: 4508 diff changeset	80 uint64_t seq = context.GetIndex().IncrementGlobalSequence(GlobalProperty_AnonymizationSequence, true /* shared */);
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	81 return "Anonymized" + boost::lexical_cast<std::string>(seq);
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	82 }
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	83
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	84
4766 388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	85 static void DocumentKeepSource(RestApiPostCall& call)
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	86 {
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	87 call.GetDocumentation()
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	88 .SetRequestField(KEEP_SOURCE, RestApiCallDocumentation::Type_Boolean,
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	89 "If set to `false`, instructs Orthanc to the remove original resources. "
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	90 "By default, the original resources are kept in Orthanc.", false);
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	91 }
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	92
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	93
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	94 static void DocumentModifyOptions(RestApiPostCall& call)
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	95 {
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	96 // Check out "DicomModification::ParseModifyRequest()"
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	97 call.GetDocumentation()
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	98 .SetRequestField(TRANSCODE, RestApiCallDocumentation::Type_String,
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	99 "Transcode the DICOM instances to the provided DICOM transfer syntax: "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	100 "https://book.orthanc-server.com/faq/transcoding.html", false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	101 .SetRequestField(FORCE, RestApiCallDocumentation::Type_Boolean,
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	102 "Allow the modification of tags related to DICOM identifiers, at the risk of "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	103 "breaking the DICOM model of the real world", false)
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	104 .SetRequestField("RemovePrivateTags", RestApiCallDocumentation::Type_Boolean,
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	105 "Remove the private tags from the DICOM instances (defaults to `false`)", false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	106 .SetRequestField(REPLACE, RestApiCallDocumentation::Type_JsonObject,
4687 fcd2dc7c8f31 "Replace", "Keep" and "Remove" in "/modify" and "/anonymize" accept paths to subsequences Sebastien Jodogne <s.jodogne@gmail.com> parents: 4607 diff changeset	107 "Associative array to change the value of some DICOM tags in the DICOM instances. " INFO_SUBSEQUENCES, false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	108 .SetRequestField(REMOVE, RestApiCallDocumentation::Type_JsonListOfStrings,
4687 fcd2dc7c8f31 "Replace", "Keep" and "Remove" in "/modify" and "/anonymize" accept paths to subsequences Sebastien Jodogne <s.jodogne@gmail.com> parents: 4607 diff changeset	109 "List of tags that must be removed from the DICOM instances. " INFO_SUBSEQUENCES, false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	110 .SetRequestField(KEEP, RestApiCallDocumentation::Type_JsonListOfStrings,
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	111 "Keep the original value of the specified tags, to be chosen among the `StudyInstanceUID`, "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	112 "`SeriesInstanceUID` and `SOPInstanceUID` tags. Avoid this feature as much as possible, "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	113 "as this breaks the DICOM model of the real world.", false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	114 .SetRequestField(PRIVATE_CREATOR, RestApiCallDocumentation::Type_String,
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	115 "The private creator to be used for private tags in `Replace`", false);
4766 388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	116
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	117 // This was existing, but undocumented in Orthanc <= 1.9.6
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	118 DocumentKeepSource(call);
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	119 }
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	120
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	121
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	122 static void DocumentAnonymizationOptions(RestApiPostCall& call)
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	123 {
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	124 // Check out "DicomModification::ParseAnonymizationRequest()"
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	125 call.GetDocumentation()
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	126 .SetRequestField(FORCE, RestApiCallDocumentation::Type_Boolean,
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	127 "Allow the modification of tags related to DICOM identifiers, at the risk of "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	128 "breaking the DICOM model of the real world", false)
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	129 .SetRequestField("DicomVersion", RestApiCallDocumentation::Type_String,
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	130 "Version of the DICOM standard to be used for anonymization. Check out "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	131 "configuration option `DeidentifyLogsDicomVersion` for possible values.", false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	132 .SetRequestField(KEEP_PRIVATE_TAGS, RestApiCallDocumentation::Type_Boolean,
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	133 "Keep the private tags from the DICOM instances (defaults to `false`)", false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	134 .SetRequestField(REPLACE, RestApiCallDocumentation::Type_JsonObject,
4687 fcd2dc7c8f31 "Replace", "Keep" and "Remove" in "/modify" and "/anonymize" accept paths to subsequences Sebastien Jodogne <s.jodogne@gmail.com> parents: 4607 diff changeset	135 "Associative array to change the value of some DICOM tags in the DICOM instances. " INFO_SUBSEQUENCES, false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	136 .SetRequestField(REMOVE, RestApiCallDocumentation::Type_JsonListOfStrings,
4687 fcd2dc7c8f31 "Replace", "Keep" and "Remove" in "/modify" and "/anonymize" accept paths to subsequences Sebastien Jodogne <s.jodogne@gmail.com> parents: 4607 diff changeset	137 "List of additional tags to be removed from the DICOM instances. " INFO_SUBSEQUENCES, false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	138 .SetRequestField(KEEP, RestApiCallDocumentation::Type_JsonListOfStrings,
4687 fcd2dc7c8f31 "Replace", "Keep" and "Remove" in "/modify" and "/anonymize" accept paths to subsequences Sebastien Jodogne <s.jodogne@gmail.com> parents: 4607 diff changeset	139 "List of DICOM tags whose value must not be destroyed by the anonymization. " INFO_SUBSEQUENCES, false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	140 .SetRequestField(PRIVATE_CREATOR, RestApiCallDocumentation::Type_String,
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	141 "The private creator to be used for private tags in `Replace`", false);
4766 388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	142
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	143 // This was existing, but undocumented in Orthanc <= 1.9.6
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	144 DocumentKeepSource(call);
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	145 }
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	146
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	147
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	148 static void ParseModifyRequest(Json::Value& request,
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	149 DicomModification& target,
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	150 const RestApiPostCall& call)
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	151 {
2640 c691fcf66071 ResourceModificationJob Sebastien Jodogne <s.jodogne@gmail.com> parents: 2639 diff changeset	152 // curl http://localhost:8042/series/95a6e2bf-9296e2cc-bf614e2f-22b391ee-16e010e0/modify -X POST -d '{"Replace":{"InstitutionName":"My own clinic"},"Priority":9}'
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	153
3709 1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	154 {
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	155 OrthancConfiguration::ReaderLock lock;
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	156 target.SetPrivateCreator(lock.GetConfiguration().GetDefaultPrivateCreator());
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	157 }
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	158
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	159 if (call.ParseJsonRequest(request))
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	160 {
2519 2e6b7862ccf2 ParseAnonymizationRequest/ParseModifyRequest now in DicomModification Sebastien Jodogne <s.jodogne@gmail.com> parents: 2518 diff changeset	161 target.ParseModifyRequest(request);
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	162 }
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	163 else
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	164 {
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	165 throw OrthancException(ErrorCode_BadFileFormat);
2309 4dc313b9a20a Argument "DicomVersion" in URIs "/{...}/{...}/anonymization" Sebastien Jodogne <s.jodogne@gmail.com> parents: 2250 diff changeset	166 }
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	167 }
2309 4dc313b9a20a Argument "DicomVersion" in URIs "/{...}/{...}/anonymization" Sebastien Jodogne <s.jodogne@gmail.com> parents: 2250 diff changeset	168
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	169
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	170 static void ParseAnonymizationRequest(Json::Value& request,
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	171 DicomModification& target,
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	172 RestApiPostCall& call)
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	173 {
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	174 // curl http://localhost:8042/instances/6e67da51-d119d6ae-c5667437-87b9a8a5-0f07c49f/anonymize -X POST -d '{"Replace":{"PatientName":"hello","0010-0020":"world"},"Keep":["StudyDescription", "SeriesDescription"],"KeepPrivateTags": true,"Remove":["Modality"]}' > Anonymized.dcm
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	175
3709 1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	176 {
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	177 OrthancConfiguration::ReaderLock lock;
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	178 target.SetPrivateCreator(lock.GetConfiguration().GetDefaultPrivateCreator());
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	179 }
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	180
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	181 if (call.ParseJsonRequest(request) &&
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	182 request.isObject())
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	183 {
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	184 bool patientNameOverridden;
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	185 target.ParseAnonymizationRequest(patientNameOverridden, request);
2518 63d2cc0fb40a refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2517 diff changeset	186
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	187 if (!patientNameOverridden)
2518 63d2cc0fb40a refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2517 diff changeset	188 {
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	189 // Override the random Patient's Name by one that is more
2518 63d2cc0fb40a refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2517 diff changeset	190 // user-friendly (provided none was specified by the user)
63d2cc0fb40a refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2517 diff changeset	191 target.Replace(DICOM_TAG_PATIENT_NAME, GeneratePatientName(OrthancRestApi::GetContext(call)), true);
63d2cc0fb40a refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2517 diff changeset	192 }
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	193 }
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	194 else
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	195 {
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	196 throw OrthancException(ErrorCode_BadFileFormat);
d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	197 }
787 ac18946afa74 refactoring of anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 782 diff changeset	198 }
ac18946afa74 refactoring of anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 782 diff changeset	199
ac18946afa74 refactoring of anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 782 diff changeset	200
ac18946afa74 refactoring of anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 782 diff changeset	201 static void AnonymizeOrModifyInstance(DicomModification& modification,
3901 603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	202 RestApiPostCall& call,
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	203 bool transcode,
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	204 DicomTransferSyntax targetSyntax)
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	205 {
3901 603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	206 ServerContext& context = OrthancRestApi::GetContext(call);
778 aebf0071020e refactoring of the mutex for the dicom cache Sebastien Jodogne <s.jodogne@gmail.com> parents: 751 diff changeset	207 std::string id = call.GetUriComponent("id", "");
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	208
3712 2a170a8f1faf replacing std::auto_ptr by std::unique_ptr Sebastien Jodogne <s.jodogne@gmail.com> parents: 3709 diff changeset	209 std::unique_ptr<ParsedDicomFile> modified;
778 aebf0071020e refactoring of the mutex for the dicom cache Sebastien Jodogne <s.jodogne@gmail.com> parents: 751 diff changeset	210
2825 8aa6aef11b70 New configuration option "OverwriteInstances" to choose how duplicate SOPInstanceUID are handled Sebastien Jodogne <s.jodogne@gmail.com> parents: 2664 diff changeset	211 {
3901 603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	212 ServerContext::DicomCacheLocker locker(context, id);
2825 8aa6aef11b70 New configuration option "OverwriteInstances" to choose how duplicate SOPInstanceUID are handled Sebastien Jodogne <s.jodogne@gmail.com> parents: 2664 diff changeset	213 modified.reset(locker.GetDicom().Clone(true));
8aa6aef11b70 New configuration option "OverwriteInstances" to choose how duplicate SOPInstanceUID are handled Sebastien Jodogne <s.jodogne@gmail.com> parents: 2664 diff changeset	214 }
8aa6aef11b70 New configuration option "OverwriteInstances" to choose how duplicate SOPInstanceUID are handled Sebastien Jodogne <s.jodogne@gmail.com> parents: 2664 diff changeset	215
787 ac18946afa74 refactoring of anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 782 diff changeset	216 modification.Apply(*modified);
3901 603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	217
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	218 if (transcode)
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	219 {
3945 0b3256c3ee14 simplified IDicomTranscoder Sebastien Jodogne <s.jodogne@gmail.com> parents: 3943 diff changeset	220 IDicomTranscoder::DicomImage source;
0b3256c3ee14 simplified IDicomTranscoder Sebastien Jodogne <s.jodogne@gmail.com> parents: 3943 diff changeset	221 source.AcquireParsed(*modified); // "modified" is invalid below this point
0b3256c3ee14 simplified IDicomTranscoder Sebastien Jodogne <s.jodogne@gmail.com> parents: 3943 diff changeset	222
0b3256c3ee14 simplified IDicomTranscoder Sebastien Jodogne <s.jodogne@gmail.com> parents: 3943 diff changeset	223 IDicomTranscoder::DicomImage transcoded;
3903 d1273d7cc200 avoid unnecessary dicom serialization Sebastien Jodogne <s.jodogne@gmail.com> parents: 3901 diff changeset	224
3945 0b3256c3ee14 simplified IDicomTranscoder Sebastien Jodogne <s.jodogne@gmail.com> parents: 3943 diff changeset	225 std::set<DicomTransferSyntax> s;
0b3256c3ee14 simplified IDicomTranscoder Sebastien Jodogne <s.jodogne@gmail.com> parents: 3943 diff changeset	226 s.insert(targetSyntax);
0b3256c3ee14 simplified IDicomTranscoder Sebastien Jodogne <s.jodogne@gmail.com> parents: 3943 diff changeset	227
3951 5fe8c6d3212e removed useless information "hasSopInstanceUidChanged" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3945 diff changeset	228 if (context.Transcode(transcoded, source, s, true))
3901 603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	229 {
3945 0b3256c3ee14 simplified IDicomTranscoder Sebastien Jodogne <s.jodogne@gmail.com> parents: 3943 diff changeset	230 call.GetOutput().AnswerBuffer(transcoded.GetBufferData(),
0b3256c3ee14 simplified IDicomTranscoder Sebastien Jodogne <s.jodogne@gmail.com> parents: 3943 diff changeset	231 transcoded.GetBufferSize(), MimeType_Dicom);
3901 603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	232 }
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	233 else
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	234 {
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	235 throw OrthancException(ErrorCode_InternalError,
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	236 "Cannot transcode to transfer syntax: " +
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	237 std::string(GetTransferSyntaxUid(targetSyntax)));
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	238 }
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	239 }
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	240 else
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	241 {
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	242 modified->Answer(call.GetOutput());
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	243 }
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	244 }
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	245
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	246
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	247 static ResourceType DetectModifyLevel(const DicomModification& modification)
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	248 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	249 if (modification.IsReplaced(DICOM_TAG_PATIENT_ID))
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	250 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	251 return ResourceType_Patient;
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	252 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	253 else if (modification.IsReplaced(DICOM_TAG_STUDY_INSTANCE_UID))
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	254 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	255 return ResourceType_Study;
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	256 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	257 else if (modification.IsReplaced(DICOM_TAG_SERIES_INSTANCE_UID))
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	258 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	259 return ResourceType_Series;
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	260 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	261 else
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	262 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	263 return ResourceType_Instance;
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	264 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	265 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	266
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	267
974 83622b0f544c refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 831 diff changeset	268 static void ModifyInstance(RestApiPostCall& call)
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	269 {
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	270 if (call.IsDocumentation())
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	271 {
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	272 DocumentModifyOptions(call);
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	273 call.GetDocumentation()
4435 23ad1b9c7800 fix tag in openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4424 diff changeset	274 .SetTag("Instances")
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	275 .SetSummary("Modify instance")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	276 .SetDescription("Download a modified version of the DICOM instance whose Orthanc identifier is provided in the URL: "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	277 "https://book.orthanc-server.com/users/anonymization.html#modification-of-a-single-instance")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	278 .SetUriArgument("id", "Orthanc identifier of the instance of interest")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	279 .AddAnswerType(MimeType_Dicom, "The modified DICOM instance");
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	280 return;
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	281 }
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	282
787 ac18946afa74 refactoring of anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 782 diff changeset	283 DicomModification modification;
1279 7f3a65e84d4b More flexible /modify and /anonymize for single instance Sebastien Jodogne <s.jodogne@gmail.com> parents: 1160 diff changeset	284 modification.SetAllowManualIdentifiers(true);
787 ac18946afa74 refactoring of anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 782 diff changeset	285
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	286 Json::Value request;
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	287 ParseModifyRequest(request, modification, call);
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	288
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	289 modification.SetLevel(DetectModifyLevel(modification));
796 e7b1ca0f1e04 creation of dicom files Sebastien Jodogne <s.jodogne@gmail.com> parents: 795 diff changeset	290
3943 b26d25d3c1c7 "/{patients\|studies\|series}/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3942 diff changeset	291 if (request.isMember(TRANSCODE))
3901 603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	292 {
3943 b26d25d3c1c7 "/{patients\|studies\|series}/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3942 diff changeset	293 std::string s = SerializationToolbox::ReadString(request, TRANSCODE);
3901 603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	294
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	295 DicomTransferSyntax syntax;
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	296 if (LookupTransferSyntax(syntax, s))
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	297 {
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	298 AnonymizeOrModifyInstance(modification, call, true, syntax);
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	299 }
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	300 else
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	301 {
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	302 throw OrthancException(ErrorCode_ParameterOutOfRange, "Unknown transfer syntax: " + s);
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	303 }
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	304 }
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	305 else
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	306 {
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	307 AnonymizeOrModifyInstance(modification, call, false /* no transcoding */,
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	308 DicomTransferSyntax_LittleEndianImplicit /* unused */);
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	309 }
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	310 }
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	311
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	312
974 83622b0f544c refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 831 diff changeset	313 static void AnonymizeInstance(RestApiPostCall& call)
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	314 {
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	315 if (call.IsDocumentation())
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	316 {
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	317 DocumentAnonymizationOptions(call);
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	318 call.GetDocumentation()
4435 23ad1b9c7800 fix tag in openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4424 diff changeset	319 .SetTag("Instances")
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	320 .SetSummary("Anonymize instance")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	321 .SetDescription("Download an anonymized version of the DICOM instance whose Orthanc identifier is provided in the URL: "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	322 "https://book.orthanc-server.com/users/anonymization.html#anonymization-of-a-single-instance")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	323 .SetUriArgument("id", "Orthanc identifier of the instance of interest")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	324 .AddAnswerType(MimeType_Dicom, "The anonymized DICOM instance");
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	325 return;
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	326 }
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	327
787 ac18946afa74 refactoring of anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 782 diff changeset	328 DicomModification modification;
1279 7f3a65e84d4b More flexible /modify and /anonymize for single instance Sebastien Jodogne <s.jodogne@gmail.com> parents: 1160 diff changeset	329 modification.SetAllowManualIdentifiers(true);
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	330
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	331 Json::Value request;
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	332 ParseAnonymizationRequest(request, modification, call);
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	333
3901 603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	334 AnonymizeOrModifyInstance(modification, call, false /* no transcoding */,
603a7b86fa5f route "/instances/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3814 diff changeset	335 DicomTransferSyntax_LittleEndianImplicit /* unused */);
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	336 }
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	337
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	338
3942 5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	339 static void SetKeepSource(CleaningInstancesJob& job,
5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	340 const Json::Value& body)
5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	341 {
5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	342 if (body.isMember(KEEP_SOURCE))
5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	343 {
5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	344 job.SetKeepSource(SerializationToolbox::ReadBoolean(body, KEEP_SOURCE));
5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	345 }
5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	346 }
5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	347
5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	348
3712 2a170a8f1faf replacing std::auto_ptr by std::unique_ptr Sebastien Jodogne <s.jodogne@gmail.com> parents: 3709 diff changeset	349 static void SubmitModificationJob(std::unique_ptr<DicomModification>& modification,
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	350 bool isAnonymization,
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	351 RestApiPostCall& call,
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	352 const Json::Value& body,
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	353 ResourceType outputLevel /* unused for multiple resources */,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	354 bool isSingleResource,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	355 const std::set<std::string>& resources)
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	356 {
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	357 ServerContext& context = OrthancRestApi::GetContext(call);
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	358
3712 2a170a8f1faf replacing std::auto_ptr by std::unique_ptr Sebastien Jodogne <s.jodogne@gmail.com> parents: 3709 diff changeset	359 std::unique_ptr<ResourceModificationJob> job(new ResourceModificationJob(context));
3942 5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	360
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	361 if (isSingleResource) // This notably configures the output format
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	362 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	363 job->SetSingleResourceModification(modification.release(), outputLevel, isAnonymization);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	364 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	365 else
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	366 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	367 job->SetMultipleResourcesModification(modification.release(), isAnonymization);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	368 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	369
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	370 job->SetOrigin(call);
3942 5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	371 SetKeepSource(*job, body);
3943 b26d25d3c1c7 "/{patients\|studies\|series}/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3942 diff changeset	372
b26d25d3c1c7 "/{patients\|studies\|series}/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3942 diff changeset	373 if (body.isMember(TRANSCODE))
b26d25d3c1c7 "/{patients\|studies\|series}/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3942 diff changeset	374 {
b26d25d3c1c7 "/{patients\|studies\|series}/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3942 diff changeset	375 job->SetTranscode(SerializationToolbox::ReadString(body, TRANSCODE));
b26d25d3c1c7 "/{patients\|studies\|series}/.../modify": New option "Transcode" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3942 diff changeset	376 }
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	377
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	378 for (std::set<std::string>::const_iterator
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	379 it = resources.begin(); it != resources.end(); ++it)
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	380 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	381 context.AddChildInstances(job, it);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	382 }
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	383
3942 5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	384 job->AddTrailingStep();
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	385
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	386 OrthancRestApi::GetApi(call).SubmitCommandsJob
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	387 (call, job.release(), true /* synchronous by default */, body);
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	388 }
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	389
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	390
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	391 static void SubmitModificationJob(std::unique_ptr<DicomModification>& modification,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	392 bool isAnonymization,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	393 RestApiPostCall& call,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	394 const Json::Value& body,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	395 ResourceType outputLevel)
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	396 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	397 // This was the only flavor in Orthanc <= 1.9.3
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	398 std::set<std::string> resources;
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	399 resources.insert(call.GetUriComponent("id", ""));
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	400
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	401 SubmitModificationJob(modification, isAnonymization, call, body, outputLevel,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	402 true /* single resource */, resources);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	403 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	404
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	405
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	406 static void SubmitBulkJob(std::unique_ptr<DicomModification>& modification,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	407 bool isAnonymization,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	408 RestApiPostCall& call,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	409 const Json::Value& body)
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	410 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	411 std::set<std::string> resources;
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	412 SerializationToolbox::ReadSetOfStrings(resources, body, RESOURCES);
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	413
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	414 SubmitModificationJob(modification, isAnonymization,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	415 call, body, ResourceType_Instance /* arbitrary value, unused */,
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	416 false /* multiple resources */, resources);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	417 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	418
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	419
2639 75a404e40323 simplification Sebastien Jodogne <s.jodogne@gmail.com> parents: 2521 diff changeset	420 template <enum ResourceType resourceType>
974 83622b0f544c refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 831 diff changeset	421 static void ModifyResource(RestApiPostCall& call)
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	422 {
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	423 if (call.IsDocumentation())
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	424 {
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	425 OrthancRestApi::DocumentSubmitCommandsJob(call);
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	426 DocumentModifyOptions(call);
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	427 const std::string r = GetResourceTypeText(resourceType, false /* plural /, false / lower case */);
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	428 call.GetDocumentation()
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	429 .SetTag(GetResourceTypeText(resourceType, true /* plural /, true / upper case */))
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	430 .SetSummary("Modify " + r)
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	431 .SetDescription("Start a job that will modify all the DICOM instances within the " + r +
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	432 " whose identifier is provided in the URL. The modified DICOM instances will be "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	433 "stored into a brand new " + r + ", whose Orthanc identifiers will be returned by the job. "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	434 "https://book.orthanc-server.com/users/anonymization.html#modification-of-studies-or-series")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	435 .SetUriArgument("id", "Orthanc identifier of the " + r + " of interest");
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	436 return;
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	437 }
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	438
3712 2a170a8f1faf replacing std::auto_ptr by std::unique_ptr Sebastien Jodogne <s.jodogne@gmail.com> parents: 3709 diff changeset	439 std::unique_ptr<DicomModification> modification(new DicomModification);
2640 c691fcf66071 ResourceModificationJob Sebastien Jodogne <s.jodogne@gmail.com> parents: 2639 diff changeset	440
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	441 Json::Value body;
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	442 ParseModifyRequest(body, *modification, call);
2517 d3d306bc4b99 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2475 diff changeset	443
2639 75a404e40323 simplification Sebastien Jodogne <s.jodogne@gmail.com> parents: 2521 diff changeset	444 modification->SetLevel(resourceType);
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	445
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	446 SubmitModificationJob(modification, false /* not an anonymization */,
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	447 call, body, resourceType);
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	448 }
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	449
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	450
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	451 // New in Orthanc 1.9.4
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	452 static void BulkModify(RestApiPostCall& call)
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	453 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	454 if (call.IsDocumentation())
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	455 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	456 OrthancRestApi::DocumentSubmitCommandsJob(call);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	457 DocumentModifyOptions(call);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	458 call.GetDocumentation()
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	459 .SetTag("System")
4694 da1edb7d6332 "/tools/bulk-delete" to delete a group of multiple, unrelated resources at once Sebastien Jodogne <s.jodogne@gmail.com> parents: 4693 diff changeset	460 .SetSummary("Modify a set of resources")
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	461 .SetRequestField(RESOURCES, RestApiCallDocumentation::Type_JsonListOfStrings,
4699 facea16b055b added the "Level" argument to "/tools/bulk-content" Sebastien Jodogne <s.jodogne@gmail.com> parents: 4694 diff changeset	462 "List of the Orthanc identifiers of the patients/studies/series/instances of interest.", true)
4766 388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	463 .SetRequestField(LEVEL, RestApiCallDocumentation::Type_String,
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	464 "Level of the modification (`Patient`, `Study`, `Series` or `Instance`). If absent, "
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	465 "the level defaults to `Instance`, but is set to `Patient` if `PatientID` is modified, "
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	466 "to `Study` if `StudyInstanceUID` is modified, or to `Series` if `SeriesInstancesUID` "
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	467 "is modified. (new in Orthanc 1.9.7)", false)
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	468 .SetDescription("Start a job that will modify all the DICOM patients, studies, series or instances "
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	469 "whose identifiers are provided in the `Resources` field.")
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	470 .AddAnswerType(MimeType_Json, "The list of all the resources that have been altered by this modification");
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	471 return;
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	472 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	473
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	474 std::unique_ptr<DicomModification> modification(new DicomModification);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	475
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	476 Json::Value body;
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	477 ParseModifyRequest(body, *modification, call);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	478
4766 388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	479 if (body.isMember(LEVEL))
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	480 {
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	481 // This case was introduced in Orthanc 1.9.7
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	482 modification->SetLevel(StringToResourceType(body[LEVEL].asCString()));
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	483 }
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	484 else
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	485 {
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	486 modification->SetLevel(DetectModifyLevel(*modification));
388d108f6e4b Added "Level" option to POST /tools/bulk-modify Sebastien Jodogne <s.jodogne@gmail.com> parents: 4715 diff changeset	487 }
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	488
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	489 SubmitBulkJob(modification, false /* not an anonymization */, call, body);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	490 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	491
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	492
2639 75a404e40323 simplification Sebastien Jodogne <s.jodogne@gmail.com> parents: 2521 diff changeset	493 template <enum ResourceType resourceType>
974 83622b0f544c refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 831 diff changeset	494 static void AnonymizeResource(RestApiPostCall& call)
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	495 {
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	496 if (call.IsDocumentation())
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	497 {
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	498 OrthancRestApi::DocumentSubmitCommandsJob(call);
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	499 DocumentAnonymizationOptions(call);
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	500 const std::string r = GetResourceTypeText(resourceType, false /* plural /, false / lower case */);
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	501 call.GetDocumentation()
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	502 .SetTag(GetResourceTypeText(resourceType, true /* plural /, true / upper case */))
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	503 .SetSummary("Anonymize " + r)
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	504 .SetDescription("Start a job that will anonymize all the DICOM instances within the " + r +
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	505 " whose identifier is provided in the URL. The modified DICOM instances will be "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	506 "stored into a brand new " + r + ", whose Orthanc identifiers will be returned by the job. "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	507 "https://book.orthanc-server.com/users/anonymization.html#anonymization-of-patients-studies-or-series")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	508 .SetUriArgument("id", "Orthanc identifier of the " + r + " of interest");
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	509 return;
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	510 }
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	511
3712 2a170a8f1faf replacing std::auto_ptr by std::unique_ptr Sebastien Jodogne <s.jodogne@gmail.com> parents: 3709 diff changeset	512 std::unique_ptr<DicomModification> modification(new DicomModification);
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	513
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	514 Json::Value body;
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	515 ParseAnonymizationRequest(body, *modification, call);
2640 c691fcf66071 ResourceModificationJob Sebastien Jodogne <s.jodogne@gmail.com> parents: 2639 diff changeset	516
2868 abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	517 SubmitModificationJob(modification, true /* anonymization */,
abce036683cd sharing code within OrthancRestAnonymizeModify Sebastien Jodogne <s.jodogne@gmail.com> parents: 2867 diff changeset	518 call, body, resourceType);
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	519 }
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	520
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	521
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	522 // New in Orthanc 1.9.4
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	523 static void BulkAnonymize(RestApiPostCall& call)
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	524 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	525 if (call.IsDocumentation())
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	526 {
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	527 OrthancRestApi::DocumentSubmitCommandsJob(call);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	528 DocumentAnonymizationOptions(call);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	529 call.GetDocumentation()
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	530 .SetTag("System")
4694 da1edb7d6332 "/tools/bulk-delete" to delete a group of multiple, unrelated resources at once Sebastien Jodogne <s.jodogne@gmail.com> parents: 4693 diff changeset	531 .SetSummary("Anonymize a set of resources")
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	532 .SetRequestField(RESOURCES, RestApiCallDocumentation::Type_JsonListOfStrings,
4699 facea16b055b added the "Level" argument to "/tools/bulk-content" Sebastien Jodogne <s.jodogne@gmail.com> parents: 4694 diff changeset	533 "List of the Orthanc identifiers of the patients/studies/series/instances of interest.", true)
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	534 .SetDescription("Start a job that will anonymize all the DICOM patients, studies, series or instances "
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	535 "whose identifiers are provided in the `Resources` field.")
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	536 .AddAnswerType(MimeType_Json, "The list of all the resources that have been created by this anonymization");
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	537 return;
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	538 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	539
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	540 std::unique_ptr<DicomModification> modification(new DicomModification);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	541
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	542 Json::Value body;
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	543 ParseAnonymizationRequest(body, *modification, call);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	544
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	545 SubmitBulkJob(modification, true /* anonymization */, call, body);
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	546 }
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	547
45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	548
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	549 static void StoreCreatedInstance(std::string& id /* out */,
1572 904096e7367e More information about the origin request in OnStoredInstance() callbacks Sebastien Jodogne <s.jodogne@gmail.com> parents: 1570 diff changeset	550 RestApiPostCall& call,
2899 5dd649de253d POST-ing a DICOM file to "/instances" also answers the patient/study/series ID Sebastien Jodogne <s.jodogne@gmail.com> parents: 2869 diff changeset	551 ParsedDicomFile& dicom,
5dd649de253d POST-ing a DICOM file to "/instances" also answers the patient/study/series ID Sebastien Jodogne <s.jodogne@gmail.com> parents: 2869 diff changeset	552 bool sendAnswer)
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	553 {
4508 8f9090b137f1 Optimization in C-STORE SCP by avoiding an unnecessary DICOM parsing Sebastien Jodogne <s.jodogne@gmail.com> parents: 4479 diff changeset	554 std::unique_ptr<DicomInstanceToStore> toStore(DicomInstanceToStore::CreateFromParsedDicomFile(dicom));
8f9090b137f1 Optimization in C-STORE SCP by avoiding an unnecessary DICOM parsing Sebastien Jodogne <s.jodogne@gmail.com> parents: 4479 diff changeset	555 toStore->SetOrigin(DicomInstanceOrigin::FromRest(call));
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	556
1572 904096e7367e More information about the origin request in OnStoredInstance() callbacks Sebastien Jodogne <s.jodogne@gmail.com> parents: 1570 diff changeset	557 ServerContext& context = OrthancRestApi::GetContext(call);
4796 94616af363ec added ReceivedCStoreInstanceFilter lua callback + OrthancPluginRegisterIncomingCStoreInstanceFilter in sdk Alain Mazy <am@osimis.io> parents: 4766 diff changeset	558 ServerContext::StoreResult result = context.Store(id, *toStore, StoreInstanceMode_Default);
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	559
4796 94616af363ec added ReceivedCStoreInstanceFilter lua callback + OrthancPluginRegisterIncomingCStoreInstanceFilter in sdk Alain Mazy <am@osimis.io> parents: 4766 diff changeset	560 if (result.GetStatus() == StoreStatus_Failure)
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	561 {
1582 bd1889029cbb encoding of exceptions Sebastien Jodogne <s.jodogne@gmail.com> parents: 1572 diff changeset	562 throw OrthancException(ErrorCode_CannotStoreInstance);
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	563 }
2899 5dd649de253d POST-ing a DICOM file to "/instances" also answers the patient/study/series ID Sebastien Jodogne <s.jodogne@gmail.com> parents: 2869 diff changeset	564
5dd649de253d POST-ing a DICOM file to "/instances" also answers the patient/study/series ID Sebastien Jodogne <s.jodogne@gmail.com> parents: 2869 diff changeset	565 if (sendAnswer)
5dd649de253d POST-ing a DICOM file to "/instances" also answers the patient/study/series ID Sebastien Jodogne <s.jodogne@gmail.com> parents: 2869 diff changeset	566 {
4796 94616af363ec added ReceivedCStoreInstanceFilter lua callback + OrthancPluginRegisterIncomingCStoreInstanceFilter in sdk Alain Mazy <am@osimis.io> parents: 4766 diff changeset	567 OrthancRestApi::GetApi(call).AnswerStoredInstance(call, *toStore, result.GetStatus(), id);
2899 5dd649de253d POST-ing a DICOM file to "/instances" also answers the patient/study/series ID Sebastien Jodogne <s.jodogne@gmail.com> parents: 2869 diff changeset	568 }
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	569 }
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	570
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	571
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	572 static void CreateDicomV1(ParsedDicomFile& dicom,
1572 904096e7367e More information about the origin request in OnStoredInstance() callbacks Sebastien Jodogne <s.jodogne@gmail.com> parents: 1570 diff changeset	573 RestApiPostCall& call,
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	574 const Json::Value& request)
796 e7b1ca0f1e04 creation of dicom files Sebastien Jodogne <s.jodogne@gmail.com> parents: 795 diff changeset	575 {
e7b1ca0f1e04 creation of dicom files Sebastien Jodogne <s.jodogne@gmail.com> parents: 795 diff changeset	576 // curl http://localhost:8042/tools/create-dicom -X POST -d '{"PatientName":"Hello^World"}'
800 ecedd89055db generation of DICOM images from PNG files Sebastien Jodogne <s.jodogne@gmail.com> parents: 796 diff changeset	577 // curl http://localhost:8042/tools/create-dicom -X POST -d '{"PatientName":"Hello^World","PixelData":"data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAAAAAA6mKC9AAAACXBIWXMAAAsTAAALEwEAmpwYAAAAB3RJTUUH3gUGDDcB53FulQAAAElJREFUGNNtj0sSAEEEQ1+U+185s1CtmRkblQ9CZldsKHJDk6DLGLJa6chjh0ooQmpjXMM86zPwydGEj6Ed/UGykkEM8X+p3u8/8LcOJIWLGeMAAAAASUVORK5CYII="}'
796 e7b1ca0f1e04 creation of dicom files Sebastien Jodogne <s.jodogne@gmail.com> parents: 795 diff changeset	578
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	579 assert(request.isObject());
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	580 LOG(WARNING) << "Using a deprecated call to /tools/create-dicom";
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	581
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	582 Json::Value::Members members = request.getMemberNames();
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	583 for (size_t i = 0; i < members.size(); i++)
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	584 {
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	585 const std::string& name = members[i];
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	586 if (request[name].type() != Json::stringValue)
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	587 {
1582 bd1889029cbb encoding of exceptions Sebastien Jodogne <s.jodogne@gmail.com> parents: 1572 diff changeset	588 throw OrthancException(ErrorCode_CreateDicomNotString);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	589 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	590
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	591 std::string value = request[name].asString();
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	592
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	593 DicomTag tag = FromDcmtkBridge::ParseTag(name);
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	594 if (tag == DICOM_TAG_PIXEL_DATA)
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	595 {
1562 2084b7c20478 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 1558 diff changeset	596 dicom.EmbedContent(value);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	597 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	598 else
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	599 {
1982 b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	600 // This is V1, don't try and decode data URI scheme
b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	601 dicom.ReplacePlainString(tag, value);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	602 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	603 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	604 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	605
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	606
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	607 static void InjectTags(ParsedDicomFile& dicom,
1641 4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	608 const Json::Value& tags,
3691 4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	609 bool decodeBinaryTags,
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	610 const std::string& privateCreator,
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	611 bool force)
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	612 {
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	613 if (tags.type() != Json::objectValue)
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	614 {
2955 bbfd95a0c429 cont Sebastien Jodogne <s.jodogne@gmail.com> parents: 2952 diff changeset	615 throw OrthancException(ErrorCode_BadRequest, "Tags field is not an array");
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	616 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	617
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	618 // Inject the user-specified tags
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	619 Json::Value::Members members = tags.getMemberNames();
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	620 for (size_t i = 0; i < members.size(); i++)
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	621 {
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	622 const std::string& name = members[i];
1641 4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	623 DicomTag tag = FromDcmtkBridge::ParseTag(name);
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	624
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	625 if (tag != DICOM_TAG_SPECIFIC_CHARACTER_SET)
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	626 {
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	627 if (!force &&
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	628 tag != DICOM_TAG_PATIENT_ID &&
1640 48224db51ee7 allow override date/time when creating DICOM files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1616 diff changeset	629 tag != DICOM_TAG_ACQUISITION_DATE &&
48224db51ee7 allow override date/time when creating DICOM files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1616 diff changeset	630 tag != DICOM_TAG_ACQUISITION_TIME &&
48224db51ee7 allow override date/time when creating DICOM files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1616 diff changeset	631 tag != DICOM_TAG_CONTENT_DATE &&
48224db51ee7 allow override date/time when creating DICOM files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1616 diff changeset	632 tag != DICOM_TAG_CONTENT_TIME &&
48224db51ee7 allow override date/time when creating DICOM files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1616 diff changeset	633 tag != DICOM_TAG_INSTANCE_CREATION_DATE &&
48224db51ee7 allow override date/time when creating DICOM files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1616 diff changeset	634 tag != DICOM_TAG_INSTANCE_CREATION_TIME &&
48224db51ee7 allow override date/time when creating DICOM files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1616 diff changeset	635 tag != DICOM_TAG_SERIES_DATE &&
48224db51ee7 allow override date/time when creating DICOM files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1616 diff changeset	636 tag != DICOM_TAG_SERIES_TIME &&
48224db51ee7 allow override date/time when creating DICOM files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1616 diff changeset	637 tag != DICOM_TAG_STUDY_DATE &&
48224db51ee7 allow override date/time when creating DICOM files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1616 diff changeset	638 tag != DICOM_TAG_STUDY_TIME &&
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	639 dicom.HasTag(tag))
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	640 {
2952 4ceb9bf7b00c added details string in OrthancException Sebastien Jodogne <s.jodogne@gmail.com> parents: 2936 diff changeset	641 throw OrthancException(ErrorCode_CreateDicomOverrideTag, name);
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	642 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	643
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	644 if (tag == DICOM_TAG_PIXEL_DATA)
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	645 {
1582 bd1889029cbb encoding of exceptions Sebastien Jodogne <s.jodogne@gmail.com> parents: 1572 diff changeset	646 throw OrthancException(ErrorCode_CreateDicomUseContent);
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	647 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	648 else
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	649 {
3691 4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	650 dicom.Replace(tag, tags[name], decodeBinaryTags, DicomReplaceMode_InsertIfAbsent, privateCreator);
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	651 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	652 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	653 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	654 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	655
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	656
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	657 static void CreateSeries(RestApiPostCall& call,
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	658 ParsedDicomFile& base /* in */,
1641 4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	659 const Json::Value& content,
3691 4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	660 bool decodeBinaryTags,
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	661 const std::string& privateCreator,
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	662 bool force)
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	663 {
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	664 assert(content.isArray());
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	665 assert(content.size() > 0);
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	666 ServerContext& context = OrthancRestApi::GetContext(call);
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	667
1982 b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	668 base.ReplacePlainString(DICOM_TAG_IMAGES_IN_ACQUISITION, boost::lexical_cast<std::string>(content.size()));
b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	669 base.ReplacePlainString(DICOM_TAG_NUMBER_OF_TEMPORAL_POSITIONS, "1");
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	670
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	671 std::string someInstance;
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	672
1566 3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	673 try
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	674 {
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	675 for (Json::ArrayIndex i = 0; i < content.size(); i++)
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	676 {
3712 2a170a8f1faf replacing std::auto_ptr by std::unique_ptr Sebastien Jodogne <s.jodogne@gmail.com> parents: 3709 diff changeset	677 std::unique_ptr<ParsedDicomFile> dicom(base.Clone(false));
1566 3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	678 const Json::Value* payload = NULL;
3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	679
3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	680 if (content[i].type() == Json::stringValue)
3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	681 {
3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	682 payload = &content[i];
3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	683 }
3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	684 else if (content[i].type() == Json::objectValue)
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	685 {
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	686 if (!content[i].isMember(CONTENT))
1566 3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	687 {
1582 bd1889029cbb encoding of exceptions Sebastien Jodogne <s.jodogne@gmail.com> parents: 1572 diff changeset	688 throw OrthancException(ErrorCode_CreateDicomNoPayload);
1566 3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	689 }
3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	690
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	691 payload = &content[i][CONTENT];
1566 3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	692
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	693 if (content[i].isMember(TAGS))
1570 2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	694 {
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	695 InjectTags(*dicom, content[i][TAGS], decodeBinaryTags, privateCreator, force);
1570 2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	696 }
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	697 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	698
1566 3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	699 if (payload == NULL \|\|
3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	700 payload->type() != Json::stringValue)
3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	701 {
1582 bd1889029cbb encoding of exceptions Sebastien Jodogne <s.jodogne@gmail.com> parents: 1572 diff changeset	702 throw OrthancException(ErrorCode_CreateDicomUseDataUriScheme);
1566 3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	703 }
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	704
1566 3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	705 dicom->EmbedContent(payload->asString());
1982 b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	706 dicom->ReplacePlainString(DICOM_TAG_INSTANCE_NUMBER, boost::lexical_cast<std::string>(i + 1));
b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	707 dicom->ReplacePlainString(DICOM_TAG_IMAGE_INDEX, boost::lexical_cast<std::string>(i + 1));
1566 3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	708
2899 5dd649de253d POST-ing a DICOM file to "/instances" also answers the patient/study/series ID Sebastien Jodogne <s.jodogne@gmail.com> parents: 2869 diff changeset	709 StoreCreatedInstance(someInstance, call, *dicom, false);
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	710 }
1566 3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	711 }
1616 644c32c07306 cppcheck Sebastien Jodogne <s.jodogne@gmail.com> parents: 1582 diff changeset	712 catch (OrthancException&)
1566 3be6eb3757c8 error handling Sebastien Jodogne <s.jodogne@gmail.com> parents: 1565 diff changeset	713 {
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	714 // Error: Remove the newly-created series
27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	715
27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	716 std::string series;
27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	717 if (context.GetIndex().LookupParent(series, someInstance))
1570 2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	718 {
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	719 Json::Value dummy;
27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	720 context.GetIndex().DeleteResource(dummy, series, ResourceType_Series);
1570 2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	721 }
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	722
1616 644c32c07306 cppcheck Sebastien Jodogne <s.jodogne@gmail.com> parents: 1582 diff changeset	723 throw;
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	724 }
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	725
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	726 std::string series;
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	727 if (context.GetIndex().LookupParent(series, someInstance))
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	728 {
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	729 OrthancRestApi::GetApi(call).AnswerStoredResource(call, series, ResourceType_Series, StoreStatus_Success);
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	730 }
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	731 }
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	732
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	733
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	734 static void CreateDicomV2(RestApiPostCall& call,
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	735 const Json::Value& request)
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	736 {
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	737 static const char* const SPECIFIC_CHARACTER_SET_2 = "SpecificCharacterSet";
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	738 static const char* const TYPE = "Type";
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	739 static const char* const VALUE = "Value";
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	740
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	741 assert(request.isObject());
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	742 ServerContext& context = OrthancRestApi::GetContext(call);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	743
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	744 if (!request.isMember(TAGS) \|\|
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	745 request[TAGS].type() != Json::objectValue)
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	746 {
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	747 throw OrthancException(ErrorCode_BadRequest);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	748 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	749
1805 f08978b1f45b c-find scu for modality worklists Sebastien Jodogne <s.jodogne@gmail.com> parents: 1699 diff changeset	750 ParsedDicomFile dicom(true);
1558 124de28b32ed fix encodings of newly created dicom files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1557 diff changeset	751
1557 ad1e127b4ed5 fix in encodings Sebastien Jodogne <s.jodogne@gmail.com> parents: 1555 diff changeset	752 {
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	753 Encoding encoding;
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	754
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	755 if (request[TAGS].isMember(SPECIFIC_CHARACTER_SET_2))
1558 124de28b32ed fix encodings of newly created dicom files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1557 diff changeset	756 {
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	757 const char* tmp = request[TAGS][SPECIFIC_CHARACTER_SET_2].asCString();
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	758 if (!GetDicomEncoding(encoding, tmp))
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	759 {
2955 bbfd95a0c429 cont Sebastien Jodogne <s.jodogne@gmail.com> parents: 2952 diff changeset	760 throw OrthancException(ErrorCode_ParameterOutOfRange,
bbfd95a0c429 cont Sebastien Jodogne <s.jodogne@gmail.com> parents: 2952 diff changeset	761 "Unknown specific character set: " + std::string(tmp));
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	762 }
1558 124de28b32ed fix encodings of newly created dicom files Sebastien Jodogne <s.jodogne@gmail.com> parents: 1557 diff changeset	763 }
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	764 else
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	765 {
2381 b8969010b534 uncoupling DCMTK primitives from Orthanc::Configuration Sebastien Jodogne <s.jodogne@gmail.com> parents: 2361 diff changeset	766 encoding = GetDefaultDicomEncoding();
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	767 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	768
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	769 dicom.SetEncoding(encoding);
1557 ad1e127b4ed5 fix in encodings Sebastien Jodogne <s.jodogne@gmail.com> parents: 1555 diff changeset	770 }
ad1e127b4ed5 fix in encodings Sebastien Jodogne <s.jodogne@gmail.com> parents: 1555 diff changeset	771
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	772 ResourceType parentType = ResourceType_Instance;
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	773
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	774 if (request.isMember(PARENT))
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	775 {
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	776 // Locate the parent tags
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	777 std::string parent = request[PARENT].asString();
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	778 if (!context.GetIndex().LookupResourceType(parentType, parent))
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	779 {
1582 bd1889029cbb encoding of exceptions Sebastien Jodogne <s.jodogne@gmail.com> parents: 1572 diff changeset	780 throw OrthancException(ErrorCode_CreateDicomBadParent);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	781 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	782
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	783 if (parentType == ResourceType_Instance)
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	784 {
1582 bd1889029cbb encoding of exceptions Sebastien Jodogne <s.jodogne@gmail.com> parents: 1572 diff changeset	785 throw OrthancException(ErrorCode_CreateDicomParentIsInstance);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	786 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	787
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	788 // Select one existing child instance of the parent resource, to
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	789 // retrieve all its tags
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	790 Json::Value siblingTags;
3009 750de70b3603 make "/tools/create-dicom" more tolerant to invalid specific character set Sebastien Jodogne <s.jodogne@gmail.com> parents: 2955 diff changeset	791 std::string siblingInstanceId;
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	792
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	793 {
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	794 // Retrieve all the instances of the parent resource
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	795 std::list<std::string> siblingInstances;
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	796 context.GetIndex().GetChildInstances(siblingInstances, parent);
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	797
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	798 if (siblingInstances.empty())
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	799 {
27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	800 // Error: No instance (should never happen)
27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	801 throw OrthancException(ErrorCode_InternalError);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	802 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	803
3009 750de70b3603 make "/tools/create-dicom" more tolerant to invalid specific character set Sebastien Jodogne <s.jodogne@gmail.com> parents: 2955 diff changeset	804 siblingInstanceId = siblingInstances.front();
750de70b3603 make "/tools/create-dicom" more tolerant to invalid specific character set Sebastien Jodogne <s.jodogne@gmail.com> parents: 2955 diff changeset	805 context.ReadDicomAsJson(siblingTags, siblingInstanceId);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	806 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	807
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	808
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	809 // Choose the same encoding as the parent resource
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	810 {
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	811 static const char* const SPECIFIC_CHARACTER_SET = "0008,0005";
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	812
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	813 if (siblingTags.isMember(SPECIFIC_CHARACTER_SET))
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	814 {
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	815 Encoding encoding;
3009 750de70b3603 make "/tools/create-dicom" more tolerant to invalid specific character set Sebastien Jodogne <s.jodogne@gmail.com> parents: 2955 diff changeset	816
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	817 if (!siblingTags[SPECIFIC_CHARACTER_SET].isMember(VALUE) \|\|
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	818 siblingTags[SPECIFIC_CHARACTER_SET][VALUE].type() != Json::stringValue \|\|
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	819 !GetDicomEncoding(encoding, siblingTags[SPECIFIC_CHARACTER_SET][VALUE].asCString()))
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	820 {
3009 750de70b3603 make "/tools/create-dicom" more tolerant to invalid specific character set Sebastien Jodogne <s.jodogne@gmail.com> parents: 2955 diff changeset	821 LOG(WARNING) << "Instance with an incorrect Specific Character Set, "
750de70b3603 make "/tools/create-dicom" more tolerant to invalid specific character set Sebastien Jodogne <s.jodogne@gmail.com> parents: 2955 diff changeset	822 << "using the default Orthanc encoding: " << siblingInstanceId;
750de70b3603 make "/tools/create-dicom" more tolerant to invalid specific character set Sebastien Jodogne <s.jodogne@gmail.com> parents: 2955 diff changeset	823 encoding = GetDefaultDicomEncoding();
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	824 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	825
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	826 dicom.SetEncoding(encoding);
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	827 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	828 }
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	829
4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	830
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	831 // Retrieve the tags for all the parent modules
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	832 typedef std::set<DicomTag> ModuleTags;
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	833 ModuleTags moduleTags;
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	834
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	835 ResourceType type = parentType;
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	836 for (;;)
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	837 {
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	838 DicomTag::AddTagsForModule(moduleTags, GetModule(type));
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	839
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	840 if (type == ResourceType_Patient)
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	841 {
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	842 break; // We're done
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	843 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	844
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	845 // Go up
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	846 std::string tmp;
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	847 if (!context.GetIndex().LookupParent(tmp, parent))
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	848 {
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	849 throw OrthancException(ErrorCode_InternalError);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	850 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	851
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	852 parent = tmp;
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	853 type = GetParentResourceType(type);
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	854 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	855
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	856 for (ModuleTags::const_iterator it = moduleTags.begin();
1616 644c32c07306 cppcheck Sebastien Jodogne <s.jodogne@gmail.com> parents: 1582 diff changeset	857 it != moduleTags.end(); ++it)
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	858 {
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	859 std::string t = it->Format();
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	860 if (siblingTags.isMember(t))
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	861 {
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	862 const Json::Value& tag = siblingTags[t];
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	863 if (tag[TYPE] == "Null")
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	864 {
1982 b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	865 dicom.ReplacePlainString(*it, "");
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	866 }
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	867 else if (tag[TYPE] == "String")
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	868 {
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	869 std::string value = tag[VALUE].asString(); // This is an UTF-8 value (as it comes from JSON)
3441 6cc72ebfd6ef Fix issue #141 (/tools/create-dicom removes non-ASCII characters from study description) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3217 diff changeset	870 dicom.ReplacePlainString(*it, value);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	871 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	872 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	873 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	874 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	875
1641 4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	876
1689 26083d84d237 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 1652 diff changeset	877 bool decodeBinaryTags = true;
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	878 if (request.isMember(INTERPRET_BINARY_TAGS))
1641 4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	879 {
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	880 const Json::Value& v = request[INTERPRET_BINARY_TAGS];
1641 4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	881 if (v.type() != Json::booleanValue)
4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	882 {
4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	883 throw OrthancException(ErrorCode_BadRequest);
4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	884 }
4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	885
1689 26083d84d237 refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 1652 diff changeset	886 decodeBinaryTags = v.asBool();
1641 4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	887 }
4e56b5a206b7 Support of binary tags encoded using data URI scheme Sebastien Jodogne <s.jodogne@gmail.com> parents: 1640 diff changeset	888
3691 4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	889
4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	890 // New argument in Orthanc 1.6.0
4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	891 std::string privateCreator;
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	892 if (request.isMember(PRIVATE_CREATOR))
3691 4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	893 {
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	894 const Json::Value& v = request[PRIVATE_CREATOR];
3691 4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	895 if (v.type() != Json::stringValue)
4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	896 {
4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	897 throw OrthancException(ErrorCode_BadRequest);
4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	898 }
4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	899
4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	900 privateCreator = v.asString();
4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	901 }
3709 1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	902 else
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	903 {
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	904 OrthancConfiguration::ReaderLock lock;
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	905 privateCreator = lock.GetConfiguration().GetDefaultPrivateCreator();
1f4910999fe7 Fix issue #168 (Plugins can't read private tags from the configuration file) Sebastien Jodogne <s.jodogne@gmail.com> parents: 3691 diff changeset	906 }
3691 4922bdd046dd Fix issue #140 (Modifying private tags with REST API changes VR from LO to UN) - DANGEROUS COMMIT Sebastien Jodogne <s.jodogne@gmail.com> parents: 3640 diff changeset	907
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	908
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	909 // New in Orthanc 1.9.0
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	910 bool force = false;
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	911 if (request.isMember(FORCE))
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	912 {
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	913 const Json::Value& v = request[FORCE];
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	914 if (v.type() != Json::booleanValue)
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	915 {
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	916 throw OrthancException(ErrorCode_BadRequest);
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	917 }
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	918
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	919 force = v.asBool();
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	920 }
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	921
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	922
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	923 // Inject time-related information
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	924 std::string date, time;
2475 8cc3ca64a534 Orthanc now uses UTC (universal time) instead of local time in its database Sebastien Jodogne <s.jodogne@gmail.com> parents: 2447 diff changeset	925 SystemToolbox::GetNowDicom(date, time, true /* use UTC time (not local time) */);
1982 b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	926 dicom.ReplacePlainString(DICOM_TAG_ACQUISITION_DATE, date);
b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	927 dicom.ReplacePlainString(DICOM_TAG_ACQUISITION_TIME, time);
b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	928 dicom.ReplacePlainString(DICOM_TAG_CONTENT_DATE, date);
b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	929 dicom.ReplacePlainString(DICOM_TAG_CONTENT_TIME, time);
b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	930 dicom.ReplacePlainString(DICOM_TAG_INSTANCE_CREATION_DATE, date);
b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	931 dicom.ReplacePlainString(DICOM_TAG_INSTANCE_CREATION_TIME, time);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	932
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	933 if (parentType == ResourceType_Patient \|\|
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	934 parentType == ResourceType_Study \|\|
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	935 parentType == ResourceType_Instance /* no parent */)
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	936 {
1982 b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	937 dicom.ReplacePlainString(DICOM_TAG_SERIES_DATE, date);
b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	938 dicom.ReplacePlainString(DICOM_TAG_SERIES_TIME, time);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	939 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	940
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	941 if (parentType == ResourceType_Patient \|\|
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	942 parentType == ResourceType_Instance /* no parent */)
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	943 {
1982 b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	944 dicom.ReplacePlainString(DICOM_TAG_STUDY_DATE, date);
b5d4f9c156ad Modification of instances can now replace PixelData Sebastien Jodogne <s.jodogne@gmail.com> parents: 1934 diff changeset	945 dicom.ReplacePlainString(DICOM_TAG_STUDY_TIME, time);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	946 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	947
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	948
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	949 InjectTags(dicom, request[TAGS], decodeBinaryTags, privateCreator, force);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	950
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	951
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	952 // Inject the content (either an image, or a PDF file)
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	953 if (request.isMember(CONTENT))
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	954 {
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	955 const Json::Value& content = request[CONTENT];
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	956
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	957 if (content.type() == Json::stringValue)
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	958 {
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	959 dicom.EmbedContent(request[CONTENT].asString());
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	960
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	961 }
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	962 else if (content.type() == Json::arrayValue)
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	963 {
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	964 if (content.size() > 0)
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	965 {
1565 4b23310eb7e8 add tags per instances in a series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1564 diff changeset	966 // Let's create a series instead of a single instance
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	967 CreateSeries(call, dicom, content, decodeBinaryTags, privateCreator, force);
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	968 return;
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	969 }
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	970 }
1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	971 else
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	972 {
1582 bd1889029cbb encoding of exceptions Sebastien Jodogne <s.jodogne@gmail.com> parents: 1572 diff changeset	973 throw OrthancException(ErrorCode_CreateDicomUseDataUriScheme);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	974 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	975 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	976
1564 1b7def486e62 creation of DICOM series Sebastien Jodogne <s.jodogne@gmail.com> parents: 1562 diff changeset	977 std::string id;
2899 5dd649de253d POST-ing a DICOM file to "/instances" also answers the patient/study/series ID Sebastien Jodogne <s.jodogne@gmail.com> parents: 2869 diff changeset	978 StoreCreatedInstance(id, call, dicom, true);
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	979 }
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	980
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	981
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	982 static void CreateDicom(RestApiPostCall& call)
d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	983 {
4422 48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	984 if (call.IsDocumentation())
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	985 {
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	986 call.GetDocumentation()
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	987 .SetTag("System")
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	988 .SetSummary("Create one DICOM instance")
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	989 .SetDescription("Create one DICOM instance, and store it into Orthanc")
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	990 .SetRequestField(TAGS, RestApiCallDocumentation::Type_JsonObject,
4422 48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	991 "Associative array containing the tags of the new instance to be created", true)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	992 .SetRequestField(CONTENT, RestApiCallDocumentation::Type_String,
4422 48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	993 "This field can be used to embed an image (pixel data) or a PDF inside the created DICOM instance. "
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	994 "The PNG image, the JPEG image or the PDF file must be provided using their "
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	995 "[data URI scheme encoding](https://en.wikipedia.org/wiki/Data_URI_scheme). "
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	996 "This field can possibly contain a JSON array, in which case a DICOM series is created "
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	997 "containing one DICOM instance for each item in the `Content` field.", false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	998 .SetRequestField(PARENT, RestApiCallDocumentation::Type_String,
4422 48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	999 "If present, the newly created instance will be attached to the parent DICOM resource "
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	1000 "whose Orthanc identifier is contained in this field. The DICOM tags of the parent "
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	1001 "modules in the DICOM hierarchy will be automatically copied to the newly created instance.", false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1002 .SetRequestField(INTERPRET_BINARY_TAGS, RestApiCallDocumentation::Type_Boolean,
4422 48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	1003 "If some value in the `Tags` associative array is formatted according to some "
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	1004 "[data URI scheme encoding](https://en.wikipedia.org/wiki/Data_URI_scheme), "
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	1005 "whether this value is decoded to a binary value or kept as such (`true` by default)", false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1006 .SetRequestField(PRIVATE_CREATOR, RestApiCallDocumentation::Type_String,
4422 48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	1007 "The private creator to be used for private tags in `Tags`", false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1008 .SetRequestField(FORCE, RestApiCallDocumentation::Type_Boolean,
4479 1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	1009 "Avoid the consistency checks for the DICOM tags that enforce the DICOM model of the real-world. "
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	1010 "You can notably use this flag if you need to manually set the tags `StudyInstanceUID`, "
1619cffd1948 "/tools/create-dicom": New flag "Force" to bypass consistency checks for the DICOM tags Sebastien Jodogne <s.jodogne@gmail.com> parents: 4437 diff changeset	1011 "`SeriesInstanceUID`, or `SOPInstanceUID`. Be careful with this feature.", false)
4422 48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	1012 .SetAnswerField("ID", RestApiCallDocumentation::Type_String, "Orthanc identifier of the newly created instance")
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	1013 .SetAnswerField("Path", RestApiCallDocumentation::Type_String, "Path to access the instance in the REST API");
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	1014 return;
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	1015 }
48303e493135 cont openapi Sebastien Jodogne <s.jodogne@gmail.com> parents: 4045 diff changeset	1016
1555 d6a93e12b1c1 Creation of DICOM files with encapsulated PDF Sebastien Jodogne <s.jodogne@gmail.com> parents: 1486 diff changeset	1017 Json::Value request;
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	1018 if (!call.ParseJsonRequest(request) \|\|
27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	1019 !request.isObject())
1570 2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	1020 {
2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	1021 throw OrthancException(ErrorCode_BadRequest);
2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	1022 }
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	1023
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1024 if (request.isMember(TAGS))
1570 2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	1025 {
2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	1026 CreateDicomV2(call, request);
2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	1027 }
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	1028 else
1570 2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	1029 {
2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	1030 // Compatibility with Orthanc <= 0.9.3
1805 f08978b1f45b c-find scu for modality worklists Sebastien Jodogne <s.jodogne@gmail.com> parents: 1699 diff changeset	1031 ParsedDicomFile dicom(true);
1572 904096e7367e More information about the origin request in OnStoredInstance() callbacks Sebastien Jodogne <s.jodogne@gmail.com> parents: 1570 diff changeset	1032 CreateDicomV1(dicom, call, request);
1569 27774f6f84e4 improved error handling in http answers jodogne parents: 1566 diff changeset	1033
1570 2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	1034 std::string id;
2899 5dd649de253d POST-ing a DICOM file to "/instances" also answers the patient/study/series ID Sebastien Jodogne <s.jodogne@gmail.com> parents: 2869 diff changeset	1035 StoreCreatedInstance(id, call, dicom, true);
1570 2bd2c280f9b5 fix indentation jodogne parents: 1569 diff changeset	1036 }
796 e7b1ca0f1e04 creation of dicom files Sebastien Jodogne <s.jodogne@gmail.com> parents: 795 diff changeset	1037 }
e7b1ca0f1e04 creation of dicom files Sebastien Jodogne <s.jodogne@gmail.com> parents: 795 diff changeset	1038
e7b1ca0f1e04 creation of dicom files Sebastien Jodogne <s.jodogne@gmail.com> parents: 795 diff changeset	1039
2844 99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1040 static void SplitStudy(RestApiPostCall& call)
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1041 {
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1042 if (call.IsDocumentation())
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1043 {
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1044 OrthancRestApi::DocumentSubmitCommandsJob(call);
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1045 call.GetDocumentation()
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1046 .SetTag("Studies")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1047 .SetSummary("Split study")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1048 .SetDescription("Start a new job so as to split the DICOM study whose Orthanc identifier is provided in the URL, "
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1049 "by taking some of its children series or instances out of it and putting them into a brand new study "
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1050 "(this new study is created by setting the `StudyInstanceUID` tag to a random identifier): "
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1051 "https://book.orthanc-server.com/users/anonymization.html#splitting")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1052 .SetUriArgument("id", "Orthanc identifier of the study of interest")
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1053 .SetRequestField(SERIES, RestApiCallDocumentation::Type_JsonListOfStrings,
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1054 "The list of series to be separated from the parent study. "
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1055 "These series must all be children of the same source study, that is specified in the URI.", false)
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1056 .SetRequestField(REPLACE, RestApiCallDocumentation::Type_JsonObject,
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1057 "Associative array to change the value of some DICOM tags in the new study. "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1058 "These tags must be part of the \"Patient Module Attributes\" or the \"General Study "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1059 "Module Attributes\", as specified by the DICOM 2011 standard in Tables C.7-1 and C.7-3.", false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1060 .SetRequestField(REMOVE, RestApiCallDocumentation::Type_JsonListOfStrings,
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1061 "List of tags that must be removed in the new study (from the same modules as in the `Replace` option)", false)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1062 .SetRequestField(KEEP_SOURCE, RestApiCallDocumentation::Type_Boolean,
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1063 "If set to `true`, instructs Orthanc to keep a copy of the original series/instances in the source study. "
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1064 "By default, the original series/instances are deleted from Orthanc.", false)
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1065 .SetRequestField(INSTANCES, RestApiCallDocumentation::Type_JsonListOfStrings,
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1066 "The list of instances to be separated from the parent study. "
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1067 "These instances must all be children of the same source study, that is specified in the URI.", false);
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1068 return;
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1069 }
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1070
2844 99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1071 ServerContext& context = OrthancRestApi::GetContext(call);
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1072
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1073 Json::Value request;
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1074 if (!call.ParseJsonRequest(request))
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1075 {
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1076 // Bad JSON request
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1077 throw OrthancException(ErrorCode_BadFileFormat);
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1078 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1079
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1080 const std::string study = call.GetUriComponent("id", "");
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1081
3712 2a170a8f1faf replacing std::auto_ptr by std::unique_ptr Sebastien Jodogne <s.jodogne@gmail.com> parents: 3709 diff changeset	1082 std::unique_ptr<SplitStudyJob> job(new SplitStudyJob(context, study));
2844 99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1083 job->SetOrigin(call);
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1084
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1085 bool ok = false;
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1086 if (request.isMember(SERIES))
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1087 {
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1088 std::vector<std::string> series;
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1089 SerializationToolbox::ReadArrayOfStrings(series, request, SERIES);
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1090
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1091 for (size_t i = 0; i < series.size(); i++)
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1092 {
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1093 job->AddSourceSeries(series[i]);
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1094 ok = true;
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1095 }
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1096 }
2844 99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1097
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1098 if (request.isMember(INSTANCES))
2844 99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1099 {
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1100 std::vector<std::string> instances;
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1101 SerializationToolbox::ReadArrayOfStrings(instances, request, INSTANCES);
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1102
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1103 for (size_t i = 0; i < instances.size(); i++)
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1104 {
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1105 job->AddSourceInstance(instances[i]);
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1106 ok = true;
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1107 }
2844 99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1108 }
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1109
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1110 if (!ok)
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1111 {
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1112 throw OrthancException(ErrorCode_BadRequest, "Both the \"Series\" and the \"Instances\" fields are missing");
bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1113 }
3942 5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	1114
2860 8b00e4cb4a6b SetOfCommandsJob Sebastien Jodogne <s.jodogne@gmail.com> parents: 2853 diff changeset	1115 job->AddTrailingStep();
8b00e4cb4a6b SetOfCommandsJob Sebastien Jodogne <s.jodogne@gmail.com> parents: 2853 diff changeset	1116
3942 5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	1117 SetKeepSource(*job, request);
2844 99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1118
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1119 if (request.isMember(REMOVE))
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1120 {
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1121 if (request[REMOVE].type() != Json::arrayValue)
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1122 {
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1123 throw OrthancException(ErrorCode_BadFileFormat);
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1124 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1125
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1126 for (Json::Value::ArrayIndex i = 0; i < request[REMOVE].size(); i++)
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1127 {
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1128 if (request[REMOVE][i].type() != Json::stringValue)
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1129 {
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1130 throw OrthancException(ErrorCode_BadFileFormat);
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1131 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1132 else
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1133 {
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1134 job->Remove(FromDcmtkBridge::ParseTag(request[REMOVE][i].asCString()));
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1135 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1136 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1137 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1138
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1139 if (request.isMember(REPLACE))
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1140 {
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1141 if (request[REPLACE].type() != Json::objectValue)
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1142 {
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1143 throw OrthancException(ErrorCode_BadFileFormat);
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1144 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1145
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1146 Json::Value::Members tags = request[REPLACE].getMemberNames();
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1147
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1148 for (size_t i = 0; i < tags.size(); i++)
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1149 {
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1150 const Json::Value& value = request[REPLACE][tags[i]];
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1151
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1152 if (value.type() != Json::stringValue)
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1153 {
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1154 throw OrthancException(ErrorCode_BadFileFormat);
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1155 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1156 else
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1157 {
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1158 job->Replace(FromDcmtkBridge::ParseTag(tags[i]), value.asString());
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1159 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1160 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1161 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1162
2869 2bd87764709c refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2868 diff changeset	1163 OrthancRestApi::GetApi(call).SubmitCommandsJob
2bd87764709c refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2868 diff changeset	1164 (call, job.release(), true /* synchronous by default */, request);
2844 99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1165 }
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1166
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1167
2853 52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1168 static void MergeStudy(RestApiPostCall& call)
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1169 {
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1170 if (call.IsDocumentation())
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1171 {
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1172 OrthancRestApi::DocumentSubmitCommandsJob(call);
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1173 call.GetDocumentation()
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1174 .SetTag("Studies")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1175 .SetSummary("Merge study")
4715 fb98db281d1d "/studies/{id}/merge" accepts instances inside its "Resources" parameter Sebastien Jodogne <s.jodogne@gmail.com> parents: 4712 diff changeset	1176 .SetDescription("Start a new job so as to move some DICOM resources into the DICOM study whose Orthanc identifier "
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1177 "is provided in the URL: https://book.orthanc-server.com/users/anonymization.html#merging")
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1178 .SetUriArgument("id", "Orthanc identifier of the study of interest")
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1179 .SetRequestField(RESOURCES, RestApiCallDocumentation::Type_JsonListOfStrings,
4715 fb98db281d1d "/studies/{id}/merge" accepts instances inside its "Resources" parameter Sebastien Jodogne <s.jodogne@gmail.com> parents: 4712 diff changeset	1180 "The list of DICOM resources (studies, series, and/or instances) to be merged "
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1181 "into the study of interest (mandatory option)", true)
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1182 .SetRequestField(KEEP_SOURCE, RestApiCallDocumentation::Type_Boolean,
4424 83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1183 "If set to `true`, instructs Orthanc to keep a copy of the original resources in their source study. "
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1184 "By default, the original resources are deleted from Orthanc.", false);
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1185 return;
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1186 }
83371ccdfe80 openapi documentation is now completed Sebastien Jodogne <s.jodogne@gmail.com> parents: 4422 diff changeset	1187
2853 52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1188 ServerContext& context = OrthancRestApi::GetContext(call);
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1189
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1190 Json::Value request;
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1191 if (!call.ParseJsonRequest(request))
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1192 {
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1193 // Bad JSON request
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1194 throw OrthancException(ErrorCode_BadFileFormat);
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1195 }
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1196
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1197 const std::string study = call.GetUriComponent("id", "");
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1198
3712 2a170a8f1faf replacing std::auto_ptr by std::unique_ptr Sebastien Jodogne <s.jodogne@gmail.com> parents: 3709 diff changeset	1199 std::unique_ptr<MergeStudyJob> job(new MergeStudyJob(context, study));
2853 52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1200 job->SetOrigin(call);
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1201
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1202 std::vector<std::string> resources;
4712 bad8935cd5f2 "/studies/{id}/split" accepts "Instances" parameter to split instances instead of series Sebastien Jodogne <s.jodogne@gmail.com> parents: 4699 diff changeset	1203 SerializationToolbox::ReadArrayOfStrings(resources, request, RESOURCES);
2853 52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1204
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1205 for (size_t i = 0; i < resources.size(); i++)
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1206 {
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1207 job->AddSource(resources[i]);
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1208 }
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1209
2860 8b00e4cb4a6b SetOfCommandsJob Sebastien Jodogne <s.jodogne@gmail.com> parents: 2853 diff changeset	1210 job->AddTrailingStep();
8b00e4cb4a6b SetOfCommandsJob Sebastien Jodogne <s.jodogne@gmail.com> parents: 2853 diff changeset	1211
3942 5b882ad2ffd0 "/{patients\|studies\|series}/.../modify": New option "KeepSource" Sebastien Jodogne <s.jodogne@gmail.com> parents: 3930 diff changeset	1212 SetKeepSource(*job, request);
2853 52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1213
2869 2bd87764709c refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2868 diff changeset	1214 OrthancRestApi::GetApi(call).SubmitCommandsJob
2bd87764709c refactoring Sebastien Jodogne <s.jodogne@gmail.com> parents: 2868 diff changeset	1215 (call, job.release(), true /* synchronous by default */, request);
2853 52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1216 }
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1217
52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1218
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	1219 void OrthancRestApi::RegisterAnonymizeModify()
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	1220 {
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	1221 Register("/instances/{id}/modify", ModifyInstance);
2639 75a404e40323 simplification Sebastien Jodogne <s.jodogne@gmail.com> parents: 2521 diff changeset	1222 Register("/series/{id}/modify", ModifyResource<ResourceType_Series>);
75a404e40323 simplification Sebastien Jodogne <s.jodogne@gmail.com> parents: 2521 diff changeset	1223 Register("/studies/{id}/modify", ModifyResource<ResourceType_Study>);
75a404e40323 simplification Sebastien Jodogne <s.jodogne@gmail.com> parents: 2521 diff changeset	1224 Register("/patients/{id}/modify", ModifyResource<ResourceType_Patient>);
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	1225 Register("/tools/bulk-modify", BulkModify);
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	1226
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	1227 Register("/instances/{id}/anonymize", AnonymizeInstance);
2639 75a404e40323 simplification Sebastien Jodogne <s.jodogne@gmail.com> parents: 2521 diff changeset	1228 Register("/series/{id}/anonymize", AnonymizeResource<ResourceType_Series>);
75a404e40323 simplification Sebastien Jodogne <s.jodogne@gmail.com> parents: 2521 diff changeset	1229 Register("/studies/{id}/anonymize", AnonymizeResource<ResourceType_Study>);
75a404e40323 simplification Sebastien Jodogne <s.jodogne@gmail.com> parents: 2521 diff changeset	1230 Register("/patients/{id}/anonymize", AnonymizeResource<ResourceType_Patient>);
4693 45bce660ce3a added routes for bulk anonymization/modification Sebastien Jodogne <s.jodogne@gmail.com> parents: 4687 diff changeset	1231 Register("/tools/bulk-anonymize", BulkAnonymize);
796 e7b1ca0f1e04 creation of dicom files Sebastien Jodogne <s.jodogne@gmail.com> parents: 795 diff changeset	1232
1120 009dce4ea2f6 /tools/create-dicom now accepts PatientID Sebastien Jodogne <s.jodogne@gmail.com> parents: 1010 diff changeset	1233 Register("/tools/create-dicom", CreateDicom);
2844 99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1234
99863d6245b2 New URI: "/studies/.../split" to split a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2825 diff changeset	1235 Register("/studies/{id}/split", SplitStudy);
2853 52b017d22a4f New URI: "/studies/.../merge" to merge a study Sebastien Jodogne <s.jodogne@gmail.com> parents: 2844 diff changeset	1236 Register("/studies/{id}/merge", MergeStudy);
751 5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	1237 }
5197fd35333c refactoring of OrthancRestApi Sebastien Jodogne <s.jodogne@gmail.com> parents: diff changeset	1238 }

Mercurial > hg > orthanc

annotate OrthancServer/Sources/OrthancRestApi/OrthancRestAnonymizeModify.cpp @ 4879:c728d8f25e5a