Solving design issue

SO far the code worked really well as expected, then we created patches and submitted them. (by the way he create patches option is pretty cool, it helps to see the changes you made, and you may not even remember). Ben Wolfe suggested that we move the javascript that control the pop-up preview from the "encounterDisplay.jsp" file so that PreviewComplexObstag class would have its own built-in js; which would make the take more reusable. We solved this by correcting the css class, instead. Also we move the resize method from PreviewComplexObstag class to ImageHandler, which made more sense in the design prospective.

thumbnail creation

we are getting closer and closer to the end of our ticket. Since we were avoiding loading the entire image when the user only need to preview the image (memory issue), we had to resize the image to make a thumbnail instead. We resized the image by redrawing the original image and assigning new dimensions.

In our new previewComplexObsTag method, we resize the image and save the its thumbnail, with "thumb_" prefix added on the original title of the image. Because they use the Obs Id to get the link to the complex obs image, we added the view to get the thumbnail; so on the normal hyper link of the complex obs image we added "&view=thumbnail" to return the reference of its thumbnail.
Another issue in resizing was keeping the ratio of the image, but Tyler fixed it too.

Addition isImageHandler() method

When we started this project the main key was to figure out if a complex observation is an image or not, so that we could provide a preview for it. We tried different ways that ended up working, but we were told they were not the best ways to test a complexObs. This week we added an isImageHandler() method in the Obs class (it returns true if the ComplexObs is an instance of ImageHanlder) and we were still trying to save the thumbnail for an image (which we haven't finished yet).


Once we finish saving the thumbnail, we will probably achieve the end of the task required on the ticket.