Update todos

This commit is contained in:
Tadgh 2020-09-14 13:41:11 -04:00
parent 3e324355f2
commit 9a09a8abf3
3 changed files with 1 additions and 3 deletions

View File

@ -59,7 +59,6 @@ public class EmpiMessageHandler implements MessageHandler {
public void handleMessage(Message<?> theMessage) throws MessagingException {
ourLog.info("Handling resource modified message: {}", theMessage);
//TODO GGG TEST THAT THE MESSAGE HEADERS COME IN HERE
if (!(theMessage instanceof ResourceModifiedJsonMessage)) {
ourLog.warn("Unexpected message payload type: {}", theMessage);
return;

View File

@ -63,7 +63,7 @@ public class SubscriptionDeliveringMessageSubscriber extends BaseSubscriptionDel
}
protected void doDelivery(ResourceDeliveryMessage theMsg, CanonicalSubscription theSubscription, IChannelProducer theChannelProducer, IBaseResource thePayloadResource) {
//TODO GGG is this the point at which we can use a BaseResourceModifiedMessage, since technically we no longer have need of a subscriptionId?
//TODO GGG/KHS Question: is this the point at which we can use a BaseResourceModifiedMessage, since technically we no longer have need of a subscriptionId?
ResourceModifiedMessage payload = new ResourceModifiedMessage(myFhirContext, thePayloadResource, theMsg.getOperationType());
payload.setParentTransactionGuid(theMsg.getParentTransactionGuid());
ResourceModifiedJsonMessage message = new ResourceModifiedJsonMessage(payload);

View File

@ -31,7 +31,6 @@ public abstract class BaseJsonMessage<T> implements Message<T>, IModelJson {
@JsonProperty("headers")
private HapiMessageHeaders myHeaders;
/**
* Constructor
*/