Mercurial > hg > orthanc
annotate OrthancServer/ServerJobs/DicomMoveScuJob.cpp @ 3875:ea1d32861cfc transcoding
moving timeout from DicomAssocation to DicomAssociationParameters
author | Sebastien Jodogne <s.jodogne@gmail.com> |
---|---|
date | Mon, 04 May 2020 14:49:31 +0200 |
parents | 2effa961f67f |
children | 4b4f387c6bb8 |
rev | line source |
---|---|
2867 | 1 /** |
2 * Orthanc - A Lightweight, RESTful DICOM Store | |
3 * Copyright (C) 2012-2016 Sebastien Jodogne, Medical Physics | |
4 * Department, University Hospital of Liege, Belgium | |
3640
94f4a18a79cc
upgrade to year 2020
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3303
diff
changeset
|
5 * Copyright (C) 2017-2020 Osimis S.A., Belgium |
2867 | 6 * |
7 * This program is free software: you can redistribute it and/or | |
8 * modify it under the terms of the GNU General Public License as | |
9 * published by the Free Software Foundation, either version 3 of the | |
10 * License, or (at your option) any later version. | |
11 * | |
12 * In addition, as a special exception, the copyright holders of this | |
13 * program give permission to link the code of its release with the | |
14 * OpenSSL project's "OpenSSL" library (or with modified versions of it | |
15 * that use the same license as the "OpenSSL" library), and distribute | |
16 * the linked executables. You must obey the GNU General Public License | |
17 * in all respects for all of the code used other than "OpenSSL". If you | |
18 * modify file(s) with this exception, you may extend this exception to | |
19 * your version of the file(s), but you are not obligated to do so. If | |
20 * you do not wish to do so, delete this exception statement from your | |
21 * version. If you delete this exception statement from all source files | |
22 * in the program, then also delete it here. | |
23 * | |
24 * This program is distributed in the hope that it will be useful, but | |
25 * WITHOUT ANY WARRANTY; without even the implied warranty of | |
26 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
27 * General Public License for more details. | |
28 * | |
29 * You should have received a copy of the GNU General Public License | |
30 * along with this program. If not, see <http://www.gnu.org/licenses/>. | |
31 **/ | |
32 | |
33 | |
34 #include "DicomMoveScuJob.h" | |
35 | |
36 #include "../../Core/SerializationToolbox.h" | |
3095
beeeb6096f27
removing dependencies upon ServerContext
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3060
diff
changeset
|
37 #include "../ServerContext.h" |
2867 | 38 |
3303
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
39 static const char* const LOCAL_AET = "LocalAet"; |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
40 static const char* const TARGET_AET = "TargetAet"; |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
41 static const char* const REMOTE = "Remote"; |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
42 static const char* const QUERY = "Query"; |
3870
09798f2b985f
added a Timeout argument to every DICOM command + 'TargetAet' not mandatory anymore in /retrieve
Alain Mazy <alain@mazy.be>
parents:
3843
diff
changeset
|
43 static const char* const TIMEOUT = "Timeout"; |
3303
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
44 |
2867 | 45 namespace Orthanc |
46 { | |
47 class DicomMoveScuJob::Command : public SetOfCommandsJob::ICommand | |
48 { | |
49 private: | |
3712
2a170a8f1faf
replacing std::auto_ptr by std::unique_ptr
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3640
diff
changeset
|
50 DicomMoveScuJob& that_; |
2a170a8f1faf
replacing std::auto_ptr by std::unique_ptr
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3640
diff
changeset
|
51 std::unique_ptr<DicomMap> findAnswer_; |
2867 | 52 |
53 public: | |
54 Command(DicomMoveScuJob& that, | |
55 const DicomMap& findAnswer) : | |
56 that_(that), | |
57 findAnswer_(findAnswer.Clone()) | |
58 { | |
59 } | |
60 | |
3658
2d90dd30858c
providing job ID to the IJob::Step() methods
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3640
diff
changeset
|
61 virtual bool Execute(const std::string& jobId) ORTHANC_OVERRIDE |
2867 | 62 { |
63 that_.Retrieve(*findAnswer_); | |
64 return true; | |
65 } | |
66 | |
3763
5ff5d5a0fd28
adding missing ORTHANC_OVERRIDE
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3713
diff
changeset
|
67 virtual void Serialize(Json::Value& target) const ORTHANC_OVERRIDE |
2867 | 68 { |
69 findAnswer_->Serialize(target); | |
70 } | |
71 }; | |
72 | |
73 | |
74 class DicomMoveScuJob::Unserializer : | |
75 public SetOfCommandsJob::ICommandUnserializer | |
76 { | |
77 private: | |
78 DicomMoveScuJob& that_; | |
79 | |
80 public: | |
81 Unserializer(DicomMoveScuJob& that) : | |
82 that_(that) | |
83 { | |
84 } | |
85 | |
86 virtual ICommand* Unserialize(const Json::Value& source) const | |
87 { | |
88 DicomMap findAnswer; | |
89 findAnswer.Unserialize(source); | |
90 return new Command(that_, findAnswer); | |
91 } | |
92 }; | |
93 | |
94 | |
95 | |
96 void DicomMoveScuJob::Retrieve(const DicomMap& findAnswer) | |
97 { | |
98 if (connection_.get() == NULL) | |
99 { | |
3874
2effa961f67f
forcing use of DicomAssociationParameters if creating an SCU
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3870
diff
changeset
|
100 DicomAssociationParameters params(localAet_, remote_); |
3875
ea1d32861cfc
moving timeout from DicomAssocation to DicomAssociationParameters
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3874
diff
changeset
|
101 |
ea1d32861cfc
moving timeout from DicomAssocation to DicomAssociationParameters
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3874
diff
changeset
|
102 if (timeout_ >= 0) |
ea1d32861cfc
moving timeout from DicomAssocation to DicomAssociationParameters
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3874
diff
changeset
|
103 { |
ea1d32861cfc
moving timeout from DicomAssocation to DicomAssociationParameters
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3874
diff
changeset
|
104 params.SetTimeout(static_cast<uint32_t>(timeout_)); |
ea1d32861cfc
moving timeout from DicomAssocation to DicomAssociationParameters
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3874
diff
changeset
|
105 } |
ea1d32861cfc
moving timeout from DicomAssocation to DicomAssociationParameters
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3874
diff
changeset
|
106 |
3874
2effa961f67f
forcing use of DicomAssociationParameters if creating an SCU
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3870
diff
changeset
|
107 connection_.reset(new DicomControlUserConnection(params)); |
2867 | 108 } |
109 | |
110 connection_->Move(targetAet_, findAnswer); | |
111 } | |
3303
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
112 |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
113 |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
114 static void AddTagIfString(Json::Value& target, |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
115 const DicomMap& answer, |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
116 const DicomTag& tag) |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
117 { |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
118 const DicomValue* value = answer.TestAndGetValue(tag); |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
119 if (value != NULL && |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
120 !value->IsNull() && |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
121 !value->IsBinary()) |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
122 { |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
123 target[tag.Format()] = value->GetContent(); |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
124 } |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
125 } |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
126 |
2867 | 127 |
128 void DicomMoveScuJob::AddFindAnswer(const DicomMap& answer) | |
129 { | |
3303
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
130 assert(query_.type() == Json::arrayValue); |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
131 |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
132 // Copy the identifiers tags, if they exist |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
133 Json::Value item = Json::objectValue; |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
134 AddTagIfString(item, answer, DICOM_TAG_QUERY_RETRIEVE_LEVEL); |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
135 AddTagIfString(item, answer, DICOM_TAG_PATIENT_ID); |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
136 AddTagIfString(item, answer, DICOM_TAG_STUDY_INSTANCE_UID); |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
137 AddTagIfString(item, answer, DICOM_TAG_SERIES_INSTANCE_UID); |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
138 AddTagIfString(item, answer, DICOM_TAG_SOP_INSTANCE_UID); |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
139 AddTagIfString(item, answer, DICOM_TAG_ACCESSION_NUMBER); |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
140 query_.append(item); |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
141 |
2867 | 142 AddCommand(new Command(*this, answer)); |
143 } | |
144 | |
145 | |
146 void DicomMoveScuJob::AddFindAnswer(QueryRetrieveHandler& query, | |
147 size_t i) | |
148 { | |
149 DicomMap answer; | |
150 query.GetAnswer(answer, i); | |
151 AddFindAnswer(answer); | |
152 } | |
153 | |
154 | |
155 void DicomMoveScuJob::SetLocalAet(const std::string& aet) | |
156 { | |
157 if (IsStarted()) | |
158 { | |
159 throw OrthancException(ErrorCode_BadSequenceOfCalls); | |
160 } | |
161 else | |
162 { | |
163 localAet_ = aet; | |
164 } | |
165 } | |
166 | |
167 | |
168 void DicomMoveScuJob::SetTargetAet(const std::string& aet) | |
169 { | |
170 if (IsStarted()) | |
171 { | |
172 throw OrthancException(ErrorCode_BadSequenceOfCalls); | |
173 } | |
174 else | |
175 { | |
176 targetAet_ = aet; | |
177 } | |
178 } | |
179 | |
180 | |
181 void DicomMoveScuJob::SetRemoteModality(const RemoteModalityParameters& remote) | |
182 { | |
183 if (IsStarted()) | |
184 { | |
185 throw OrthancException(ErrorCode_BadSequenceOfCalls); | |
186 } | |
187 else | |
188 { | |
189 remote_ = remote; | |
190 } | |
191 } | |
192 | |
193 | |
194 void DicomMoveScuJob::Stop(JobStopReason reason) | |
195 { | |
196 connection_.reset(); | |
197 } | |
198 | |
199 | |
200 void DicomMoveScuJob::GetPublicContent(Json::Value& value) | |
201 { | |
202 SetOfCommandsJob::GetPublicContent(value); | |
203 | |
204 value["LocalAet"] = localAet_; | |
205 value["RemoteAet"] = remote_.GetApplicationEntityTitle(); | |
3303
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
206 value["Query"] = query_; |
2867 | 207 } |
208 | |
209 | |
210 DicomMoveScuJob::DicomMoveScuJob(ServerContext& context, | |
211 const Json::Value& serialized) : | |
212 SetOfCommandsJob(new Unserializer(*this), serialized), | |
3303
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
213 context_(context), |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
214 query_(Json::arrayValue) |
2867 | 215 { |
216 localAet_ = SerializationToolbox::ReadString(serialized, LOCAL_AET); | |
217 targetAet_ = SerializationToolbox::ReadString(serialized, TARGET_AET); | |
218 remote_ = RemoteModalityParameters(serialized[REMOTE]); | |
3303
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
219 |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
220 if (serialized.isMember(QUERY) && |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
221 serialized[QUERY].type() == Json::arrayValue) |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
222 { |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
223 query_ = serialized[QUERY]; |
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
224 } |
3870
09798f2b985f
added a Timeout argument to every DICOM command + 'TargetAet' not mandatory anymore in /retrieve
Alain Mazy <alain@mazy.be>
parents:
3843
diff
changeset
|
225 |
09798f2b985f
added a Timeout argument to every DICOM command + 'TargetAet' not mandatory anymore in /retrieve
Alain Mazy <alain@mazy.be>
parents:
3843
diff
changeset
|
226 // New in Orthanc in 1.7.0 |
09798f2b985f
added a Timeout argument to every DICOM command + 'TargetAet' not mandatory anymore in /retrieve
Alain Mazy <alain@mazy.be>
parents:
3843
diff
changeset
|
227 timeout_ = SerializationToolbox::ReadInteger(serialized, TIMEOUT, -1); |
2867 | 228 } |
229 | |
230 | |
231 bool DicomMoveScuJob::Serialize(Json::Value& target) | |
232 { | |
233 if (!SetOfCommandsJob::Serialize(target)) | |
234 { | |
235 return false; | |
236 } | |
237 else | |
238 { | |
239 target[LOCAL_AET] = localAet_; | |
240 target[TARGET_AET] = targetAet_; | |
3303
a215182a0c2f
"DicomMoveScu" jobs provide the associated C-FIND answer in their "Query" public field
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3095
diff
changeset
|
241 target[QUERY] = query_; |
3870
09798f2b985f
added a Timeout argument to every DICOM command + 'TargetAet' not mandatory anymore in /retrieve
Alain Mazy <alain@mazy.be>
parents:
3843
diff
changeset
|
242 target[TIMEOUT] = timeout_; |
2871
6eebc2eb3168
refactoring serialization of RemoteModalityParameters
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
2867
diff
changeset
|
243 remote_.Serialize(target[REMOTE], true /* force advanced format */); |
2867 | 244 return true; |
245 } | |
246 } | |
247 } |