IRC Chat : 2013-01-02 - OpenMRS

00:16:22 *** k-joseph has quit IRC
05:15:51 *** kishoreyekkanti has joined #openmrs
06:38:14 *** djazayeri has joined #openmrs
06:38:14 *** ChanServ sets mode: +o djazayeri
07:21:35 *** dkayiwa has joined #openmrs
07:28:13 *** judy has joined #openmrs
07:32:44 <judy> happy new year dkayiwa
07:32:50 <judy> seen my email on dev list cced to you ?
07:32:55 <dkayiwa> judy: happy new year :)
07:33:05 <dkayiwa> judy: even responded already :)
07:34:18 <judy> when is that invite to kampala.? dkayiwa
07:34:28 <judy> i am 2 hours away from the border
07:34:32 <dkayiwa> judy: when do you want it? :)
07:34:59 <judy> march... to escape election troubles in kenya :)
07:36:21 <dkayiwa> judy: lollllllllll
07:36:30 <dkayiwa> judy: granted :)
07:37:12 <judy> you even know lol? when are you coming to kenya?
07:39:05 <dkayiwa> judy: i completely have no idea :)
07:39:43 *** djazayeri has quit IRC
07:43:27 <judy> dkayiwa
07:43:31 <judy> my file is too big
07:43:35 <judy> needs pro
07:43:47 <judy> can i attach the xml output instead?
07:43:51 <judy> needs pro users
07:44:13 <dkayiwa> judy: ok then send me attachment. but privately such that the dev list does not get troubles with the big attachement
07:49:00 <dkayiwa> judy: wowwwww. such a huge form :)
07:49:29 <judy> ha hah .. I know.. The users are better off with forms...
07:49:36 <judy> tahts teh only way tehy will manage...
07:49:42 <judy> I know you discourage it though
07:50:07 <dkayiwa> judy: am not the expert on that :)
07:50:27 <judy> I did a little more of investigation and tried to just run one of the concept sets
07:50:31 <dkayiwa> judy: never new some people love the orange theme :)
07:50:37 <judy> ina new form and it fails
07:50:49 <judy> that specific clinic has an orange theme...
07:51:07 <dkayiwa> judy: oh i see
07:51:18 <dkayiwa> judy: is it an AMPATH clinic?
07:56:03 <judy> no
07:56:33 <judy> its the clinic i have helped out with in kibera
08:03:53 *** dkayiwa has quit IRC
08:05:20 *** dkayiwa has joined #openmrs
08:07:07 <judy> hey dkayiwa did you find the error?
08:07:15 <dkayiwa> judy: yes
08:14:43 <judy> whats is it?
08:14:54 <dkayiwa> judy: a bug in the module :)
08:18:11 <dkayiwa> judy: can you create a ticket for it? will fix it immediately :)
08:34:02 <judy> sawa
08:34:09 <judy> sorry .. sawa = ok
08:34:20 <judy> you should know some kiswahili by now...
08:34:24 <dkayiwa> judy: ofcourse :)
08:34:25 <judy> dkayiwa
08:47:35 <judy> created the ticket @dkayiwa
08:47:52 <dkayiwa> judy: did you add me as a watcher?
09:00:36 <judy> yes
09:00:39 <judy> and here is the link..
09:00:41 <judy> https://tickets.openmrs.org/browse/XFRM-115
09:00:46 <dkayiwa> judy: ok
09:03:21 <dkayiwa> judy: see my comment on the ticket
09:32:38 <judy> works
09:32:43 <judy> dkayiwa
09:32:45 <judy> thank you
09:32:47 <dkayiwa> judy: excellent :)
09:33:47 *** judy has quit IRC
10:00:52 *** rkorytkowski has joined #openmrs
10:00:52 *** ChanServ sets mode: +v rkorytkowski
10:01:05 *** rkorytkowski is now known as rafa
11:31:52 *** kishoreyekkanti has quit IRC
12:18:03 *** jkeiper has quit IRC
12:28:35 *** dkayiwa has quit IRC
12:29:20 *** dkayiwa has joined #openmrs
12:53:37 *** fwdew has joined #openmrs
12:56:05 *** fwdew has quit IRC
12:57:22 *** supun has joined #openmrs
13:12:06 *** supun has quit IRC
13:12:26 *** supun has joined #openmrs
13:20:05 *** jkeiper has joined #openmrs
13:20:05 *** ChanServ sets mode: +v jkeiper
13:24:55 *** wyclif has quit IRC
14:15:36 *** wyclif has joined #openmrs
14:22:24 *** k-joseph has joined #openmrs
14:27:05 *** mseaton has joined #openmrs
14:27:05 *** ChanServ sets mode: +v mseaton
14:28:03 <mseaton> rafa, let me know if you need to talk through reporting at all...
14:40:38 *** mseaton has quit IRC
14:43:55 *** mseaton has joined #openmrs
14:43:55 *** ChanServ sets mode: +v mseaton
14:47:03 *** kishoreyekkanti has joined #openmrs
14:52:08 *** kishoreyekkanti has quit IRC
14:53:53 *** dkayiwa has quit IRC
14:55:12 *** rafa has quit IRC
14:55:15 *** dkayiwa has joined #openmrs
14:59:01 <cpower> .
14:59:21 <cpower> Good morning everyone
15:00:31 <cpower> Are we ready for our Scrum this morning?
15:00:34 *** kishoreyekkanti has joined #openmrs
15:00:35 *** andrea has joined #openmrs
15:00:56 <cpower> Ok, let's begin
15:00:58 *** andrea is now known as Guest57564
15:01:02 <cpower> !scrumon cpower
15:01:02 * OpenMRSBot says the DAILY SCRUM MEETING is STARTING. This meeting should not last longer than 15 minutes. Please hold other comments until the end of the meeting, or message someone privately. Thank you! ScrumMaster cpower- you may begin when ready.
15:01:20 *** Guest57564 is now known as andrea__
15:01:39 <cpower> Order: jkeiper, mseaton, dkayiwa, andrea__, wyclif, and darius when he appears
15:02:40 <mseaton> nothing to report on my end
15:02:55 *** rafa has joined #openmrs
15:02:55 *** ChanServ sets mode: +v rafa
15:02:56 <cpower> ok double next, dkayiwa you are up
15:03:06 <jkeiper> plowing through AMPATH documents today, and cleaning up the REGCORE sprint
15:03:26 <dkayiwa> Committed: Xforms module throws error of "Cannot read property 'd' of null xforms" - XFRM-115
15:03:26 <dkayiwa> About to finish: Potential Memory Leak reported by Tomcat 7 - TRUNK-3056
15:03:27 <dkayiwa> No Blockers
15:03:50 <cpower> andrea__ your turn
15:03:54 <andrea__> Friday 12-21 and last week
15:03:55 <andrea__> TRUNK-2768
15:03:55 <andrea__> Added test for deathdateEstimated (like birthdateEstimated) in PatientServiceTest
15:03:55 <andrea__> Testing in UI
15:03:55 <andrea__> Test passed - but still not retaining value in UI
15:03:55 <andrea__> Wednesday
15:03:57 <andrea__> Finish 1 remaining JSP
15:03:59 <andrea__> Find problem in UI
15:04:01 <andrea__> No blockers
15:04:33 <cpower> wyclif the floor is yours
15:05:33 *** kishoreyekkanti has quit IRC
15:05:37 <wyclif> Monday(worked 4hrs):
15:05:37 <wyclif> emails
15:05:37 <wyclif> reviewing and applying pull requests
15:05:37 <wyclif>
15:05:37 <wyclif> Wednesday:
15:05:39 <wyclif> reviewing and applying pull requests
15:05:41 <wyclif> design call
15:05:43 <wyclif> pick a ticket
15:05:45 <wyclif>
15:05:47 <wyclif> Blockers: None
15:06:18 <rafa> Today:
15:06:18 <rafa> * Continued with providing support for Reporting by MDS.
15:06:18 <rafa> No blockers.
15:06:27 <rafa> * Univ call ;)
15:06:34 <dkayiwa> :D
15:06:56 <cpower> ok, so no blockers or discussion points... for many this is a first day back so get caught up and I let's make 2013 a great year for OpenMRS!
15:07:09 <cpower> !scrumoff
15:07:09 * OpenMRSBot says the DAILY SCRUM MEETING has ENDED. This channel is now returned to normal hacking operations. Post-scrum meeting follow-up conversations may now begin.
15:52:06 *** lh has joined #openmrs
15:52:59 *** k-joseph_ has joined #openmrs
15:54:36 *** k-joseph has quit IRC
16:02:05 *** suranga has joined #openmrs
16:02:05 *** ChanServ sets mode: +v suranga
16:07:22 <cpower> rafa, I'm trying to get my memory back on what we decided. Are you currently working on Reporting + MDS Integration or is that what we are scheduling you to do starting 1/7?
16:10:54 *** hybridivy has joined #openmrs
16:11:20 *** hybridivy has left #openmrs
16:11:58 *** supun has quit IRC
16:12:11 *** supun has joined #openmrs
16:14:17 *** mseaton has quit IRC
16:20:43 <rafa> cpower: I'm currently working on that
16:23:35 *** supun has quit IRC
16:38:07 <cpower> ok thanks rafa
16:39:51 *** mseaton has joined #openmrs
16:39:51 *** ChanServ sets mode: +v mseaton
16:40:35 *** andrea__ has quit IRC
16:58:04 <k-joseph_> dkayiwa: hi
16:58:33 *** k-joseph_ is now known as k-joseph
17:00:01 *** mseaton1 has joined #openmrs
17:01:48 *** mseaton has quit IRC
17:08:31 <suranga> hi jkeiper
17:08:59 <suranga> jkeiper, about that comment tha you just made to trunk 3769...
17:09:17 *** rafa has quit IRC
17:09:45 <suranga> jkeiper, what should be the correct behavior if personId is null ? I initally thought that there will ALways be a patient...
17:11:57 <dkayiwa> k-joseph: hi
17:15:12 <k-joseph> dkayiwa: i have looking at the tests available in the code and it looks to me as if they take up the same form, the test cases that are for rejectIfEmptyOrWhitespace(), is this right? i have again gone through the unit test links
17:15:59 <dkayiwa> k-joseph: like which test have you looked at?
17:17:42 <k-joseph> dkayiwa: FieldTypeValidatorTest, EncounterTypeValidatorTest, PatientIdentifierTypeValidatorTest, ConceptDatatypeValidatorTest
17:18:34 <dkayiwa> k-joseph: can you also send me your pastebin link?
17:18:45 <k-joseph> dkayiwa: ok
17:27:20 *** dkayiwa has quit IRC
17:28:11 <k-joseph> dkayiwa: http://pastebin.com/1Eegm01v
17:28:16 *** dkayiwa has joined #openmrs
17:31:44 <dkayiwa> k-joseph: are you there?
17:31:53 <k-joseph> dkayiwa: yes
17:32:06 <dkayiwa> k-joseph: still waiting for your pastebin
17:32:22 <k-joseph> dkayiwa: http://pastebin.com/1Eegm0
17:32:51 <dkayiwa> k-joseph: that url is not correct
17:32:56 <k-joseph> et by peer )
17:32:56 <k-joseph> [20:28] <k-joseph> dkayiwa: http://pastebin.com/1Eegm0 1v
17:33:00 <jkeiper> suranga, hey man
17:33:22 <k-joseph> dkayiwa: sorry, http://pastebin.com/1Eegm01v
17:33:23 <jkeiper> suranga, that's why i asked the question ... not sure if we are happy with saying patient will always exist on encounter
17:33:41 <jkeiper> suranga, but afaik that's how we roll
17:33:43 <jkeiper> :-D
17:34:06 <suranga> jkeiper, woopsy. but wont validation prevent the encounter from making it this far without a patient assigned ?
17:34:08 <dkayiwa> k-joseph: can you take a close look at FieldTypeValidatorTest ?
17:34:30 <jkeiper> suranga, probably?
17:35:04 <jkeiper> suranga, answer is: i don't know :-D ... but I do know that NPEs are annoying
17:35:22 <jkeiper> suranga, just found one in Concept.addSetMember()
17:35:22 <k-joseph> dkayiwa: looking at it now
17:35:25 <suranga> jkeiper, totally agreee :(
17:35:42 <dkayiwa> k-joseph: look at only this method: validate_shouldFailValidationIfNameIsNullOrEmptyOrWhitespace
17:35:52 <suranga> jkeiper, whom should we consult ? or, what would u recommend ?
17:35:56 <dkayiwa> k-joseph: and tell me if you understand its contents
17:36:53 <k-joseph> dkayiwa: to some extent, i do understand the content
17:37:13 <dkayiwa> k-joseph: what don't you understand about it?
17:37:53 <jkeiper> suranga, we can bring it up briefly on today's design call
17:38:12 <suranga> jkeiper++
17:38:35 <suranga> jkeiper, you know about my connection. but ill try to be there..
17:38:50 <suranga> jkeiper, starts in 3o mins, right ?
17:38:52 <jkeiper> suranga, ok .. i'll bring it up
17:40:55 <dkayiwa> k-joseph: are you there?
17:41:07 <k-joseph> dkayiwa: yes
17:51:52 *** djazayeri has joined #openmrs
17:51:52 *** ChanServ sets mode: +o djazayeri
18:00:06 *** djazayeri has quit IRC
18:00:44 *** djazayeri has joined #openmrs
18:00:44 *** ChanServ sets mode: +o djazayeri
18:19:01 <jkeiper> suranga, there's your answer (in github) :-)
18:19:53 <suranga> jkeiper, ooh, I see it :)
18:20:29 <suranga> jkeiper, looks like we are good to close that ticket then, it seems that alls well :) :)
18:26:25 <jkeiper> :-)
18:33:13 <k-joseph> dkayiwa: paste link== http://pastebin.com/5R2rBV4i
18:33:22 <dkayiwa> k-joseph: ok
18:37:13 *** mseaton1 has quit IRC
18:40:46 <dkayiwa> k-joseph: this is much better than before
18:42:16 <dkayiwa> k-joseph: can you add another @should and corresponding test?
18:42:41 <k-joseph> dkayiwa: which one exactly?
18:42:41 <dkayiwa> k-joseph: something like this:
18:43:00 <dkayiwa> k-joseph: @should pass validation if all required fields are set
18:43:25 <k-joseph> dkayiwa: let me try doing that
18:43:29 <dkayiwa> k-joseph: ok
18:45:44 <k-joseph> dkayiwa: am now having a problem of my validator for description, it has stopped working, which way should i implement for this?
18:46:14 <dkayiwa> k-joseph: what did you change
18:46:26 <dkayiwa> k-joseph: can you pastebin it
18:46:45 <k-joseph> dkayiwa: nothing, ok let me paste bin the two
18:49:49 <k-joseph> dkayiwa: http://pastebin.com/xn9FSUvu
18:50:33 *** mseaton has joined #openmrs
18:50:33 *** ChanServ sets mode: +v mseaton
18:51:47 *** rafa has joined #openmrs
18:51:47 *** ChanServ sets mode: +v rafa
19:01:16 <k-joseph> dkayiwa: still for all the previous tests to pass, i have noticed that i have to change description to Description in RelationshipTypeValidator for that field,
19:01:48 *** cpower has quit IRC
19:01:50 <dkayiwa> k-joseph: for it to pass?
19:02:33 <k-joseph> dkayiwa: yes, i mean to pass the junit test
19:03:01 <dkayiwa> k-joseph: ok good that it now passes
19:24:26 *** k-joseph_ has joined #openmrs
19:25:37 *** k-joseph has quit IRC
19:33:39 *** travis-ci has joined #openmrs
19:33:39 <travis-ci> [travis-ci] [openmrs/openmrs-core] [2cf03b9] [wluyima] The build passed. - http://travis-ci.org/openmrs/openmrs-core/builds/3921001
19:33:39 *** travis-ci has left #openmrs
19:33:42 <OpenMRSBot> <http://ln-s.net/+OW7> (at travis-ci.org)
19:45:12 *** lh has quit IRC
19:53:34 *** djazayeri has quit IRC
19:54:26 *** djazayeri has joined #openmrs
19:54:27 *** ChanServ sets mode: +o djazayeri
20:01:48 *** k-joseph_ is now known as k-joseph
20:05:01 *** mseaton has quit IRC
20:05:39 *** mseaton has joined #openmrs
20:05:39 *** ChanServ sets mode: +v mseaton
20:15:36 *** rafa has quit IRC
21:12:39 <k-joseph> dkayiwa: i have @should pass validation if all required fields are set, pasted at http://pastebin.com/r5KsheL3
21:12:49 <dkayiwa> k-joseph: ok
21:14:26 <dkayiwa> k-joseph: can you look at shouldFailValidationIfaIsToBIsNullOrEmptyOrWhitespace
21:15:27 <k-joseph> dkayiwa: looking at it now
21:17:06 <dkayiwa> k-joseph: if you change from type.setaIsToB(null); to type.setaIsToB("some value"); does it pass?
21:17:37 <k-joseph> dkayiwa: yes, it does
21:18:37 <dkayiwa> k-joseph: can you pastebin the validator
21:19:14 <k-joseph> dkayiwa:when i replace some value with null, the test fails
21:19:42 *** Echidna has quit IRC
21:21:30 <k-joseph> dkayiwa: http://pastebin.com/j9yqddhy
21:22:29 <dkayiwa> k-joseph: can you create a pull request with all your changes
21:22:51 <k-joseph> dkayiwa: ok
21:37:07 *** jkeiper has quit IRC
21:41:34 *** Echidna has joined #openmrs
21:41:34 *** ChanServ sets mode: +v Echidna
21:44:27 *** travis-ci has joined #openmrs
21:44:27 <travis-ci> [travis-ci] [openmrs/openmrs-core] [3efc34f] [wluyima] The build was broken. - http://travis-ci.org/openmrs/openmrs-core/builds/3923135
21:44:27 *** travis-ci has left #openmrs
21:44:31 <OpenMRSBot> <http://ln-s.net/+OYx> (at travis-ci.org)
21:44:57 <k-joseph> dkayiwa: please take a look at this commit, is it okay https://github.com/k-joseph/openmrs-core/commit/25f5b792f35213a8ffe38f91ecf4bb7ae0027718
21:44:59 <OpenMRSBot> <http://ln-s.net/+OZ0> (at github.com)
21:45:05 <dkayiwa> k-joseph: ok
22:31:22 *** dkayiwa has quit IRC
22:34:12 *** mseaton has quit IRC
22:39:04 *** wyclif has quit IRC
22:45:33 *** suranga has quit IRC
22:49:12 *** lh has joined #openmrs
22:49:12 *** lh has joined #openmrs
23:15:29 *** lh has quit IRC
23:39:38 *** k-joseph has quit IRC
23:47:02 *** lh has joined #openmrs
23:47:02 *** lh has joined #openmrs
23:51:33 *** mseaton has joined #openmrs
23:51:33 *** ChanServ sets mode: +v mseaton
23:53:22 *** mseaton has quit IRC