IRC Chat : 2013-04-16 - OpenMRS

00:25:28 *** dkayiwa has quit IRC
00:38:28 *** harshadura_ has joined #openmrs
00:41:33 *** harshadura has quit IRC
00:53:33 *** harshadura_ has quit IRC
00:55:27 *** davwise has joined #openmrs
01:20:34 *** r0bby_ has joined #openmrs
01:20:34 *** ChanServ sets mode: +v r0bby_
01:22:55 *** cospih_ has quit IRC
01:25:11 *** r0bby_ is now known as robbyoconnor
01:25:21 <robbyoconnor> hi all
01:30:10 *** davidpardz has joined #openmrs
01:30:21 *** davidpardz has left #openmrs
01:31:12 *** davidpardz has joined #openmrs
01:33:21 *** davidpardz has quit IRC
01:35:36 *** davidpardz has joined #openmrs
01:40:02 *** robbyoconnor has quit IRC
01:43:27 *** NameFILIP has joined #openmrs
01:46:37 *** harshadura has joined #openmrs
01:59:34 *** Anuruddha has joined #openmrs
02:12:32 *** wyclif has joined #openmrs
02:16:33 *** harshadura has quit IRC
02:17:55 *** harshadura has joined #openmrs
03:03:32 *** annarm has left #openmrs
03:11:17 *** harsz89 has joined #openmrs
03:19:47 *** DraggonZ has joined #openmrs
03:48:46 *** robbyoconnor has joined #openmrs
03:48:46 *** ChanServ sets mode: +v robbyoconnor
04:11:35 *** Anuruddha has quit IRC
04:14:07 *** h3llborn has joined #openmrs
04:32:23 *** GitHub133 has joined #openmrs
04:32:23 <GitHub133> [openmrs-core] wluyima opened pull request #274: Concept search results sometimes link to incorrect concepts - TRUNK-3961 (master...TRUNK-3961) http://git.io/vJsNCg
04:32:23 *** GitHub133 has left #openmrs
04:39:32 *** djazayeri1 has quit IRC
04:40:19 *** h3llborn has quit IRC
04:50:40 *** kavuri has joined #openmrs
05:05:56 *** harshadura has quit IRC
05:26:26 *** davidpardz has left #openmrs
05:28:05 *** NameFILIP has quit IRC
05:28:13 *** harshadura has joined #openmrs
05:50:47 *** portablejim has joined #openmrs
05:50:47 *** portablejim has joined #openmrs
06:03:42 *** kavuri has quit IRC
06:24:55 *** harshadura_ has joined #openmrs
06:27:17 *** harshadura has quit IRC
06:28:43 *** NameFILIP has joined #openmrs
06:49:06 *** rcrichton has joined #openmrs
06:50:47 *** nimby has quit IRC
07:14:35 *** davwise has quit IRC
07:24:34 *** kavuri has joined #openmrs
07:29:35 *** nimby has joined #openmrs
07:38:27 *** harsz89 has quit IRC
07:38:50 *** harsz89 has joined #openmrs
07:53:53 *** kavuri has quit IRC
07:54:31 *** harsha89 has joined #openmrs
07:55:16 *** harsz89 has quit IRC
08:02:26 *** kavuri has joined #openmrs
08:16:48 *** davwise has joined #openmrs
08:25:06 *** k-joseph has joined #openmrs
08:26:30 *** NameFILIP has quit IRC
08:33:53 *** nimby has quit IRC
08:44:19 *** dkayiwa has joined #openmrs
08:50:44 *** joeseff has joined #openmrs
08:50:58 <joeseff> dkayiwa: Hi
08:51:04 <dkayiwa> joeseff: hi
08:51:16 <joeseff> dkayiwa: How are you today?
08:51:24 <dkayiwa> joeseff: good and u
08:51:40 *** garima_ has joined #openmrs
08:51:40 *** davwise has quit IRC
08:52:02 <joeseff> dkayiwa: not so good. I have small problem I'm hoping you could help me solve. It has to do with using an xform for patient registration
08:52:16 <dkayiwa> joeseff: i can try :)
08:52:25 <joeseff> dkayiwa: thanks
08:53:10 *** shortend has joined #openmrs
08:56:40 <joeseff> dkayiwa: I created a patient registration form and saved the xform. I then went to the patient xform design, deleted the existing schema and opened the form that I had created. The problem is the form I created initially has a form id of 28, but when I open the form in the XForm editor from the Patient xform design, it adopts a form id of 0. This brings this error when I try to submit - http://pastebin.com/hAuzg8Bt. Can you help me
09:02:01 <k-joseph> dkayiwa: hi
09:02:45 <dkayiwa> joeseff: hi
09:03:10 <joeseff> dkayiwa: Hello :) Did u get my message?
09:03:28 <dkayiwa> joeseff: yes. what value does your global property for it have right now?
09:03:59 <joeseff> dkayiwa: Bear with me for a moment. I don't know which global property that is
09:05:03 <dkayiwa> how do you enter data in the patient registration form?
09:06:36 <joeseff> dkayiwa: I go to the Find/Create patient page, fill in the required details, then I click on the Create Person button. The form appears after which I enter the details. If you are talking about this attribute: xforms.newPatientFormId - its blank
09:07:07 <dkayiwa> joeseff: do a file save as and send me a pastebin link of the form
09:07:38 <joeseff> dkayiwa: Ok
09:13:21 <joeseff> dkayiwa: Here is the link. http://pastebin.com/a8w4JdCG
09:16:42 <dkayiwa> joeseff: how did you create this form?
09:18:01 <joeseff> dkayiwa: I duplicated the basic form, edited the schema, clicked on Design Xform. I developed it there
09:18:26 <dkayiwa> joeseff: have you read this? https://wiki.openmrs.org/display/docs/XForms+Module+Patient+Registration
09:18:27 <OpenMRSBot> <http://ln-s.net/8a:K> (at wiki.openmrs.org)
09:19:28 <joeseff> dkayiwa: Yes, I found it when I was looking for a way to test the xform. Are you suggesting that I try that method instead?
09:20:01 <dkayiwa> joeseff: what alternative method do you have apart from that which is documented? :)
09:20:54 <joeseff> dkayiwa: I was hoping there was a loophole somewhere. Alas! Thanks, at least I explored my options :)
09:21:06 <dkayiwa> joeseff: :)
09:21:20 <dkayiwa> joeseff: you always start with the official way before hacking :)
09:22:01 <dkayiwa> joeseff: though if you have plenty of time to play, then hacking is cool :)
09:22:28 <joeseff> dkayiwa: Right, I'll keep that in mind. thanks again!
09:24:50 <joeseff> dkayiwa: being new in openmrs, I didn't know that the patient registration form is designed differently from the rest. But I'm sure I'll get better at finding out things before jumping in head first, as time goes by. With a friend like you, I know that won't be too long
09:25:34 *** dkayiwa has quit IRC
09:26:16 *** kavuri has quit IRC
09:26:40 *** kavuri has joined #openmrs
09:26:54 *** dkayiwa has joined #openmrs
09:27:13 <k-joseph> dkayiwa: hi
09:27:21 <dkayiwa> k-joseph: hi
09:29:10 <k-joseph> dkayiwa: i have created one already before, with Note note = new Note(); yet didn't help clear up
09:29:29 <dkayiwa> k-joseph: do you get the same error?
09:29:49 <k-joseph> dkayiwa: yes
09:30:07 <dkayiwa> k-joseph: are you sure?
09:30:55 <k-joseph> dkayiwa, yes am sure also added a constuctor that takes in an integer as a parameter and tried Note note = new Note(1);
09:31:12 <k-joseph> and still didnt help
09:31:14 <dkayiwa> k-joseph: can you pastebin the two errors?
09:31:17 *** joeseff has quit IRC
09:31:36 <dkayiwa> k-joseph: the one you had before and that after instantiating a note object
09:31:56 <dkayiwa> k-joseph: i want to confirm if what you are telling me (that you get the same error) is correct
09:32:41 <k-joseph> getting the two error logs
09:41:19 <k-joseph> dkayiwa before instantiation it is http://pastebin.com/eU91ezEd and after http://pastebin.com/Bh4zyZdq
09:42:11 <dkayiwa> k-joseph: on which line and in which class is the null pointer exception?
09:42:57 <k-joseph> dkayiwa: in the test class
09:43:09 <dkayiwa> dkayiwa: k-joseph: on which line and in which class is the null pointer exception?
09:46:03 <k-joseph> dkayiwa: am debugging the code to check where that is
09:46:32 <dkayiwa> k-joseph: before you even debug, doesn't the error log tell it?
09:50:41 <k-joseph> dkayiwa: the the error log points lots ot at, and am not sure where now and how to fix yet
09:51:55 <dkayiwa> k-joseph: when analyzing an error log, you start by looking at any classes mentioned that your wrote
09:52:16 <dkayiwa> k-joseph: for most likely the bug is not in the other classes but the ones you write
09:52:20 <dkayiwa> k-joseph: does this make sense?
09:52:40 <k-joseph> dkayiwa: fully makes sence
09:53:02 <dkayiwa> k-joseph: so from just the long alone, at what line and in what class is the exception?
09:53:18 <dkayiwa> long = error log
09:53:55 <k-joseph> dkayiwa: at org.openmrs.api.NoteServiceTest.voidNote_shouldVoidTheNoteAndSetTheVoidReason(NoteServiceTest.java:28)
09:54:20 <dkayiwa> dkayiwa: k-joseph: so from just the long alone, at what line and in what class is the exception?
09:55:27 <k-joseph> dkayiwa in NoteServiceTest at line 28!!!
09:56:48 <dkayiwa> k-joseph: can you look there and tell what you think is null?
09:57:49 <k-joseph> dkayiwa: nothing
09:58:07 <k-joseph> dkayiwa: am checking now via the debugger
09:58:10 <dkayiwa> k-joseph: now can you debug and tell which variable is null?
09:58:21 <k-joseph> ok
10:03:50 <k-joseph> dkayiwa: confirmed that at line 28, note is null
10:04:18 <dkayiwa> k-joseph: but you told me that you instantiated it. were you telling me false information?
10:09:43 <k-joseph> dkayiwa, i think that was beecause of line 27, i have commented at it and no longer a nullpointer exception now but a classCastException
10:10:05 <dkayiwa> k-joseph: can you fix it?
10:10:37 <k-joseph> dkayiwa: still trying
10:10:45 <dkayiwa> k-joseph: ok
10:30:19 *** NameFILIP has joined #openmrs
10:38:44 *** harshadura__ has joined #openmrs
10:42:12 *** harshadura_ has quit IRC
10:46:58 *** harsha89 has quit IRC
11:00:53 *** harshadura__ has quit IRC
11:04:27 *** harshadura has joined #openmrs
11:15:31 *** k-joseph has quit IRC
11:20:36 *** Anuruddha has joined #openmrs
11:23:47 *** garima_ has quit IRC
11:36:07 *** effysam has joined #openmrs
11:40:27 *** effysam has quit IRC
11:46:15 *** k-joseph has joined #openmrs
11:47:24 *** harshadura_ has joined #openmrs
11:49:36 *** harshadura has quit IRC
11:50:06 *** garima has joined #openmrs
11:50:58 <garima> dkayiwa: hi, i got your comment, i would change the message today and get back to you
11:51:10 <dkayiwa> garima: ok thanks
11:56:34 <dkayiwa> k-joseph: can you pastebin the log
11:57:32 *** garima has quit IRC
12:02:51 <k-joseph> dkayiwa: http://pastebin.com/2EFEfGiw
12:04:06 <k-joseph> dkayiwa: http://pastebin.com/2EFEfGiw
12:05:01 <dkayiwa> k-joseph: do you know the line and class where it happens?
12:05:21 <k-joseph> dkayiwa: yes i do
12:05:34 <dkayiwa> k-joseph: have you looked at it?
12:05:49 <k-joseph> dkayiwa: yes i have
12:06:21 <dkayiwa> k-joseph: paste that one line here
12:07:24 <k-joseph> return dao.voidNote(note, reason);
12:07:48 <k-joseph> dkayiwa: that is the one in at org.openmrs.notification.impl.NoteServiceImpl.voidNote(NoteServiceImpl.java:119
12:08:11 <dkayiwa> k-joseph: you have now confused me
12:08:16 <dkayiwa> dkayiwa: k-joseph: do you know the line and class where it happens?
12:08:24 <dkayiwa> am asking again as above
12:09:02 <k-joseph> yes, i do, there are two classes where it happens
12:09:34 <k-joseph> dkayiwa: according to the failure trace
12:09:34 <dkayiwa> k-joseph: you always start investigation using the closest class in the stack trace
12:09:56 <k-joseph> dkayiwa: ok, that is return (Note) sessionFactory.getCurrentSession().save(note);
12:09:59 <dkayiwa> k-joseph: for instance if A call B and calls C and calls D, then you start with the innermost
12:10:29 <dkayiwa> k-joseph: what is the return type for getCurrentSession().save()
12:11:24 <k-joseph> dkayiwa: SessionFactory
12:11:59 <dkayiwa> k-joseph: is that the return type for? getCurrentSession().save()
12:13:27 <k-joseph> dkayiwa: am confused what the return type of getCurrentSession() is,
12:13:54 <dkayiwa> k-joseph: do you read what is type?
12:14:32 *** rcrichton_ has joined #openmrs
12:14:39 <k-joseph> dkayiwa: yes i do am replying what is the return type for getCurrentSession().save(
12:14:53 <dkayiwa> k-joseph: is it the same as this?
12:14:54 <dkayiwa> k-joseph: dkayiwa: am confused what the return type of getCurrentSession() is,
12:15:32 *** rcrichton has quit IRC
12:15:53 <k-joseph> dkayiwa: i thought to answer that it requires me to know the return type for what is the return type for getCurrentSession()
12:16:30 <dkayiwa> k-joseph: there is a difference between return type for getCurrentSession() and getCurrentSession().save()
12:16:48 <k-joseph> dkayiwa
12:16:56 <k-joseph> dkayiwa: you are right
12:17:09 <dkayiwa> k-joseph: so what is the return type for getCurrentSession().save()
12:19:27 <k-joseph> dkayiwa: am i right to say that it returns the object passed into save
12:19:40 <k-joseph> ?
12:19:58 <dkayiwa> k-joseph: don't you know how to check for a return type of a method?
12:20:09 <dkayiwa> k-joseph: are you just guessing?
12:21:41 <k-joseph> dkayiwa: the return type of a method is the type defined for that method in the method header
12:21:58 <dkayiwa> k-joseph: so what is it?
12:23:13 <k-joseph> dkayiwa: that is the type of save()
12:23:21 <dkayiwa> k-joseph: yes
12:23:38 <dkayiwa> k-joseph: not type, it is return type
12:36:53 *** cospih has joined #openmrs
12:37:45 *** moin92 has joined #openmrs
12:38:45 <k-joseph> dkayiwa: i have ressolved that statement into two and that has solved that error
12:39:17 <dkayiwa> k-joseph: what change did you do?
12:40:01 <k-joseph> dkayiwa: sessionFactory.getCurrentSession().save(note);
12:40:01 <k-joseph> return note;
12:40:24 <dkayiwa> k-joseph: ok
12:40:35 <k-joseph> dkayiwa: i changed return (Note) sessionFactory.getCurrentSession().save(note); to that above
12:40:59 <dkayiwa> k-joseph: but you have not yet answered my question
12:41:34 <dkayiwa> k-joseph: am not simply interested in your making it work, but also understanding things. so can you answer my pending question?
12:42:45 <k-joseph> dkayiwa: can you please pardon for me which quesstion among all others
12:43:36 <dkayiwa> dkayiwa: k-joseph: what is the return type for getCurrentSession().save()
12:44:06 *** evmw has joined #openmrs
12:44:39 <k-joseph> dkayiwa: the return type for that is the return type for the last method invoked, in this case it is the return type of save()
12:44:59 <dkayiwa> k-joseph: so what is it?
12:50:45 <k-joseph> dkayiwa: dkayiwa: that took me some quick googling, the return type for that is nothing, it is void
12:51:39 <dkayiwa> k-joseph: can you show me where you have seen that?
12:52:15 <k-joseph> dkayiwa: http://www.dil.univ-mrs.fr/~massat/docs/hibernate-3.1/api/org/hibernate/classic/Session.html#save%28java.lang.Object,%20java.io.Serializable%29
12:52:19 <OpenMRSBot> <http://ln-s.net/-1:U> (at www.dil.univ-mrs.fr)
12:52:32 *** moin92 has quit IRC
12:52:43 *** nyoman has joined #openmrs
12:53:04 <dkayiwa> k-joseph: ok good
12:53:12 <dkayiwa> k-joseph: so can you now proceed?
13:03:15 <k-joseph> dkayiwa: actually assertTrue(note.isVoided()); is failing with the trace http://pastebin.com/T5SKeFeN ignoring it passes the test
13:04:15 <dkayiwa> k-joseph: can you also pastebin the test
13:13:29 <k-joseph> dkayiwa: the test case is here http://pastebin.com/iuqFBXEb
13:18:49 <dkayiwa> k-joseph: which of these lines fails?
13:19:35 <k-joseph> dkayiwa: line 34, when i commented it out, it was successful
13:19:53 <dkayiwa> k-joseph: what is your base class for Note?
13:20:24 <k-joseph> dkayiwa: BaseOpenmrsData
13:20:37 <k-joseph> dkayiwa: then also implements Serializable
13:20:58 <dkayiwa> k-joseph: can you pastebin the Note class?
13:21:09 <dkayiwa> k-joseph: oh sorry i have it here
13:21:20 *** kavuri has quit IRC
13:21:36 *** NameFILIP has quit IRC
13:22:03 <dkayiwa> k-joseph: do a commit and i have a review of all that you have
13:22:06 *** DraggonZ has quit IRC
13:22:38 <k-joseph> dkayiwa: ok, so i should remove the assert i had commented?
13:22:59 <dkayiwa> k-joseph: why do you want to remove it?
13:23:28 <k-joseph> dkayiwa: since it is testing nothing when commented out
13:23:32 <dkayiwa> k-joseph: i just want to review such that i tell you where the problem is
13:23:46 <k-joseph> dkayiwa: ok
13:23:48 <dkayiwa> k-joseph: why do you comment it out
13:24:30 <dkayiwa> k-joseph: it is like throwing away your car because you do not have fuel :)
13:24:43 *** ankujarora has joined #openmrs
13:24:47 <k-joseph> dkayiwa: it fails the testcase when un commented out, i think i should not comment it out but seek to make it pass instead,
13:25:00 <dkayiwa> k-joseph: exactly
13:29:08 *** evmw has quit IRC
13:30:26 *** evmw has joined #openmrs
13:36:17 *** evmw has quit IRC
13:38:20 *** DraggonZ has joined #openmrs
13:52:50 *** wyclif has quit IRC
13:55:20 <jkeiper> dkayiwa: nice analogy ;-)
13:55:41 <dkayiwa> jkeiper: lollllllll. Did not know you are awake :D
13:55:59 <jkeiper> dkayiwa: haha, it is nearly 10:00am here
13:56:18 <dkayiwa> jkeiper: ah i see!!! :)
13:56:34 <dkayiwa> jkeiper: you know am not used to waking up that early :)
13:56:59 <jkeiper> dkayiwa: remember, I have little girls ... 7:30am is the latest they will wake me, and I usually have to be up by 6:45am to get to work by 8:00
13:57:17 <dkayiwa> jkeiper: oh now i remember!!! :)
13:57:24 <jkeiper> dkayiwa: so 7:30am is sleeping in hahaha
13:57:40 <dkayiwa> jkeiper: hahahah. sleeping as early as birds :)
13:58:02 <jkeiper> dkayiwa: my wife, on the other hand ... she can sleep for an entire day if given the time
13:58:17 *** isharapremadasa has joined #openmrs
13:58:24 <dkayiwa> jkeiper: OOOOOOOOOOHHHHHHHHHHH :D
13:58:44 *** isharapremadasa is now known as isha
13:59:00 *** wyclif has joined #openmrs
13:59:02 <jkeiper> ahahha
13:59:13 *** isha is now known as ishara-p
13:59:28 *** wyclif has joined #openmrs
14:02:33 <cpower> Good morning!
14:02:48 <cpower> We ready to scrum?
14:03:11 <cpower> silence is golden
14:03:16 <cpower> !scrumon cpower
14:03:16 * 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.
14:05:13 <cpower> Order: mseaton , djazayeri_may_be , dkayiwa , wyclif , Andrea (maybe),
14:05:42 *** k-joseph has quit IRC
14:06:46 <cpower> djazayeri_may_be: are you there?
14:07:03 <cpower> okay, dkayiwa you are up
14:07:46 <dkayiwa> Working on: Investigate possible memory leak in 1.9.0 or one of its bundled modules. - TRUNK-3440
14:07:46 <dkayiwa> Installed and setup Yourkit.
14:07:46 <dkayiwa> Installed and setup tomcat 7 which has a memory leak detector.
14:07:47 <dkayiwa> Started by removing the logic module to see that i have no module at all and see if i can get memory leaks after the application is stopped
14:07:47 <dkayiwa> Now testing to see if
14:08:03 <dkayiwa> i get any memory leak on stopping openmrs
14:08:03 <dkayiwa> No Blockers
14:08:52 *** k-joseph has joined #openmrs
14:09:04 <cpower> wyclif:
14:09:17 <wyclif> Monday:
14:09:17 <wyclif> -Emails
14:09:17 <wyclif> -Created and curated TRUNK-3965, TRUNK-3964
14:09:17 <wyclif> -TRUNK-3961 - Concept search results sometimes link to incorrect concepts
14:09:17 <wyclif> Tuesday:
14:09:18 <wyclif> -Review pull requests
14:09:20 <wyclif> -Pick a ticket in core or bundled module
14:09:22 <wyclif>
14:09:24 <wyclif> Blockers: None
14:11:30 <cpower> Okay anyone else have an update to give?
14:12:22 <cpower> Ok thank you everyone
14:12:26 <cpower> !scrumoff
14:12:26 * 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.
14:13:24 *** k-joseph has quit IRC
14:16:44 <wyclif> dkayiwa, are you in the community dev swimlane this week?
14:17:30 *** bhashitha has joined #openmrs
14:17:36 <wyclif> dkayiwa, ?
14:18:46 <dkayiwa> wyclif: yes
14:19:08 *** k-joseph has joined #openmrs
14:19:21 <wyclif> dkayiwa, i see alot of un reviewed pull requests especially mine, do you mind reviewing them
14:19:44 <dkayiwa> wyclif: sure
14:19:56 <wyclif> dkayiwa, i personally have a bunch of them and it will get harder to merge them if they stay for long unreviewed
14:20:00 <wyclif> dkayiwa, thanks
14:20:26 <dkayiwa> wyclif: rafal recommends that you merge immediately. Unless you are very very unsure of it :)
14:21:21 *** garima has joined #openmrs
14:21:30 <wyclif> dkayiwa, ok
14:21:35 <dkayiwa> wyclif: am now also getting into the habit of merging immediately because am tire of conflicts which keep wasting my time :)
14:21:49 <dkayiwa> wyclif: after all we can always fix it later even if you merged it :)
14:21:50 *** breeze has joined #openmrs
14:22:07 <wyclif> dkayiwa, then i guess i will pushing direct to the master repo
14:22:19 <dkayiwa> wyclif: yes thats even better
14:22:47 <wyclif> dkayiwa, merging is a pain to backport
14:22:55 <dkayiwa> wyclif: the process should make our lives easier instead of giving us more unnecessary work :)
14:23:11 <dkayiwa> wyclif: oh yes
14:23:21 <wyclif> dkayiwa, i.e if you use the github merge button, it becomes hard to cherry-pick the merge commit
14:23:34 <dkayiwa> wyclif: oh i see!!!
14:24:06 <dkayiwa> wyclif: i cherry pick the original (non merged) commit in such cases :)
14:24:19 <wyclif> dkayiwa, hmm..
14:25:01 <wyclif> dkayiwa, the only issue with pushing to master is that tracking code to review becomes tricky
14:25:24 <dkayiwa> wyclif: even when you include the commit url on the ticket?
14:25:31 <wyclif> dkayiwa, because currently i depend on pull requests to find what needs to be reviewed
14:25:55 <dkayiwa> wyclif: me i use this: https://tickets.openmrs.org/secure/Dashboard.jspa?selectPageId=11753
14:26:04 <dkayiwa> wyclif: and look for the Needs review tickets
14:26:30 <dkayiwa> wyclif: so i get the to review urls from the tickets themselves
14:27:32 *** ishara-p has quit IRC
14:27:57 *** bhashitha has quit IRC
14:28:12 <wyclif> dkayiwa, ok
14:29:40 *** bhashitha has joined #openmrs
14:31:17 *** lh has joined #openmrs
14:31:17 *** lh has joined #openmrs
14:33:47 *** garima has quit IRC
14:37:06 *** breeze has quit IRC
14:37:57 *** ankujarora has quit IRC
14:45:17 <k-joseph> dkayiwa: hi
14:45:52 <dkayiwa> k-joseph: hi
14:46:19 <k-joseph> dkayiwa: hi
14:46:46 <k-joseph> dkayiwa: here is the commit https://github.com/k-joseph/openmrs-core/commit/TRUNK-3258
14:46:47 <OpenMRSBot> <http://ln-s.net/+t57> (at github.com)
14:47:12 <dkayiwa> k-joseph: ok will review it today and give you the response on github comments
14:47:14 <dkayiwa> k-joseph: thanks
14:49:56 <k-joseph> dkayiwa: ok
14:49:57 *** DraggonZ has quit IRC
14:51:18 *** bhashitha has quit IRC
14:52:56 <k-joseph> dkayiwa: now about what i previously mailed you about,
14:52:59 *** breeze has joined #openmrs
14:53:11 <k-joseph> what iss the cause of having openmrs failing to run successsfully via eclipse to the browser, even when mvn clean install succeeds running at command line?
14:53:17 <dkayiwa> k-joseph: will respond by tonight
14:53:42 *** kishoreyekkanti has joined #openmrs
14:53:49 <k-joseph> dkayiwa: ok please, thanks a lot too
14:54:50 *** dkayiwa has quit IRC
15:00:58 *** k-joseph has quit IRC
15:04:54 *** ishara-p has joined #openmrs
15:09:29 *** portablejim has quit IRC
15:16:44 *** harshadura_ has quit IRC
15:16:57 *** harshadura has joined #openmrs
15:18:03 *** harshadura has quit IRC
15:19:31 *** rcrichton_ has quit IRC
15:25:48 *** shortend has quit IRC
15:27:10 *** Anuruddha_ has joined #openmrs
15:27:24 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Modules: Kenya EMR UI Library 1.3.2 uploaded to OpenMRS Module Repository <https://modules.openmrs.org/modules/view.jsp?module=kenyaui&ampversion=&amp1.3.2>
15:28:47 *** Anuruddha has quit IRC
15:35:09 *** djazayeri has joined #openmrs
15:35:09 *** ChanServ sets mode: +o djazayeri
15:35:40 *** Anuruddha_ is now known as Anuruddha
15:38:29 *** mogoodrich has joined #openmrs
15:38:41 <mogoodrich> hey wyclif, are you there?
15:40:26 <djazayeri> hi breeze
15:47:15 <mogoodrich> anybody know how we configure Hibernate Interceptors in modules? I'm looking at the Sync and Event modules and don't see any specific configuration besides creating the bean... I feel like I've seen this before, but don't remember where...
15:50:21 *** bhashitha has joined #openmrs
15:51:37 *** NameFILIP has joined #openmrs
15:52:07 <mogoodrich> ah... looks like they are autowired to HibernateSessionFactoryBean...
15:53:22 *** evmw has joined #openmrs
15:57:52 <ishara-p> Hi i am getting this weird error when trying to do some module operations in openmrs 1.10, when i try to use html form entry module, it shows that,
15:58:05 <ishara-p> The page "/openmrs/admin/module/htmlformentry/htmlForms.list" cannot be found. Check the link and try again.
15:58:32 <ishara-p> this just happend one day back and i have used the module without any trouble before that
15:58:50 <ishara-p> can someone help me pls>
15:59:32 <ishara-p> tested with few more modules and all are same
15:59:32 <breeze> djazayeri: hi
15:59:57 <djazayeri> breeze: hope all went well yesterday
16:00:30 <breeze> djazayeri: ok...I have to wait for the insurance companies to fight it out
16:00:52 <djazayeri> breeze: meh, well, hope that isn't too painful
16:01:32 <breeze> djazayeri: nah, my car is still driveable just needs some body work
16:01:46 <djazayeri> breeze: I was looking at https://ci-stg.openmrs.org/allPlans.action and noticing that now we have two builds failing due to the maven 2 / maven 3 issue (about not being able to run that ruby gem)
16:02:14 <breeze> djazayeri: I'll take a look
16:02:23 <djazayeri> breeze: are you working on something now, or can you google around and see if there are any comments about us being able to run that under maven 2?
16:02:34 <breeze> djazayeri: can you give me a link to a log with the specific error?
16:02:38 <djazayeri> breeze: and if not, work with downey to upgrade the maven version on the bamboo server to 3
16:02:56 <djazayeri> breeze: https://ci-stg.openmrs.org/browse/UICM-UICOMMONS/latest
16:03:27 <breeze> djazayeri: thanks. I think the maven version on there was installed with jpackage...so if needs be we can do a manual install
16:03:52 <djazayeri> breeze: meaning that won't be too hard, since it's not the ubuntu-provided one?
16:04:49 <breeze> djazayeri: not sure I totally understand you, but regardless, it will not be too hard
16:05:16 <djazayeri> breeze: okay, in that case don't spend _too_ long exploring how to make it work in maven 2. :-)
16:05:40 <breeze> djazayeri: ok
16:05:52 *** ishara-p has quit IRC
16:06:13 <breeze> djazayeri: is there any news on stuff you brought up last week on combining refapp and PIH work?
16:07:03 <breeze> djazayeri: the reason I ask is there are at least 2 other TWers who'd like to help out part time
16:07:30 <djazayeri> breeze: working on details—I'll send an email later today.
16:07:37 <breeze> djazayeri: ok
16:07:37 <djazayeri> breeze: these are DevOps / CD types?
16:08:25 <breeze> djazayeri: devs who want to get some devops experience as well
16:08:44 <breeze> djazayeri: and one BA/QA if needed as well
16:09:25 <djazayeri> breeze: okay, so I'd say that for the immediate future can you manage the two devops-interested ones yourself, i.e. you can point them to tickets, but make them transparent to the broader group until things have gotten more stable?
16:09:46 <breeze> djazayeri: np
16:12:32 *** kishoreyekkanti has quit IRC
16:15:45 <mogoodrich> djazayeri: don't suppose you have any idea how in OpenMRS we'd go about wiring a hibernate inteceptor to the HibernateTransactionManager instead of the HibernateSessionFactory?
16:20:07 *** harsha89 has joined #openmrs
16:23:43 *** NameFILIP has quit IRC
16:29:33 *** nimby has joined #openmrs
16:35:51 *** annarm has joined #openmrs
16:41:55 <annarm> Hi. I got here via the Outreach Program for Women internship listings. Would be interested in learning more about documentation opportunities, particulary the implementation profiles project you mention on the wiki.
16:47:46 *** DraggonZ has joined #openmrs
16:49:41 *** lh has quit IRC
17:09:28 *** bhashitha_ has joined #openmrs
17:09:37 *** bhashitha has quit IRC
17:09:40 <downey> annarm: Hi! Would be happy to talk with you more about it. Could you drop me a line at michael at OpenMRS dot org? I'm away from the computer today though so may take a bit of time to respond.
17:10:43 *** Mr_Alpha has joined #openmrs
17:11:59 *** Anuruddha has quit IRC
17:12:48 <Mr_Alpha> Hey all,I was trying to build my first module in openmrs.I was trying to create a module using https://wiki.openmrs.org/display/docs/Using+the+Module+Maven+Archetype link and got the following error while building module
17:12:49 <OpenMRSBot> <http://ln-s.net/9$wo> (at wiki.openmrs.org)
17:12:49 <Mr_Alpha> https://wiki.openmrs.org/display/docs/Using+the+Module+Maven+Archetype
17:13:04 <Mr_Alpha> [ERROR] Failed to execute goal org.openmrs.maven.plugins:module-wizard-plugin:1. 1.1:generate (default-cli) on project standalone-pom: null: MojoFailureException : NullPointerException -> [Help 1]
17:13:10 <Mr_Alpha> please help
17:13:18 <Mr_Alpha> I am using a stanalone openmrs
17:13:24 <annarm> downey: thanks, will do!
17:13:54 *** nyoman has quit IRC
17:14:08 <djazayeri> Mr_Alpha: please use pastebin or gist to show the complete error message, and to show what values you had put into the wizard
17:15:02 <Mr_Alpha> ok.Thanks.
17:18:12 <Mr_Alpha> http://pastebin.com/sasAKqG5
17:18:22 <Mr_Alpha> This is the error I am getting
17:19:50 <djazayeri> Mr_Alpha: can you try going through again from scratch?
17:20:11 <djazayeri> Mr_Alpha: I ran into a problem in the past where if you say No to are the above values correct, then things don't work right
17:20:22 <Mr_Alpha> as in installing maven agai?
17:20:46 <Mr_Alpha> *again
17:20:58 <djazayeri> Mr_Alpha: no, just the command like "mvn module-wizard:generate" (or whatever that is)
17:21:14 <Mr_Alpha> ok..will try and get back to tou
17:21:15 <Mr_Alpha> thanks
17:21:35 <Mr_Alpha> should I say yes to everything?
17:21:58 *** djazayeri has left #openmrs
17:22:20 *** k-joseph has joined #openmrs
17:23:36 *** djazayeri has joined #openmrs
17:23:36 *** ChanServ sets mode: +o djazayeri
17:23:55 <djazayeri> Mr_Alpha: it depends what you're trying to do...
17:24:10 <djazayeri> Mr_Alpha: the questions are meaningful...
17:24:15 <djazayeri> brb
17:26:00 <Mr_Alpha> its ot working.Yeah I know the qustion are meaningful.
17:26:02 <Mr_Alpha> http://pastebin.com/fGvh9CSR
17:26:10 <Mr_Alpha> its not working.
17:27:58 <Mr_Alpha> From where I can find a basic mvn project so that I can start working.Maven not working in my pc
17:28:12 <Mr_Alpha> *maven
17:28:43 *** bhashitha_ has quit IRC
17:30:05 <harsha89> hi Mr_Alpha
17:30:29 *** harsha89 has quit IRC
17:30:44 *** harsha89 has joined #openmrs
17:31:13 *** harsz89 has joined #openmrs
17:31:22 <harsz89> hi Mr_Alpha
17:31:43 <harsz89> harsz89, there is basic module structure available in https://github.com/openmrs/openmrs-module-basicmodule
17:31:47 <OpenMRSBot> <http://ln-s.net/-1FC> (at github.com)
17:31:55 <harsz89> harsz89, you may try that
17:31:58 <harsz89> harsz89, :)
17:32:05 <Mr_Alpha> thanks man
17:32:17 <harsz89> harsz89, :)
17:33:58 *** mogoodrich has quit IRC
17:34:34 <Mr_Alpha> harsz89:http://pastebin.com/fGvh9CSR Do you know what th prolem may be? I guess something with the plugin
17:36:42 <harsz89> Mr_Alpha, wait i'll have a look
17:38:41 <harsz89> Mr_Alpha, this is relates to create module from maven architype right?
17:39:26 <harsz89> Mr_Alpha, you can simple use basic module structure to create your module
17:40:10 <harsz89> Mr_Alpha, i'll have a look at building module from maven architype since I not tried it recently :)
17:41:51 *** mogoodrich has joined #openmrs
17:42:03 *** annarm has quit IRC
17:42:50 <DraggonZ> hi, Mr_Alpha! Is there is no pom files in your workspace directory?
17:45:52 *** Mr_Alpha has quit IRC
17:47:07 <sunbiz> Mr_Alpha: look here https://github.com/openmrs/openmrs-contrib-maven-archetype-basicmodule-creation/blob/master/pom.xml
17:47:12 <OpenMRSBot> <http://ln-s.net/-1FW> (at github.com)
17:47:14 <sunbiz> you seem to be using an old version
17:47:24 <sunbiz> oops... he's gone :P
17:47:50 *** lh has joined #openmrs
17:47:50 *** lh has joined #openmrs
17:48:33 <DraggonZ> but old version still should work, i guess :)
17:50:26 <breeze> djazayeri: installing maven3 on the bamboo box fixed the build problem for uicommons, but introduced a deploy problem for all plans, even if they're still set to use maven 2. looking into it, but might need downey's help
17:50:48 <djazayeri> breeze: hmm, okay
17:51:03 <djazayeri> breeze: I don't know anything about this topic, but I can look at the error message if you think it's helpful
17:51:14 <djazayeri> breeze: could it be that we created all the other build plans as maven2 ?
17:52:37 <breeze> djazayeri: we probably did, but not sure why that is effecting deploy...to nexus I mean, not tomcat
17:52:44 <breeze> https://ci-stg.openmrs.org/browse/UICM-UICOMMONS-28/log
17:52:49 <breeze> as an example
17:53:25 <djazayeri> breeze: is it looking in a different place for its config/settings?
17:53:35 <breeze> djazayeri: unautorized seems simple enough of a cause, but not sure why maven3 intrdocued this prob
17:53:36 <djazayeri> breeze: e.g. not ~/.m2/settings.xml
17:53:46 <djazayeri> breeze: maybe being run by a different user?
17:54:18 <breeze> djazayeri: let me check bamboo's settings
18:00:04 *** kishoreyekkanti has joined #openmrs
18:02:14 <breeze> djazayeri: looks like that did it. thanks for the extra set of eyes
18:02:31 <breeze> djazayeri: the settings.xml that is
18:04:19 <DraggonZ> is hithub working now?
18:04:26 <DraggonZ> github, sorry
18:04:36 <breeze> djazayeri: maven3 installed it's global settings.xml in a different location than maven2
18:04:39 *** kishoreyekkanti has left #openmrs
18:08:32 *** djazayeri1 has joined #openmrs
18:10:53 *** djazayeri has quit IRC
18:29:33 *** lh has quit IRC
18:33:57 *** evmw has quit IRC
18:34:34 *** evmw has joined #openmrs
18:43:37 *** jb360 has joined #openmrs
18:55:09 *** djazayeri1 is now known as djazayeri
18:55:09 *** ChanServ sets mode: +o djazayeri
19:03:46 *** k-joseph has quit IRC
19:05:22 <djazayeri> breeze: regarding https://tickets.openmrs.org/browse/RA-28, we're no longer doing what it says in the title right? (build WAR with all relevant modules in the bundledModules folger)?
19:14:33 *** prabhat has joined #openmrs
19:15:47 *** cospih_ has joined #openmrs
19:16:41 *** cospih has quit IRC
19:17:23 *** k-joseph has joined #openmrs
19:36:42 *** DraggonZ has quit IRC
19:36:56 *** DraggonZ has joined #openmrs
19:43:51 <wyclif> mogoodrich, djazayeri mseaton am sorry for my cross postings
19:45:16 <mogoodrich> wyclif: sure, no worries..
19:45:19 <wyclif> mogoodrich, djazayeri mseaton first i hadnt really looked at the code closely, secondly i kept on commeting on ticket/commit and replyin to emails back and forth, but i think its is clear to me with just one issue which Mark is going to fix
19:47:05 <wyclif> mogoodrich, solution looks elegant otherwise, i will test the scenario was bringing up and create a new ticket for it in case am able to reproduce it after applying yoru commit
19:52:03 <mogoodrich> wyclif: sure, thanks... i will add the lines to clear out the thread local (probably not to tomorrow)
19:52:22 <mogoodrich> wyclif: any problem if I do a point release tomorrow? we want to get this rolled out in mirebalais...
19:53:03 <wyclif> mogoodrich, keep in mind clearing those thread locals is what is going to cause the bug i was trying to explain
19:53:37 <wyclif> mogoodrich, does this make sense?
19:54:32 <mogoodrich> wyclif: not exactly... i have to read your response in more detail...
19:54:32 <wyclif> mogoodrich, however not clearing them will avoid the NPE but won't make event module catch those operations in case another interceptor in the chain performs any of them
19:54:39 *** monicaaa has joined #openmrs
19:55:50 <wyclif> mogoodrich, today am pretty open work on anything since we have no current sprint, i will be more than glad to try reproduce the problem, create a ticket and fix since i already have an idea of how to solve it
19:56:17 <wyclif> mogoodrich, would be nice to include it in the release
19:57:01 <wyclif> mogoodrich, i can have it fixed probably end of today
19:57:14 *** dkayiwa has joined #openmrs
19:57:54 <wyclif> mogoodrich, am going to write a test interceptor that does an insert and try to reproduce the problem in a unit test
20:00:07 *** mogoodrich has quit IRC
20:01:33 <DraggonZ> what's wrong with rest module repository? cannot open it on github, 500 internal server error
20:01:48 <DraggonZ> https://github.com/openmrs/openmrs-module-webservices.rest
20:01:49 <OpenMRSBot> <http://ln-s.net/+lGa> (at github.com)
20:04:31 <k-joseph> dkayiwa: hi
20:04:36 <dkayiwa> k-joseph: hi
20:05:05 <k-joseph> dkayiwa: is it ok for us to proceed now
20:05:08 <k-joseph> !!
20:05:09 <OpenMRSBot> k-joseph: Error: "!" is not a valid command.
20:06:01 <dkayiwa> k-joseph: i have not yet reviewed the commit. (planning to review it like 30 mins from now) but we can proceed :)
20:07:21 <k-joseph> dkayiwa: ok, now on the issue of mvn clean install succeeding and openmrs fails to run from eclipse, what is the pottential cause for that?
20:07:32 *** mogoodrich has joined #openmrs
20:07:53 <dkayiwa> k-joseph: can you pastebin the log?
20:08:53 *** garima has joined #openmrs
20:09:05 <harsz89> DraggonZ, looks like something wrong with github
20:10:27 <harsz89> it's looks like github bit going crazy :D
20:10:42 <harsz89> having 500 status
20:14:00 <dkayiwa> harsz89: i thought github was such a good boy to behave like that. :)
20:15:09 <harsz89> dkayiwa, :D :D
20:15:53 <harsz89> dkayiwa, still getting 500 status code :( can't even access to my profile
20:16:04 <harsz89> dkayiwa, did you experience it? :)
20:16:15 <DraggonZ> harsz89, first time see server errors on github )
20:16:36 <harsz89> DraggonZ, same here
20:16:37 <DraggonZ> harsz89, some repositories works fine
20:16:39 <dkayiwa> harsz89: not practically. just by hearsay :)
20:17:00 <harsz89> dkayiwa, :)
20:17:06 <harsz89> DraggonZ, yeah same here
20:18:29 <DraggonZ> wait and believe :)
20:18:57 <k-joseph> dkayiwa: here is the failure trace when i run openmrs under eclipse http://pastebin.com/h0sEahFJ
20:20:03 <dkayiwa> k-joseph: do you have any application running on port 8080?
20:21:35 <k-joseph> dkayiwa: i dont think i do have any
20:22:42 <dkayiwa> k-joseph: can you make jetty run on another port? e.g 8081?
20:24:25 <k-joseph> dkayiwa: i think i need to do a little research on doing that
20:25:52 *** monicaaa has quit IRC
20:30:23 <dkayiwa> k-joseph: hi
20:30:35 <k-joseph> dkayiwa: hi
20:31:02 <dkayiwa> k-joseph: you can use something like this: -Djetty.port=8081 in the run configurations
20:31:43 <dkayiwa> k-joseph: that is under the JRE tab in the VM arguments textaera
20:31:50 <dkayiwa> k-joseph: does that make sense?
20:33:50 *** rday has joined #openmrs
20:34:02 <k-joseph> dkayiwa: it does, still does that replace the initial statement i had there i. e -Xmx1024M -Xms1024M -XX:PermSize=256m -XX:MaxPermSize=256m -XX:NewSize=128m
20:34:57 <dkayiwa> k-joseph: no
20:35:07 <dkayiwa> k-joseph: you just add it there after space
20:35:38 <k-joseph> dkayiwa:done that and now running
20:35:56 <dkayiwa> k-joseph: are you able to login?
20:36:17 <dkayiwa> k-joseph: or atleast see the openmrs page?
20:36:33 <k-joseph> dkayiwa: still running
20:39:45 <garima> dkayiwa: hi
20:39:50 <dkayiwa> garima: hi
20:40:33 <garima> dkayiwa: i have changed the error message to clearly define the valid combinations
20:40:44 <dkayiwa> garima: ok good
20:40:45 <garima> dkayiwa: kindly review
20:40:51 <dkayiwa> garima: ok
20:42:25 <k-joseph> dkayiwa: it fails with error log http://pastebin.com/jAdwrSC7
20:47:25 <dkayiwa> k-joseph: where did you put what i told you?
20:49:55 <k-joseph> dkayiwa: exactly where you said, may be because i first before placed the enter key between the previous statement and the one you gave me
20:50:14 <wyclif> mogoodrich, did you see my last msg to you?
20:50:21 <dkayiwa> k-joseph: what did you place there?
20:51:15 <mogoodrich> wyclif: yes... i am in a meeting now... but will follow up later...
20:51:26 <k-joseph> dkayiwa: Djetty.port=8081
20:51:32 <wyclif> mogoodrich, basically wait to release
20:52:15 <dkayiwa> k-joseph: is that what i told you to put?
20:52:17 <mogoodrich> wyclif: okay, i will get back to you in a bit...
20:52:23 *** breeze has quit IRC
20:52:55 <k-joseph> dkayiwa: k-joseph: you can use something like this: -Djetty.port=8081 in the run configurations
20:52:58 *** annarm has joined #openmrs
20:53:08 <dkayiwa> k-joseph: is that the same as yours?
20:53:11 <k-joseph> dkayiwa: that is where i picked it from
20:53:22 <dkayiwa> k-joseph: are you sure its the same?
20:53:26 <mogoodrich> wyclif: what else do you think i need to do (besides clearing out the thread locals) before it is releasable?
20:53:59 <wyclif> mogoodrich, that is fine, am saying fixing the issue i was bringing up
20:54:40 <wyclif> mogoodrich, am writing a unit test, if it fails i will let you know so that we can included a fix for it before you release
20:54:50 <mogoodrich> wyclif: okay, great...
20:54:53 <k-joseph> dkayiwa: it is the same
20:55:25 <dkayiwa> k-joseph: can you just copy and paste it instead of typing?
20:55:36 <mogoodrich> wyclif: if it fails, can you commit it into a branch (it may help me understand the bug you are trying to explain to me... :)
20:58:47 *** DraggonZ has quit IRC
21:00:56 *** k-joseph has quit IRC
21:04:10 *** annarm has quit IRC
21:04:34 *** annarm has joined #openmrs
21:08:44 *** mogoodrich has quit IRC
21:08:58 *** mogoodrich has joined #openmrs
21:17:29 <wyclif> mogoodrich, it would be good if you had included a unit test with a nested transaction
21:18:11 <mogoodrich> wyclif: yeah, i was thinking of that... any idea of how to go about that? it seems really hard to test transactions within a unit test...
21:18:24 <mogoodrich> wyclif: (i don't think it picks up @Transactional ?)
21:18:30 <mogoodrich> wyclif: i can try tomorrow...
21:19:33 <wyclif> mogoodrich, you need to add a test service with 2 transactional methods where one has propagation required new which gets called by the other
21:20:14 <mogoodrich> wyclif: yeah, fair enough... i can work on that tomorrow... i guess it is easy enough to create a test service...
21:20:21 <wyclif> mogoodrich, you will need to add some log to see if actually transactions gets created as expected
21:20:46 <wyclif> mogoodrich, i guess :)
21:24:06 <mogoodrich> wyclif: cool, i will probably work on it tomorrow morning, so if you do anything tonight that i should look at, send me an email to keep me posted...
21:24:31 <wyclif> mogoodrich, sure i will
21:25:18 *** rday has quit IRC
21:35:03 *** k-joseph has joined #openmrs
21:36:52 <k-joseph> dkayiwa: i had actually left out (-) character, ran well and am now installing it for the first time, thanks
21:41:12 *** rday has joined #openmrs
21:47:29 <dkayiwa> k-joseph: yes thats exactly what i wanted you to see for yourelf :)
21:47:33 <dkayiwa> s
21:47:53 <k-joseph> dkayiwa :)
21:48:04 <dkayiwa> k-joseph: how did you see it? :)
21:49:00 <k-joseph> dkayiwa: that was after made some little googling on Resolving port conflicts on Jetty
21:49:09 <k-joseph> :)
21:49:12 <dkayiwa> k-joseph: ok :)
21:53:15 *** jblaya has joined #openmrs
21:53:38 <jblaya> sunbiz, hey, you around?
21:55:39 <sunbiz> jblaya: yes, Im here
21:56:14 <jblaya> sunbiz, I'm getting an error in the panel and I can't figure out why, do you mind if I ask you?
21:56:19 <sunbiz> cooking, so might be slow to respond... but please ask
21:56:30 *** rday has quit IRC
21:57:24 <jblaya> sunbiz, I've created a new server and put the panel there, and then I've placed the id of one saved patient search, however, when I go to find/create patient I get the following error
21:57:29 <jblaya> sunbiz, SEVERE: Error on page /midoctor/WEB-INF/view/module/midoctorpanel/portlets/midoctorHome.jsp
21:57:29 <jblaya> javax.servlet.jsp.JspException: java.lang.IllegalArgumentException: search must be saved, composition, or must have a class specified
21:57:43 <sunbiz> cooking and EMR go quite well I understand ;-) (that paper reference)!!
21:58:01 <sunbiz> can you pastebin the full stacktrace?
21:58:02 <jblaya> sunbiz, :-D if only hamish were here
21:58:06 <jblaya> sure
21:58:47 <jblaya> sunbiz, http://pastebin.com/SvPpUTiw
21:58:48 <sunbiz> has he ever been in the IRC
21:58:54 <jblaya> sunbiz, I don't think so
21:58:54 <sunbiz> !seen hfrazer
21:58:54 <OpenMRSBot> sunbiz: I have not seen hfrazer.
21:59:01 <sunbiz> !seen hamishf
21:59:01 <OpenMRSBot> sunbiz: I have not seen hamishf.
21:59:11 <sunbiz> nope... the bot hasnt seen him :D
21:59:18 <sunbiz> !seen bmamline
21:59:18 <OpenMRSBot> sunbiz: I have not seen bmamline.
21:59:20 <sunbiz> !seen bmamlin
21:59:20 <OpenMRSBot> sunbiz: I have not seen bmamlin.
21:59:26 <sunbiz> !seen burke
21:59:26 <OpenMRSBot> sunbiz: burke was last seen in #openmrs 1 day, 4 hours, and 14 seconds ago: <burke> I mean... sure is!
21:59:46 <sunbiz> sorry... lemme see that stacktrace... this OpenMRSBot is distracting me ;-)
22:00:44 <sunbiz> jblaya: it seems to be coming from PatientSearchCohortDefinitionProvider.evaluate
22:00:45 <jblaya> sunbiz, your hilarious, and all this while you're cooking
22:00:50 <sunbiz> hmm... it looks odd
22:00:57 <jblaya> sunbiz, yeah, it's funny because other patient searches do work
22:01:00 <sunbiz> mostly why I burn my dishes!
22:01:09 <jblaya> sunbiz, if instead of putting 22 I put 1,2 it works
22:02:17 <sunbiz> looks like that saved search doesnt have a class... or isnt saved
22:02:18 <sunbiz> search must be saved, composition, or must have a class specified
22:02:28 <sunbiz> but if it isnt saved... u could not hadd it to the GP
22:02:47 <sunbiz> *add
22:03:10 <jblaya> sunbiz, could it be because it doesn't have any patients? the ones that don't give me errors have patients
22:03:32 <sunbiz> jblaya: nope, that seems unlikely
22:03:48 *** annarm has quit IRC
22:03:52 <jblaya> sunbiz, ok, because it's actually almost all of the saved patient searches I have
22:03:58 *** annarm has joined #openmrs
22:04:32 <sunbiz> lemme check the reporting compatibility code...
22:04:35 <jblaya> sunbiz, your right it's not that, there's one with a patient and it still gives me that error
22:04:50 *** mogoodrich has quit IRC
22:05:02 <jblaya> sunbiz, I created them with 1.5.8, but when I started seeing these errors, I changed back to 1.5.7, but still getting them
22:05:42 <sunbiz> its coming from here - Class clz = search.getFilterClass();
22:05:42 <sunbiz> if (clz == null)
22:05:42 <sunbiz> throw new IllegalArgumentException("search must be saved, composition, or must have a class specified");
22:06:50 *** cospih_ has quit IRC
22:08:02 <sunbiz> jblaya: Could u describe the cohort??
22:08:16 <sunbiz> like whats it searching?
22:08:25 <jblaya> would you like to get into our system sunbiz, it's completely empty for now
22:08:42 *** annarm has quit IRC
22:08:50 <jblaya> sunbiz, it's all in spanish, but I could translate :-D
22:08:57 <sunbiz> jblaya: I could get into... but if u could describe it to me
22:09:02 <sunbiz> exactly... might be easy
22:09:03 *** cospih has joined #openmrs
22:09:12 *** garima has quit IRC
22:10:01 <jblaya> sunbiz, one for example that gives me problems is a sql query finding patients who have more than 3 of a specific observation
22:10:09 <jblaya> sunbiz, here's the sql code select o1.person_id
22:10:09 <jblaya> from obs o1, obs o2
22:10:09 <jblaya> where o1.concept_id = 113 #formulario de llamada
22:10:09 <jblaya> and o2.concept_id = 112 #resultado
22:10:09 <jblaya> and o1.obs_group_id = o2.obs_group_id
22:10:10 <jblaya> and o2.value_text in ('failed','no-answer')
22:10:11 <jblaya> and o1.voided=0
22:10:13 <jblaya> and o2.voided=0
22:10:15 <jblaya> group by o2.person_id, o1.value_text
22:10:17 <jblaya> having count(o1.person_id) >= 3;
22:10:20 <jblaya> sunbiz, maybe the problem is the sql query
22:12:26 <sunbiz> jblaya: could u add just a gender property... all
22:12:42 <sunbiz> and try making it a composition and try that?
22:12:53 <jblaya> sunbiz, ok
22:14:02 <jblaya> sunbiz, that one works
22:14:22 <sunbiz> hmm... to the same one
22:14:28 *** wyclif_ has joined #openmrs
22:14:29 *** annarm has joined #openmrs
22:14:38 <sunbiz> u did a composition... right??
22:14:41 *** wyclif has quit IRC
22:15:03 <sunbiz> so the results are the same as SQL...
22:15:04 <jblaya> sunbiz, yep, I did the age (saved it), then a concept look up, composition (saved that one)
22:15:07 <jblaya> sunbiz, and both work
22:15:17 <jblaya> let me try a simple sql one
22:16:03 <k-joseph> dkayiwa: hi
22:16:08 <dkayiwa> k-joseph: hi
22:16:09 <jblaya> sunbiz, yep, there it gave the problem
22:16:21 <jblaya> sunbiz, and it was just select patient_id from patient;
22:18:26 <sunbiz> hmm...
22:18:42 <sunbiz> so the prob is with SQL-only searches
22:19:07 <jblaya> sunbiz, let me try a composition with sql
22:19:28 <sunbiz> but if u pair SQL-search with another... using composition it works?
22:20:17 <jblaya> sunbiz, with composition it also does not work
22:20:29 <jblaya> sunbiz, ie a sql-search with a non-sql search
22:20:51 <sunbiz> hmm... will need to investigate deeper
22:21:06 <sunbiz> why that core method is not getting a class
22:21:18 <jblaya> sunbiz, any way in the short term to fix it?
22:21:27 <jblaya> sunbiz, ie that i can fix it
22:21:38 <sunbiz> probably circumvent through the module
22:22:04 <jblaya> what does that mean?
22:22:38 <sunbiz> write some code in the module
22:23:00 <sunbiz> to bypass using that method if its a SQL
22:23:28 <jblaya> sunbiz, but the better fix is for those searches to get a class right?
22:23:58 <sunbiz> yes, OpenmrsUtil has this method
22:24:01 <sunbiz> its part of core
22:25:21 <jblaya> sunbiz, could you give me a brief outline of what a programmer would have to do to fix it, that way I can send it to our programmer
22:27:41 *** k-joseph has quit IRC
22:31:47 <sunbiz> jblaya: thinking...
22:32:02 <jblaya> sunbiz, ok :-D
22:52:18 *** harsz89 has quit IRC
22:53:05 *** annarm has quit IRC
23:08:56 <jblaya> sunbiz, could you email me that?
23:15:42 *** NameFILIP has joined #openmrs
23:21:40 <sunbiz> jblaya: sorry... yup will email
23:22:40 <jblaya> sunbiz, thanks!
23:24:34 *** dkayiwa has left #openmrs
23:25:45 <sunbiz> downey: or anyone else... have u heard about OpenScrum?? the one that pentaho want to use, but hasnt worked very well??
23:26:08 <sunbiz> http://wiki.pentaho.com/display/OpenScrum/07.+Open+Scrum
23:26:12 <OpenMRSBot> <http://ln-s.net/-1Lf> (at wiki.pentaho.com)
23:26:44 <sunbiz> burke, djazayeri cpower ... might be interest to know, what didnt work
23:32:23 *** portablejim has joined #openmrs
23:47:16 *** NameFILIP has quit IRC
23:54:53 *** jblaya has quit IRC