01:04:39
|
*** Felipo has joined #openmrs
|
01:09:57
|
*** harshadura has joined #openmrs
|
01:12:20
|
*** mseaton has joined #openmrs
|
01:12:20
|
*** ChanServ sets mode: +v mseaton
|
01:40:57
|
*** mathiaslin has joined #openmrs
|
01:45:51
|
*** Felipo has quit IRC
|
01:45:51
|
*** applecool has quit IRC
|
02:03:04
|
*** harshadura has quit IRC
|
02:09:53
|
*** finbrein has quit IRC
|
02:13:19
|
*** mseaton has quit IRC
|
02:20:11
|
*** applecool has joined #openmrs
|
02:32:39
|
*** applecool has quit IRC
|
02:38:33
|
*** jmiranda has quit IRC
|
02:40:10
|
*** applecool has joined #openmrs
|
02:48:09
|
<r0bby_> lh!!!!!!!!!!
|
03:15:30
|
*** upul` has joined #openmrs
|
03:15:30
|
*** ChanServ sets mode: +v upul`
|
03:26:24
|
*** applecool has quit IRC
|
03:34:12
|
*** suranga has joined #openmrs
|
03:34:12
|
*** ChanServ sets mode: +v suranga
|
03:34:39
|
*** pulasthi7 has joined #openmrs
|
03:38:53
|
<suranga> hi djazayeri ... are u still awake ? :)
|
03:39:03
|
* suranga hopes so :)
|
03:39:19
|
<djazayeri> suranga: it's only 8:40pm. yes
|
03:39:39
|
<suranga> wow, great.. djazayeri , i have a question :)
|
03:40:42
|
<suranga> so im supposed to be sending a hl7 message into the rest module.. the hl7 message will contain either 1. Save patient encounter
|
03:40:43
|
<suranga> 2. Query for previous patient encounter(s)
|
03:40:43
|
<suranga> 3. Get encounter...
|
03:41:08
|
<suranga> can u see any possibility that this works in the current setup ? (which I would need to implement?)
|
03:41:52
|
<suranga> what I read up says it should work fine, and that the HL7 hnandler will convert the message into an object... but I just wanted to check :-)
|
03:48:42
|
<suranga> mm... hi djazayeri (hope he is not at dinner:)
|
03:48:57
|
<djazayeri> hi
|
03:49:42
|
<djazayeri> This window isn't at the top of my attention
|
03:50:13
|
<djazayeri> suranga: you're trying to fetch data?
|
03:50:16
|
<djazayeri> or insert it?
|
03:50:27
|
<suranga> both :-)
|
03:50:40
|
*** rohitgoyal18 has joined #openmrs
|
03:50:55
|
<suranga> djazayeri, i need to save an encounter, retreive it or query for previous
|
03:51:46
|
<djazayeri> so, fetching data won't work (via hl7) because that's not implemented in core
|
03:51:54
|
<djazayeri> though you can obviously fetch data via web services
|
03:52:49
|
<djazayeri> suranga: sending an ORU-R01 (to create an encounter) or an ADT-A28 (create patient) will work
|
03:53:17
|
<djazayeri> one key point is that you need to pass a value for hl7source in the rest call
|
03:53:34
|
<djazayeri> actually, that may not be right
|
03:53:42
|
<djazayeri> anyway, it should work, as far as I know.
|
03:53:42
|
<suranga> djazayeri, .. hmm... so this means, to create I send a HL7 message, and to retreive the data I make a web service call, is it ? :)
|
03:54:14
|
<djazayeri> suranga: are you specifically sending data from a non-OpenMRS system, and that's why you want to use hl7?
|
03:54:23
|
<suranga> djazayeri, mm.. sorry, im a bit mixed up now...
|
03:54:39
|
<suranga> djazayeri, yep, thats the idea... so we need to send via Hl7 :)
|
03:54:58
|
<djazayeri> suranga: you can use web services to post an hl7 message to OpenMRS that will create an encounter
|
03:55:04
|
<djazayeri> as far as I know, that will work
|
03:55:37
|
<suranga> aha ! this is done via a call to POST /openmrs/ws/rest/v1/hl7, with the hl87 mesage in the body, correct ?
|
03:56:02
|
<djazayeri> I believe you need to POST some json content
|
03:56:41
|
<suranga> djazayeri, and you mean to say that I cant retrieve via HL7, but only via a web serivce call, right ?
|
03:57:10
|
<djazayeri> suranga: if you want to retrieve an encounter, you'd do that via GET .../ws/rest/v1/encounter
|
03:58:02
|
<suranga> djazayeri, this would retrive an OpenMRS encounter ojbect, correct ? :)
|
03:58:06
|
<djazayeri> yes
|
03:58:31
|
<djazayeri> suranga: and for what it's worth, the most straightforward way to create an encounter is just by doing a POST to .../encounter
|
03:58:39
|
<djazayeri> but you can do it by hl7 if there's some reason to
|
03:59:11
|
<suranga> djazayeri, i will need to check on that... will consult my boss :-)
|
03:59:38
|
<suranga> djazayeri, but please, one last one... can I do a Query for previous patient encounter(s) using hl7 (or the normal HL7) ? :-)
|
03:59:59
|
<djazayeri> suranga: i guess technically you should be able to fetch back an hl7 message that has been sent to OpenMRS.
|
04:00:06
|
<djazayeri> you can't fetch a general encounter
|
04:00:50
|
<suranga> djazayeri, hmm... so web services wont let me retrive an encounter .....?
|
04:01:01
|
<djazayeri> I mean: if you send an hl7 message to create an encounter, I think you can re-fetch that same hl7 message back
|
04:01:27
|
<suranga> oh, the entire hl7 message is stored in the db after processing ? wow !
|
04:01:28
|
<djazayeri> but if someone creates an encounter via some other mechanism, you can't get that as hl7
|
04:03:16
|
<suranga> djazayeri, so if someone creates an encounter via a web service call, I cant get it back as Hl7.... but if I create the encounter using HL7, then I can retreive that old Hl7 message back, OR the encounter object via a web services call ?
|
04:03:27
|
<djazayeri> yes
|
04:03:34
|
<suranga> djazayeri, awesomeee!
|
04:07:41
|
*** rohitgoyal18 has quit IRC
|
04:14:31
|
*** lh has quit IRC
|
04:14:35
|
*** suranga has quit IRC
|
04:20:04
|
*** rohitgoyal18 has joined #openmrs
|
04:36:03
|
*** rohitgoyal18 has quit IRC
|
04:43:28
|
*** suranga has joined #openmrs
|
04:43:28
|
*** ChanServ sets mode: +v suranga
|
04:54:19
|
<suranga> mm.. hi djazayeri ... still there ? :)
|
05:26:14
|
*** goutham has joined #openmrs
|
05:32:14
|
*** deadpool has quit IRC
|
05:42:55
|
*** Teddy_ has joined #openmrs
|
05:44:17
|
*** deadpool has joined #openmrs
|
05:48:25
|
<Teddy_> What is the replacement for Activator class coz its seen as depricated
|
05:49:07
|
*** rohitgoyal18 has joined #openmrs
|
05:50:16
|
*** finbrein has joined #openmrs
|
05:56:52
|
*** bryq has joined #openmrs
|
05:56:52
|
*** ChanServ sets mode: +v bryq
|
05:57:35
|
*** rohitgoyal18 has quit IRC
|
06:05:49
|
*** snoppy has joined #openmrs
|
06:06:15
|
*** bryq has quit IRC
|
06:14:50
|
<goutham> Teddy_:guess replacement for activator class is org.openmrs.module.ModuleActivator
|
06:20:53
|
*** deadpool_ has joined #openmrs
|
06:20:55
|
*** mathiaslin has quit IRC
|
06:24:19
|
*** deadpool has quit IRC
|
06:32:32
|
*** mathiaslin has joined #openmrs
|
06:37:18
|
<Teddy_> goutham: Thanks
|
06:40:01
|
*** goutham has quit IRC
|
06:40:53
|
*** pulasthi7 has quit IRC
|
06:41:13
|
*** deadpool_ has quit IRC
|
06:41:59
|
*** deadpool_ has joined #openmrs
|
06:43:46
|
*** Teddy_ has quit IRC
|
06:47:27
|
*** goutham has joined #openmrs
|
06:52:32
|
*** applecool has joined #openmrs
|
06:53:40
|
*** suranga has quit IRC
|
07:01:47
|
*** dkayiwa has joined #openmrs
|
07:09:53
|
*** deadpool has joined #openmrs
|
07:12:14
|
*** deadpool_ has quit IRC
|
07:22:18
|
<applecool> hi dkayiwa
|
07:22:26
|
<dkayiwa> hi applecool
|
07:22:32
|
<applecool> i added some patches
|
07:23:37
|
<applecool> the update form schema is a small project
|
07:23:46
|
<applecool> cant we merge it with the filters
|
07:23:52
|
<applecool> as an extra credit
|
07:24:09
|
<dkayiwa> applecool: small for GSOC?
|
07:24:38
|
<applecool> i mean three months is a long time
|
07:24:41
|
<applecool> isnt it
|
07:24:58
|
<dkayiwa> applecool: less than that
|
07:26:16
|
<applecool> all my tickets are "In Progress"
|
07:26:23
|
<applecool> would that be OK ?
|
07:26:43
|
<applecool> i am having my internal exams going on and wont be able to solve any more of them
|
07:26:51
|
*** deadpool_ has joined #openmrs
|
07:27:22
|
<applecool> how was you easter dkayiwa
|
07:27:45
|
<dkayiwa> applecool: was a beautiful one. spent it with my parents at their country home :)
|
07:27:50
|
<dkayiwa> applecool: how was yours?
|
07:28:06
|
<applecool> i was busy with my exam preparation
|
07:28:14
|
<applecool> :(
|
07:30:21
|
<applecool> dkayiwa: i tried to give provide as much information as i could
|
07:30:38
|
<applecool> anything that i can do
|
07:30:48
|
*** deadpool has quit IRC
|
07:32:24
|
<applecool> but in the update form schema we would be changing the existing page itself right
|
07:32:34
|
<applecool> dkayiwa:
|
07:32:50
|
<applecool> I estimated very less time in my proposal
|
07:33:30
|
*** Teddy_ has joined #openmrs
|
07:34:16
|
<dkayiwa> applecool: when estimating time for software development, we are always advised to more than double what we think it could take us :)
|
07:34:40
|
<applecool> oh...
|
07:34:47
|
<applecool> would that affect my porposal
|
07:34:54
|
<applecool> i cant change it now :(
|
07:35:22
|
<applecool> i know this time the competetion is very tough
|
07:35:35
|
<applecool> still would try my luck ;)
|
07:40:56
|
*** deadpool has joined #openmrs
|
07:44:21
|
*** deadpool_ has quit IRC
|
07:51:17
|
<applecool> dkayiwa: any further feedback ?
|
07:51:40
|
<dkayiwa> applecool: will give it along the way
|
07:51:52
|
<applecool> didnt get you
|
07:52:06
|
<dkayiwa> applecool: meaning will give if anything crosses my mind
|
07:52:27
|
<dkayiwa> applecool: in the meantime, you can fix as many tickets as you can :)
|
07:54:06
|
<applecool> that would be difficult at the moment , coz i am having my exams , would try fixing them as soon as i am done with them
|
07:54:11
|
<applecool> till then tc
|
07:54:18
|
*** applecool has quit IRC
|
08:06:55
|
*** deadpool has quit IRC
|
08:20:10
|
*** deadpool has joined #openmrs
|
08:27:55
|
*** rafa has joined #openmrs
|
08:28:00
|
*** ChanServ sets mode: +v rafa
|
08:53:59
|
*** dkayiwa has quit IRC
|
08:54:11
|
*** Teddy_ has quit IRC
|
08:58:54
|
*** Teddy_ has joined #openmrs
|
09:08:10
|
*** bryq has joined #openmrs
|
09:08:10
|
*** ChanServ sets mode: +v bryq
|
09:13:25
|
*** deadpool has quit IRC
|
09:21:09
|
*** dkayiwa has joined #openmrs
|
09:24:23
|
<dkayiwa> hi rafa
|
09:24:29
|
<rafa> hi djazayeri
|
09:24:34
|
<rafa> hi dkayiwa :)
|
09:24:48
|
<dkayiwa> rafa: how was your long weekend? :)
|
09:25:41
|
<rafa> dkayiwa: good, thanks! yours?
|
09:26:03
|
<dkayiwa> rafa: great too :)
|
09:26:21
|
<dkayiwa> rafa: do you have any 1.9 tickets that you are still working on?
|
09:26:32
|
<dkayiwa> rafa: or did you finish them all?
|
09:26:42
|
<rafa> dkayiwa: no, I finished last week
|
09:27:11
|
<rafa> dkayiwa: can you test the rth module?
|
09:27:24
|
<dkayiwa> rafa: ok
|
09:27:37
|
<rafa> dkayiwa: it worked for me, but if I remember right you reported some problems
|
09:27:52
|
<rafa> dkayiwa: I'm curious if my fixes will work for you
|
09:28:04
|
<dkayiwa> rafa: it works perfectly for small databases :)
|
09:28:16
|
*** deadpool has joined #openmrs
|
09:28:25
|
<dkayiwa> rafa: did you try a big one like the one we had for the pentaho sprint?
|
09:28:39
|
<rafa> dkayiwa: yes, I tried that and it worked
|
09:29:18
|
<dkayiwa> rafa: oh, did it work only after your fixes? or even before? :)
|
09:29:28
|
<rafa> dkayiwa: after the fixes
|
09:29:39
|
<dkayiwa> rafa: cool, will test it again today
|
09:30:05
|
<rafa> dkayiwa: &zeroDateTimeBehavior=convertToNull is still necessary
|
09:30:13
|
<dkayiwa> rafa: ok
|
09:30:23
|
<rafa> dkayiwa: I added it to the rth wiki
|
09:30:34
|
<dkayiwa> rafa: did you notice my comment on the ticket about BLOBS?
|
09:31:18
|
<rafa> dkayiwa: I don't remember
|
09:31:36
|
<dkayiwa> rafa: just sent it a few mins ago :)
|
09:31:46
|
<rafa> dkayiwa: ohh then no :)
|
09:31:59
|
<dkayiwa> rafa: 8)
|
09:33:10
|
<rafa> dkayiwa: I copied that part over from the sync module
|
09:33:29
|
<rafa> dkayiwa: I'm not sure what is the purpouse of that cast
|
09:33:40
|
<rafa> dkayiwa: I'll think it over
|
09:34:21
|
<dkayiwa> rafa: it completely changes the data :)
|
09:35:41
|
<rafa> dkayiwa: how do you compare it?
|
09:38:10
|
<rafa> dkayiwa: in.read() returns a byte so casting it to char shouldn't change it
|
09:40:16
|
<dkayiwa> rafa: when i looked at the data in the sql dump file
|
09:40:23
|
<dkayiwa> rafa: they look very very different
|
09:40:32
|
<rafa> dkayiwa: but from what?
|
09:41:06
|
<dkayiwa> rafa: the two output.print(char c) from output.print(int i)
|
09:42:03
|
<rafa> dkayiwa: ohh that should be different
|
09:42:11
|
<dkayiwa> rafa: in.read() returns an integer
|
09:42:47
|
<rafa> dkayiwa: yes, but we need to encode it for a dump file as char
|
09:42:48
|
<dkayiwa> rafa: so after casting to char, we are using output.print(char c) instead of output.print(int i)
|
09:43:16
|
<dkayiwa> rafa: ok
|
09:43:21
|
<rafa> dkayiwa: we should check if after import it's decoded correctly
|
09:43:29
|
<dkayiwa> rafa: sure
|
09:43:47
|
<rafa> dkayiwa: i.e. if the source is the same as the destination
|
09:43:59
|
<dkayiwa> rafa: correct
|
09:44:32
|
<rafa> dkayiwa: I copied the code over and I assumed it's right, but it's worth checking
|
09:44:43
|
<dkayiwa> rafa: ok
|
09:45:52
|
<rafa> dkayiwa: formentry_xsn uses longblob
|
09:48:45
|
<dkayiwa> rafa: ok
|
09:50:26
|
*** upul` has quit IRC
|
09:57:56
|
<rafa> dkayiwa: when I try to open a form to enter I get 2012-04-10 11:55:55.460::WARN: /openmrs/moduleServlet/formentry/formDownload
|
09:57:57
|
<rafa> java.lang.UnsupportedOperationException: Templates no longer exist on Forms. Use Form Attributes.
|
09:58:12
|
<rafa> dkayiwa: do you know how to go around that issue?
|
09:59:00
|
*** kmit has joined #openmrs
|
09:59:37
|
<dkayiwa> rafa: ok
|
09:59:41
|
<kmit> \nick Maurya
|
09:59:48
|
*** kmit is now known as maurya
|
09:59:58
|
<maurya> hi sgithens_
|
10:00:01
|
<dkayiwa> rafa: does it allow you enter the data
|
10:00:25
|
<rafa> dkayiwa: no it displays the 500 error page
|
10:00:53
|
<rafa> dkayiwa: I can't open the form
|
10:00:56
|
<dkayiwa> rafa: which version of openmrs
|
10:01:17
|
<rafa> dkayiwa: the latest and imported forms from 1.6 pentaho
|
10:01:45
|
<dkayiwa> rafa: are you using htmlformentry?
|
10:02:03
|
<rafa> dkayiwa: no it's form entry
|
10:02:43
|
<dkayiwa> rafa: the latest formentry module?
|
10:03:28
|
<rafa> dkayiwa: it's the one from 1.6 packaged by the rth module I guess
|
10:03:32
|
<dkayiwa> i think formentry is not yet fixed
|
10:03:32
|
<dkayiwa> that is why we have not even bundled it
|
10:03:41
|
<rafa> dkayiwa: ohh I see
|
10:03:45
|
<rafa> dkayiwa: ok
|
10:04:02
|
<rafa> dkayiwa: then I need a different way to test if blob is transfered correctly
|
10:04:21
|
<dkayiwa> rafa: yes
|
10:04:42
|
*** docaeby has joined #openmrs
|
10:05:15
|
*** deadpool has quit IRC
|
10:06:04
|
*** pulasthi7 has joined #openmrs
|
10:06:22
|
<docaeby> Hello?
|
10:06:57
|
*** deadpool has joined #openmrs
|
10:07:18
|
<docaeby> I'm an obstetrician/gynecologist and working on a masters in ETEC. How can I get involved in OpenMRS?
|
10:08:11
|
*** bryq has quit IRC
|
10:08:14
|
<rafa> docaeby: welcome! See http://openmrs.org/help/
|
10:08:40
|
<docaeby> Thanks!
|
10:12:13
|
<docaeby> I'll be back!
|
10:12:32
|
*** docaeby has quit IRC
|
10:13:46
|
*** Teddy_ has quit IRC
|
10:15:03
|
*** deadpool has quit IRC
|
10:27:05
|
*** deadpool has joined #openmrs
|
10:36:41
|
*** maurya has quit IRC
|
10:42:33
|
*** deadpool has quit IRC
|
10:44:13
|
*** anu_GSOC has joined #openmrs
|
10:56:23
|
*** deadpool has joined #openmrs
|
11:00:05
|
*** deadpool_ has joined #openmrs
|
11:00:36
|
*** james_regen has joined #openmrs
|
11:00:36
|
*** ChanServ sets mode: +v james_regen
|
11:00:46
|
*** deadpool has quit IRC
|
11:05:16
|
<rafa> dkayiwa: hi
|
11:37:26
|
<dkayiwa> hi rafa
|
11:37:47
|
<rafa> dkayiwa: blobs are wrong indeed
|
11:38:06
|
<rafa> dkayiwa: we may need to encode them in hex
|
11:38:07
|
<dkayiwa> rafa: aha tell me
|
11:38:15
|
<dkayiwa> rafa: oh yah
|
11:38:38
|
<dkayiwa> thats the way of transfering blobs as char
|
11:38:49
|
<dkayiwa> rafa: makes perfect sense
|
11:39:46
|
<dkayiwa> so rafa are you working on that now?
|
11:39:49
|
<rafa> dkayiwa: yes
|
11:39:57
|
<dkayiwa> rafa: ok
|
11:56:18
|
*** nribeka has joined #openmrs
|
11:56:18
|
*** ChanServ sets mode: +v nribeka
|
12:16:58
|
<deadpool_> hey has anyone noticed that the mrn number generator jumps some numbers
|
12:17:06
|
<deadpool_> has anyone found a reason why it does that?
|
12:25:42
|
*** mathiaslin has quit IRC
|
12:44:46
|
*** deadpool_ has quit IRC
|
12:49:44
|
*** mathiaslin has joined #openmrs
|
12:51:38
|
*** snoppy has quit IRC
|
12:54:17
|
<dkayiwa> hi rafa
|
12:54:25
|
<rafa> dkayiwa: hi
|
12:54:27
|
<dkayiwa> rafa: can you assign yourself to RELTEST-21
|
12:54:33
|
<rafa> dkayiwa: ok
|
12:54:42
|
<dkayiwa> rafa: thanks
|
12:57:34
|
*** deadpool_ has joined #openmrs
|
12:58:12
|
*** sthaiya has joined #openmrs
|
12:58:22
|
<dkayiwa> rafa: did you see my comment for TRUNK-3241 ?
|
12:58:50
|
*** sthaiya has quit IRC
|
12:59:43
|
<rafa> dkayiwa: it may be I forgot to run the formatter
|
12:59:52
|
*** mvorobey has joined #openmrs
|
12:59:56
|
*** anu_GSOC has quit IRC
|
13:00:23
|
<dkayiwa> rafa: ok, would that require another commit after formatting?
|
13:00:43
|
<mvorobey> hi all
|
13:00:51
|
<dkayiwa> hi
|
13:03:52
|
*** deadpool_ has quit IRC
|
13:06:56
|
*** mseaton has joined #openmrs
|
13:06:56
|
*** ChanServ sets mode: +v mseaton
|
13:07:02
|
*** deadpool_ has joined #openmrs
|
13:11:25
|
<dkayiwa> rafa: do you have sometime to commit the formatted code :)
|
13:17:23
|
*** sgithens_ has quit IRC
|
13:19:12
|
<dkayiwa> hi mseaton
|
13:19:21
|
<dkayiwa> mseaton: do you think we can now close TRUNK-1727?
|
13:19:27
|
<dkayiwa> !ticket TRUNK-1727
|
13:19:28
|
<OpenMRSBot> dkayiwa: [#TRUNK-1727] Inconsistencies between exiting a patient from care and marking them as deceased - OpenMRS JIRA - https://tickets.openmrs.org/browse/TRUNK-1727
|
13:22:05
|
*** snoppy has joined #openmrs
|
13:25:14
|
*** pulasthi7 has quit IRC
|
13:25:43
|
*** deadpool_ has quit IRC
|
13:29:01
|
*** dawn_ has joined #openmrs
|
13:29:09
|
*** ChanServ sets mode: +v dawn_
|
13:34:44
|
<mvorobey> mseaton, hi
|
13:37:06
|
*** goutham has quit IRC
|
13:43:35
|
*** deadpool_ has joined #openmrs
|
13:47:09
|
*** bryq has joined #openmrs
|
13:47:09
|
*** ChanServ sets mode: +v bryq
|
13:47:48
|
*** dkayiwa has quit IRC
|
13:50:41
|
*** dkayiwa has joined #openmrs
|
13:50:43
|
*** rohitgoyal18 has joined #openmrs
|
13:50:49
|
*** Jibesh has joined #openmrs
|
13:59:19
|
*** MarkG has joined #openmrs
|
14:00:33
|
<dawn_> !scrumon dawn
|
14:00:33
|
* 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 dawn- you may begin when ready.
|
14:00:36
|
<dawn_> Hi Team!
|
14:00:52
|
<dawn_> Today's order: wyclif , sgithens , rafa , mvorobey , mseaton, dkayiwa , djazayeri
|
14:00:59
|
<rafa> hi!
|
14:01:09
|
<mvorobey> hi, dawn_
|
14:01:16
|
<sgithens> word
|
14:01:24
|
<dawn_> :D
|
14:01:36
|
<wyclif> Monday:
|
14:01:36
|
<wyclif> * REPORT-49 - Add a mechanism for tagging / categorizing reports and other reporting elements
|
14:01:36
|
<wyclif> * GSoC appication reviews
|
14:01:36
|
<wyclif> Tuesday:
|
14:01:36
|
<wyclif> * TRUNK-2442 - Person attributes do not display on patient search results
|
14:01:37
|
<wyclif> * HTML-71 - Add search to location selector
|
14:01:39
|
<wyclif> Blockers: None
|
14:02:02
|
<dawn_> sgithens: your turn
|
14:02:06
|
<sgithens> !ticket REPORT-337
|
14:02:06
|
<sgithens> Filling in portlet contents
|
14:02:06
|
<sgithens> Will have some questions in a bit about the
|
14:02:06
|
<sgithens> right hand side of the dialog.
|
14:02:07
|
<sgithens> No blockers.
|
14:02:07
|
<OpenMRSBot> sgithens: [#REPORT-337] Improve UI of the Report Evaluation Page - OpenMRS JIRA - https://tickets.openmrs.org/browse/REPORT-337
|
14:02:21
|
<mvorobey> * Re-worked REPORT-312 as portlet
|
14:02:21
|
<mvorobey> * Work in progress - REPORT-353
|
14:02:21
|
<mvorobey> * No blockers
|
14:02:26
|
<rafa> Today:
|
14:02:26
|
<rafa> * Started with REPORT-242: Update reporting dashboard UI to better support systems with lots of reports in a variety of categories
|
14:02:26
|
<rafa> https://tickets.openmrs.org/browse/REPORT-242
|
14:02:26
|
<rafa> * Committed RELTEST-21: BLOB data is not properly dumped when using the Test Installation method
|
14:02:26
|
<rafa> https://tickets.openmrs.org/browse/RELTEST-21
|
14:02:26
|
<rafa> No blockers.
|
14:02:46
|
<dawn_> mseaton: you turn :)
|
14:03:53
|
<dawn_> dkayiwa: let's skip over mike. can you share your updates now, please? :)
|
14:04:10
|
<dkayiwa> Did some 1.9 RC3 ticket administration
|
14:04:11
|
<dkayiwa> Did code review and closed most of the remaining 1.9 RC3 post commits
|
14:04:11
|
<dkayiwa> Now working on: Location hierarchy, tree widget, should be used by the Reporting Module - REPORT-287
|
14:04:12
|
<dkayiwa> No Blockers
|
14:04:34
|
<dawn_> djazayeri: you turn :)
|
14:04:40
|
<dkayiwa> r
|
14:04:53
|
<djazayeri> Monday 09-Apr-2012
|
14:04:53
|
<djazayeri> * (worked on) UIFR-7 (Modules need to be able to provide resources)
|
14:04:53
|
<djazayeri> * (worked on) Create UI Widget Library module so I can finish fixing the UI Framework docs
|
14:04:53
|
<djazayeri> * Created TRUNK-3271, TRUNK-3270
|
14:04:53
|
<djazayeri> * Created EMMA-24 (required modules should have scope=provided)
|
14:04:53
|
<djazayeri> * Quick Bug-Fixer JIRA query for wyclif
|
14:04:53
|
<djazayeri> * Meeting with Jan from ITech
|
14:04:54
|
<djazayeri> Tuesday 10-Apr-2012
|
14:04:55
|
<djazayeri> * Finish UIFR-7
|
14:04:55
|
<djazayeri> * Finish UI Widget Library module
|
14:04:56
|
<djazayeri> * More work on REPORT-330
|
14:04:56
|
<djazayeri> * Prepare for University call tomorrow (about git)
|
14:05:44
|
<dawn_> djazayeri: any blockers?
|
14:05:50
|
<dawn_> mseaton: you there?
|
14:05:52
|
<sgithens> git++
|
14:06:09
|
<djazayeri> a bit stuck on UIFR-7, and trying to reimplement something similar to to ModuleResourcesServlet
|
14:06:13
|
<djazayeri> but I'll figure it out
|
14:06:48
|
<dawn_> !scrumoff dawn
|
14:06:48
|
* 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:07:06
|
<dawn_> Discussion points: 1) dkayiwa : update on 1.9 release 2) mseaton update on sprint progress overall
|
14:07:48
|
<dkayiwa> dawn_: waiting for rafa to finish the BLOB issue on release testing helper module
|
14:07:53
|
*** mathiaslin has joined #openmrs
|
14:08:12
|
<rafa> dkayiwa: I committed it a sec ago
|
14:08:25
|
<dkayiwa> rafa: oh thanks
|
14:08:27
|
<dkayiwa> :)
|
14:09:04
|
<dkayiwa> rafa: can you also respond to TRUNK-3241 ?
|
14:09:19
|
<rafa> dkayiwa: !ticket TRUNK-3241
|
14:09:21
|
<dawn_> dkayiwa: now that rafa committed that ticket, what else needs to be done?
|
14:09:26
|
<rafa> !ticket TRUNK-3241
|
14:09:27
|
<OpenMRSBot> rafa: [#TRUNK-3241] PersonAddresses get voided and recreated on form submission - OpenMRS JIRA - https://tickets.openmrs.org/browse/TRUNK-3241
|
14:09:57
|
<dkayiwa> dawn_: waiting on rafa and wyclif response on TRUNK-3242
|
14:10:28
|
<deadpool_> djazayeri: can i ask you a mysql question?
|
14:10:42
|
<deadpool_> i have a manual counter for patient ids
|
14:10:43
|
<rafa> !ticket TRUNK-3242
|
14:10:45
|
<OpenMRSBot> rafa: [#TRUNK-3242] Overview tab on patient dashboard fails to load when patient has been exited from care - OpenMRS JIRA - https://tickets.openmrs.org/browse/TRUNK-3242
|
14:11:04
|
<deadpool_> sometimes it skips the ids i think because sometimes the transactions fail and it doesn't commit
|
14:11:18
|
<djazayeri> dkayiwa: TRUNK-3242 seems like it has already been backported to 1.9.x, right?
|
14:11:37
|
<djazayeri> deadpool_: okay, so what's the question?
|
14:11:50
|
<djazayeri> deadpool_: (that sounds plausible)
|
14:12:30
|
<dkayiwa> djazayeri: yes
|
14:12:39
|
<deadpool_> so my question is that the counter skips numbers sometimes which is kinda annoying
|
14:12:48
|
<deadpool_> how do we stop a failed transaction
|
14:12:57
|
<deadpool_> or rollback
|
14:12:59
|
<deadpool_> so the counter doesn't get updated
|
14:13:21
|
<djazayeri> dkayiwa: so, what are we actually waiting on for 1.9 RC3?
|
14:13:29
|
<wyclif> hi mseaton
|
14:14:18
|
<dkayiwa> djazayeri: testing and then release RC3
|
14:14:58
|
<djazayeri> deadpool_: there's no general way to do that; depends on what your code looks like. But if you're relying on the internal patient_id primary key being meaningful, that's bad style.
|
14:15:27
|
<deadpool_> i am using the mrn generator
|
14:15:31
|
<deadpool_> and that is what it is using
|
14:15:46
|
<djazayeri> deadpool_: i.e. you shouldn't care if the primary key column skips some values.
|
14:15:56
|
<djazayeri> deadpool_: oh, not the pk, you mean the identifiers...
|
14:16:23
|
<dawn_> dkayiwa: does that mean another week of testing since we were waiting on a couple of tickets?
|
14:16:53
|
<dkayiwa> dawn_: no
|
14:17:12
|
<dkayiwa> dawn_: i plan to release by tmorrow
|
14:17:21
|
<djazayeri> deadpool_: I don't know how that module works; possibly one could tweak the module to reclaim ids if the patient creation fails, but I have no idea.
|
14:17:36
|
<dkayiwa> dawn_: that is assuming you are asking RC3
|
14:20:19
|
<dawn_> dkayiwa: ah, sorry. i mean the final 1.9
|
14:20:54
|
<deadpool_> djazayeri: thanks i will look online to see how to do it
|
14:24:07
|
<dkayiwa> dawn_: yes after one week
|
14:30:52
|
*** MarkG has quit IRC
|
14:33:47
|
<mseaton> hi wyclif
|
14:33:54
|
*** deadpool_ has quit IRC
|
14:34:10
|
*** dkayiwa has quit IRC
|
14:34:53
|
*** rohitgoyal18 has quit IRC
|
14:38:46
|
<wyclif> can look at the comments and UI for TRUNK-1727
|
14:38:56
|
<wyclif> mseaton, can you look at the comments and UI for TRUNK-1727
|
14:39:02
|
<dawn_> mseaton: how is the overall sprint going from your perspective? do you see any issues that may cause us not to finish on time?
|
14:41:34
|
<mseaton> dkayiwa: regarding TRUNK-1727, i don't have strong opinions and have only been somewhat following. if all involved in this ticket are happy with the result, then i'm happy to have it closed. thanks!
|
14:44:31
|
<mseaton> dawn_ sorry i missed the sprint. things are looking good i think. will have a better answer tomorrow for you.
|
14:46:29
|
*** rohitgoyal18 has joined #openmrs
|
14:48:18
|
*** Felipo has joined #openmrs
|
14:50:18
|
*** deadpool_ has joined #openmrs
|
14:52:06
|
*** deadpool has joined #openmrs
|
14:52:54
|
<dawn_> mseaton: okay. thanks!
|
14:54:18
|
<Felipo> hello
|
14:54:38
|
*** deadpool_ has quit IRC
|
14:56:59
|
*** Jibesh has quit IRC
|
14:57:19
|
<Felipo> i am developing a module and load the jsp but no renderized the content, i use tag lib Form
|
14:57:45
|
*** lh has joined #openmrs
|
14:57:45
|
*** lh has joined #openmrs
|
14:59:11
|
<djazayeri> hola Felipo
|
14:59:23
|
<djazayeri> I don't understand
|
14:59:49
|
<djazayeri> You have a jsp that uses <form:form>â¦, but it isn't working?
|
15:04:44
|
<djazayeri> @Felipo, do you have a controller for the page also? What url are you using to access it? One that ends in .form/.list, or .htm?
|
15:05:59
|
*** jrliey has joined #openmrs
|
15:11:54
|
<Felipo> hi
|
15:11:58
|
<djazayeri> hi
|
15:12:01
|
<djazayeri> (si, aqui!)
|
15:13:01
|
<Felipo> no appears my sent msg
|
15:13:17
|
*** juano has joined #openmrs
|
15:13:21
|
<juano> hi all
|
15:13:28
|
<djazayeri> Felipo: are you getting an error message?
|
15:13:32
|
<djazayeri> on the screen, or in the logs?
|
15:14:50
|
<juano> sorry but i have the next problem in the instalation of openMRS 1,9RC2 and Postgresql
|
15:14:52
|
<juano> Server log file output: WARN - OpenmrsUtil.getRuntimePropertiesFilePathName(2464) |2012-04-10 12:01:22,154| Unable to find properties file: C:\\Application Data\OpenMRS\openmrs (1)-runtime.properties WARN - OpenmrsUtil.getRuntimeProperties(2401) |2012-04-10 12:01:22,155| Unable to find a runtime properties file. Initial setup is needed. View the webapp to run the setup wizard. WARN - OpenmrsUtil.getRuntimePropertiesFilePathName(2438)
|
15:15:56
|
<djazayeri> Felipo: are you using jetty or tomcat?
|
15:16:09
|
<juano> ERROR - Context.getServiceContext(255) |2012-04-10 12:09:49,856| serviceContext is null. Creating new ServiceContext()
|
15:16:24
|
<juano> sorry
|
15:16:33
|
<djazayeri> juano: if it's a long error message, use pastebin!
|
15:16:53
|
<juano> but the specific error is ERROR - Context.getServiceContext(255) |2012-04-10 12:09:49,856| serviceContext is null. Creating new ServiceContext()
|
15:17:27
|
<djazayeri> Felipo: yes, you need to fix that error
|
15:17:37
|
<djazayeri> maybe it should be "email" instead of "Email"?
|
15:17:58
|
<djazayeri> juano: we'd need to see the full error message
|
15:19:14
|
<djazayeri> juano: and I have zero experience using OpenMRS with postgresql...
|
15:19:18
|
<juano> ok the error ... http://pastebin.com/fRVCxTCK
|
15:19:57
|
<djazayeri> juano: did you run the initial setup wizard once?
|
15:20:51
|
*** nribeka has quit IRC
|
15:21:33
|
<juano> yes, you run the wizard and create the tables when you get this erroe in the log.
|
15:21:37
|
*** dawn_ has quit IRC
|
15:21:57
|
<juano> yes, running the wizard and create the tables when you get this erroe in the log.
|
15:22:12
|
<juano> error, sorry
|
15:22:58
|
<djazayeri> juano: can you check whether the runtime properties file that it mentions exists?
|
15:23:12
|
<djazayeri> it seems like you deployed something like "openmrs (1).war" to tomcat?
|
15:23:41
|
<juano> yes
|
15:23:49
|
<juano> openmrs(1).war
|
15:24:11
|
<djazayeri> I assume tomcat is okay with having ( and ) in the context path?
|
15:25:02
|
*** p_m has joined #openmrs
|
15:25:16
|
*** harshadura has joined #openmrs
|
15:25:34
|
*** nribeka has joined #openmrs
|
15:25:34
|
*** ChanServ sets mode: +v nribeka
|
15:25:40
|
<djazayeri> juano: anyway, is there an "openmrs (1)-runtime.properties" file somewhere?
|
15:25:58
|
<djazayeri> it's possible that OpenMRS isn't dealing well with the space and the parens, but I don't know.
|
15:29:33
|
*** deadpool has quit IRC
|
15:30:09
|
*** deadpool has joined #openmrs
|
15:30:28
|
*** rafa has quit IRC
|
15:33:02
|
*** suranga has joined #openmrs
|
15:33:02
|
*** ChanServ sets mode: +v suranga
|
15:34:27
|
<juano> ok thans
|
15:38:46
|
*** juano has quit IRC
|
15:39:45
|
*** p_m has quit IRC
|
15:53:47
|
*** p_m has joined #openmrs
|
16:05:34
|
*** p_m has quit IRC
|
16:10:24
|
*** isharapremadasa has joined #openmrs
|
16:13:08
|
*** snoppy has quit IRC
|
16:17:15
|
*** isharapremadasa is now known as isharap
|
16:18:44
|
<isharap> hi i'm having some problem in testing with TRUNK-248, and need some help in working with hibernate criterias?
|
16:19:46
|
<isharap> i am writing a unit test to check 'prepare a criteria from a patient name and identifier'
|
16:20:32
|
<isharap> for PatientSearchCriteriaTest class prepareCriteriaTest method
|
16:20:54
|
<isharap> i created a session and added a criteria for Patient.class
|
16:21:28
|
<isharap> but i have no clue how this refers to the patient data in a table or whereever
|
16:22:26
|
<isharap> can someone pls tell me where does this criteria look for patient data when it is created?
|
16:24:33
|
*** p_m has joined #openmrs
|
16:30:13
|
*** p_m has quit IRC
|
16:31:32
|
<djazayeri> isharap: have you skimmed http://docs.jboss.org/hibernate/core/3.3/reference/en/html/querycriteria.html ?
|
16:31:35
|
<OpenMRSBot> <http://ln-s.net/9z4b> (at docs.jboss.org)
|
16:31:46
|
<djazayeri> generally the Patient object is mapped to the patient table
|
16:31:55
|
<djazayeri> and it extends the person table
|
16:32:05
|
<djazayeri> so it has a collection of rows in the person_name table, etc
|
16:35:07
|
*** p_m has joined #openmrs
|
16:39:11
|
*** Felipo has quit IRC
|
16:39:40
|
<isharap> Is there any test records table or etc. that i can use with this criteria? as when i create the criteria and calls it with a given name, it always returns an empty list
|
16:40:02
|
<isharap> it is may be that the name doesnt included in the records
|
16:40:18
|
<isharap> however i tried with Context classes entries but didnt succeed
|
16:40:40
|
<djazayeri> isharap: if you're writing a unit test, make sure it extends BaseContextSensitiveTest, and then it will have data (defined in standardTestDataset.xml)
|
16:40:55
|
*** p_m has quit IRC
|
16:43:11
|
<isharap> djazayeri : yes i extended BaseContextSensitiveTest, but i was using PatientServiceTest-findPatients.xml file's data. can't this be done using that?
|
16:43:24
|
<djazayeri> isharap: yes, it can be done using that
|
16:43:35
|
<djazayeri> did you do executeDataSet() ?
|
16:43:40
|
<djazayeri> e.g. in an @Before?
|
16:43:44
|
<djazayeri> or in the test itself?
|
16:44:17
|
<isharap> yes, executeDataSet(FIND_PATIENTS_XML) but inside my test method
|
16:45:42
|
<djazayeri> isharap: then I don't know why doing a criteria query won't find them
|
16:45:47
|
<djazayeri> can you pastebin your test method?
|
16:45:52
|
<djazayeri> or test class?
|
16:46:24
|
<isharap> sure i will do it now
|
16:46:49
|
<mseaton> hi mvorobey
|
16:47:01
|
<mvorobey> mseaton, hi
|
16:47:28
|
<mseaton> mvorobey: good work on the cron widget. unfortunately, i have a few issues and a few requests :)
|
16:47:49
|
<mvorobey> mseaton, go ahead, am listening :)
|
16:47:57
|
<mseaton> how would you prefer to walk through those? here? on the ticket? on the review?
|
16:48:32
|
<mvorobey> mseaton, I would prefer to have them listed on ticket
|
16:48:40
|
<mseaton> ok, will do
|
16:49:01
|
<mvorobey> mseaton, thanks for quick review :)
|
16:49:16
|
<mseaton> one question mvorobey - where did you get that javascript - did you write it yourself? or did you find it online?
|
16:49:34
|
<mvorobey> mseaton, I wrote it by myself
|
16:49:44
|
<mseaton> mvorobey: wow, great. nice!
|
16:50:52
|
<mvorobey> mseaton, You are putting me to the blush :)
|
16:51:49
|
<isharap> djazayeri: http://pastebin.com/RzfGapFy this is the test method
|
16:54:10
|
*** p_m has joined #openmrs
|
16:54:46
|
*** suranga has quit IRC
|
16:55:18
|
*** suranga has joined #openmrs
|
16:55:59
|
<djazayeri> isharap: try moving the executeDataSet to the top of the method
|
16:56:11
|
<djazayeri> and do an assert for ensuring that testpatient is not null
|
16:56:40
|
<djazayeri> if you're getting back a result from there, but not getting back a result from criteria.list(), then you might be calling the criteria wrong
|
16:57:03
|
<djazayeri> isharap: I don't know what the code for PatientSearchCriteria looks like, but is it supposed to be a sub-criteria? or an independent one?
|
16:58:39
|
<isharap> it prepares a criteria by taking the name and identifier of the pateint
|
16:58:56
|
<isharap> wait i will paste it for you djazayeri :)
|
16:59:09
|
<djazayeri> isharap: I don't particularly want to look at it now...
|
16:59:13
|
<djazayeri> :-)
|
16:59:21
|
<djazayeri> that was more of a hint for you
|
16:59:44
|
*** p_m has quit IRC
|
17:00:50
|
*** lh has quit IRC
|
17:01:39
|
<isharap> sure i will try the swapping executeDataSet() and see if anything returns in that case
|
17:01:39
|
<isharap> :)
|
17:02:20
|
<djazayeri> isharap: if you verify that the service call *is* giving you a patient, then the problem is the way you're calling the criteria...
|
17:02:21
|
* suranga sees everyone is busy so goes back to have another go :-)
|
17:03:11
|
*** lh has joined #openmrs
|
17:05:58
|
<isharap> i did debug the code and testpatient is not null, it has all the fields in it. I simply cant get it applied through prepareCriteria methodd
|
17:07:18
|
<isharap> i fel that i am not exactly getting what this prepareCriteria method exactly does :(
|
17:07:34
|
<isharap> is there anyone i can contact for tht?
|
17:08:04
|
<djazayeri> isharap: I'm not sure that method supports giving it both name and identifier
|
17:08:16
|
<djazayeri> isharap: try just giving it one?
|
17:11:39
|
*** p_m has joined #openmrs
|
17:15:56
|
<mvorobey> mseaton, I have couple of questions about your comments on review
|
17:16:10
|
<mseaton> mvorobey: sure, still in progress, but happy to discuss
|
17:16:43
|
<mvorobey> mseaton, ok, then see my replies on your comments :)
|
17:19:22
|
*** p_m has quit IRC
|
17:20:18
|
*** dawn_ has joined #openmrs
|
17:20:18
|
*** ChanServ sets mode: +v dawn_
|
17:20:38
|
<mvorobey> mseaton, I can not clearly understand what you meant about date format localization on datepicker
|
17:21:38
|
<isharap> djazayeri : i did what you told me and figured out what i have been doing wrong, actually the original method restrcits the voided pateint entries from the criteria, that is why it gave an empty list, i have been using the asserting wrong
|
17:22:01
|
<isharap> will create a patch and add soon with the tests
|
17:22:09
|
<isharap> thanks a lot for the guide :)
|
17:22:39
|
<djazayeri> np
|
17:32:12
|
*** p_m has joined #openmrs
|
17:32:25
|
<suranga> jkeiper, helooo
|
17:32:30
|
<suranga> jkeiper, r u here ?
|
17:37:14
|
*** harshadura has quit IRC
|
17:42:05
|
*** jrliey has left #openmrs
|
17:43:15
|
*** isharap has quit IRC
|
17:48:14
|
*** lh has quit IRC
|
17:52:06
|
*** lh has joined #openmrs
|
17:52:06
|
*** lh has joined #openmrs
|
17:59:26
|
*** p_m has quit IRC
|
17:59:42
|
*** Collins has joined #openmrs
|
18:00:44
|
<Collins> How do I get the name of a location to be displayed in a jsp file in a module?
|
18:06:52
|
*** kernelhunter92 has joined #openmrs
|
18:11:54
|
*** wyclif has quit IRC
|
18:13:38
|
*** p_m has joined #openmrs
|
18:17:06
|
<Collins> How do I get the name of a location to be displayed in a jsp file in a module?
|
18:28:00
|
*** jkeiper has quit IRC
|
18:40:17
|
*** suranga has quit IRC
|
18:40:17
|
<djazayeri> Collins: you mean something like ${ loc.name } ?
|
18:40:32
|
<djazayeri> Collins: or better: <openmrs:formatlocation="${loc}"/>
|
18:43:02
|
*** p_m has quit IRC
|
18:45:07
|
*** rohitgoyal18 has quit IRC
|
18:46:54
|
<Collins> djazayeri: please check the code at http://pastebin.com/NHFbXRLk
|
18:48:05
|
<djazayeri> Collins: what's the question? I see you're iterating over locations, and you are displaying some of their values
|
18:48:08
|
<djazayeri> (properties)
|
18:49:34
|
<Collins> Am trying to display location names on the jsp page. Sorry, the network is shaky over here.
|
18:49:53
|
<djazayeri> Collins: ${ locations.name } should work
|
18:50:06
|
<djazayeri> just like you did with locationId, but use the name property
|
18:51:10
|
<Collins> djazayeri: let me try that. thanks
|
18:52:08
|
<mvorobey> mseaton, have a minute ?
|
18:52:28
|
<mseaton> mvorobey: sure
|
18:53:52
|
<Collins> djazayeri: It worked. Thanks alot
|
18:54:06
|
<mvorobey> mseaton, I am afraid that we can not use dateField widget within our cron editing widget because it loads via iframe that has -index lower that modal window and we can not change iframe z-index at all. may be you know any workaround on this ?
|
18:56:35
|
*** wyclif has joined #openmrs
|
18:57:21
|
<mseaton> mvorobey: if you can't use dateField.tag, then try just using the code that is encapsulated in it. you can see it uses the <openmrs:formatDate.../> tag to display the defaultValue.
|
18:57:50
|
*** deadpool has quit IRC
|
18:58:26
|
*** deadpool has joined #openmrs
|
19:07:08
|
<mvorobey> mseaton, but how it can help us ?
|
19:07:45
|
<mvorobey> mseaton, it won't solve issue with locale specific date format
|
19:07:55
|
<mseaton> mvorobey: it will not?
|
19:08:47
|
<mvorobey> mseaton, jsp tag openmrs:formatDate renders on server, and datepicker works in client browser at least
|
19:08:54
|
*** MarkG has joined #openmrs
|
19:10:47
|
<mvorobey> mseaton, basically, as workaround we can use jquery datepicker and pass <openmrs:datePattern/> as dateFormat to it
|
19:11:03
|
<mseaton> mvorobey: that'll probably work, yes.
|
19:11:25
|
<mvorobey> mseaton, good :) ok
|
19:11:37
|
*** Felipo has joined #openmrs
|
19:11:59
|
<Felipo> Hello, the Module Maven Archetype must be replace the tags in the config.xml and in the messages.properties file??
|
19:17:51
|
<mvorobey> mseaton, shoot, it still won't work because openmrs date format is dd/mm/yyyy which is not a valid javascript date format (dd/mm/yy) because 4 digits year in javascript has patter yy :(
|
19:18:15
|
<mseaton> mvorobey: i think there is an existing solution here. djazayeri ?
|
19:20:24
|
<mvorobey> mseaton, how about jQuery('#${id}once-scheduleDate').datepicker({ dateFormat : '<openmrs:datePattern/>'.replace('yyyy', 'yy')}); ?
|
19:20:38
|
<djazayeri> mvorobey: look and see how it's done in the htmlformentry module, for example
|
19:20:45
|
<djazayeri> or, presumably, in core
|
19:21:46
|
<mseaton> mvorobey: can you use the dateField.tag widget, and make it's z-index lower by wrapping it in a span?
|
19:22:51
|
<mvorobey> mseaton, will it work ? i think that iframe has own z-index and it does not inherit from span
|
19:23:16
|
<mseaton> mvorobey: i mean to change the z-index of the datepicker
|
19:24:39
|
<mvorobey> mseaton, but openmrs datepicker is provided by iframe and we can not modify it
|
19:25:21
|
<mseaton> mvorobey: maybe with css?
|
19:25:33
|
<mseaton> on it's class?
|
19:25:48
|
<mvorobey> mseaton, we can not modify css of iframe too
|
19:28:00
|
<mvorobey> mseaton, but jQuery('#${id}once-scheduleDate').datepicker({ dateFormat : '<openmrs:datePattern/>'.replace('yyyy', 'yy')}); works fine for me
|
19:29:12
|
<mseaton> mvorobey: leave it this way for now to get it working. we can raise another ticket as needed to convert this to use a standard openmrs datepicker tag in the future.
|
19:29:49
|
<mvorobey> mseaton, ok, it seems to be more reasonable for now
|
19:30:08
|
<mseaton> djazayeri: i want to get REPORT-233 ready for work
|
19:30:13
|
<mseaton> !ticket REPORT-233
|
19:30:14
|
<OpenMRSBot> mseaton: [#REPORT-233] Reports should have a published vs non-published property, which controls what reports are displayed on Reporting homepage - OpenMRS JIRA - https://tickets.openmrs.org/browse/REPORT-233
|
19:30:34
|
<mseaton> we have been going back and forth on ticket comments. can we find a resolution?
|
19:31:33
|
<mseaton> and djazayeri, on a related note, should we modify REPORT-49 in some way to support localizing tags for display?
|
19:31:39
|
<mseaton> !ticket REPORT-49
|
19:31:40
|
<OpenMRSBot> mseaton: [#REPORT-49] Add a mechanism for tagging / categorizing reports and other reporting elements - OpenMRS JIRA - https://tickets.openmrs.org/browse/REPORT-49
|
19:31:58
|
*** lh has quit IRC
|
19:32:39
|
*** dawn_ has quit IRC
|
19:34:46
|
<djazayeri> mseaton: so, what approach are you suggesting?
|
19:35:07
|
<djazayeri> that we allow messages.properties to do the localization?
|
19:35:26
|
<djazayeri> and that when someone creates one, we do a reverse lookup in messages.properties?
|
19:36:06
|
<mvorobey> mseaton, I finished re-working cron-editing widget as tag and fixed all your comments. Please, see update review on that ticket.
|
19:36:53
|
<mseaton> djazayeri: well, not sure. we could potentially add a property to the DefinitionTag class called "displayCode", which if non-null, tries to display the tag with this message code. or we could do something like the metadata localization project aimed to do, where for displayCode we allow "fr:Oui:en:Yes:es:Si"
|
19:37:02
|
<mseaton> mvorobey: great! will look soon
|
19:37:12
|
<mvorobey> mseaton, thanks :)
|
19:37:54
|
<djazayeri> mseaton: I like the metadata localization project idea, but not sure about putting it into just the reporting module
|
19:38:06
|
<mseaton> but djazayeri, if we do this, we probably want to have DefinitionTag be many-to-many
|
19:38:17
|
<mseaton> rather than one-to-many
|
19:40:01
|
*** Collins has quit IRC
|
19:40:18
|
<djazayeri> mseaton: seems like a lot of workâdo you see it as worthwhile?
|
19:40:24
|
<djazayeri> I agree that it would be better that way
|
19:42:17
|
<mseaton> djazayeri: you think it would be better to have a single DefinitionTag that represented "Favorite", which many definitions could link to? rather than just essentially a String on a Definition that could be used to categorize it?
|
19:43:08
|
<djazayeri> mseaton: In some data modeling sense, I think it would be better to have an object that represents tags, and definitions just point to those
|
19:43:22
|
<djazayeri> butâ¦if I were implement things, I probably wouldn't bother
|
19:43:52
|
<djazayeri> though it's hard to do localization without that
|
19:43:58
|
<djazayeri> hard/impossible
|
19:44:33
|
<mseaton> djazayeri: right.
|
19:45:09
|
<mseaton> djazayeri: localization aside for now. what about figuring out the published/non-published and quick-link things?
|
19:45:23
|
<mseaton> are you convinced that these should be not gp-configurable?
|
19:45:31
|
*** james_regen has quit IRC
|
19:47:36
|
<mseaton> djazayeri: or i guess we could make the tags have these properties directly? or something similar.
|
19:47:51
|
<djazayeri> mseaton: my take is that we have a single hardcoded tag for "published"
|
19:48:23
|
<djazayeri> like the way that gmail's "inbox" works (at least to the end-user)
|
19:49:02
|
<djazayeri> I prefer the idea of publishing individual reports, rather than saying that _other_ tags are published
|
19:49:21
|
<djazayeri> but I'm not _convinced_, no
|
19:49:59
|
<mseaton> djazayeri: do you think tags are right for this, or should we just punt this published issue and solve it by having a proper "published" boolean property on each definition type when we re-do the data model?
|
19:51:22
|
<djazayeri> using tags seems fine to me
|
19:52:04
|
<djazayeri> you mean that each individual class (and thus each instance) would get a published property?
|
19:52:19
|
<mseaton> djazayeri: yes.
|
19:52:49
|
<djazayeri> I guess that's better than tags, if nobody is pressed to do publishing now, before that refactoring
|
19:53:21
|
<mseaton> djazayeri: or alternatively, we could take a convention-driven approach and allow a user to configure a global property that says "hideAllDefinitionsWithNameContainingText"
|
19:53:36
|
<mseaton> so someone could, say, put "DRAFT" in the name of anything they didn't want to show up
|
19:54:30
|
<djazayeri> that seems okay as a quick solution
|
19:54:44
|
<djazayeri> but i'd prefer to add a proper boolean in the long run
|
19:54:53
|
<mseaton> djazayeri: sounds like a plan
|
19:55:50
|
<mseaton> wyclif: have you been following this at all?
|
19:56:32
|
<wyclif> mseaton, no
|
19:56:59
|
<mseaton> wyclif: how far along are you with the ticket to support definition tags?
|
19:57:14
|
<wyclif> mseaton, done
|
19:57:25
|
<mseaton> wyclif: checked in?
|
19:57:32
|
<wyclif> i hadn't yet committed the code though
|
19:57:41
|
<wyclif> mseaton, because i didnt like th UI i had created
|
19:57:54
|
<wyclif> mseaton, but let me check it in in the next 5min
|
19:57:58
|
<mseaton> wyclif: ok
|
19:58:05
|
<wyclif> mseaton, and you guys loot at it
|
19:58:16
|
<wyclif> mseaton, the ticket was really bigger than i thought
|
19:58:35
|
<mseaton> wyclif: ok, well one thing we were just discussing is that we might need tags to stand alone, and be linked to definitions via a many-to-many mapping table
|
19:58:42
|
<mseaton> :)
|
19:58:56
|
<wyclif> mseaton, i added a small UI to the report editor page just for the fast pass, it is open for change or be implemented as a portlet
|
19:59:14
|
<wyclif> ok
|
19:59:32
|
<mseaton> but check in what you've got and we can take it from there. can you create a post-commit review too please?
|
20:01:30
|
*** dkayiwa has joined #openmrs
|
20:01:54
|
<wyclif> mseaton, sure
|
20:12:43
|
*** lh has joined #openmrs
|
20:12:43
|
*** lh has joined #openmrs
|
20:23:29
|
*** dkayiwa has quit IRC
|
20:24:55
|
*** dkayiwa has joined #openmrs
|
20:28:01
|
*** nribeka has quit IRC
|
20:34:13
|
*** mvorobey has quit IRC
|
20:39:56
|
*** deadpool has quit IRC
|
20:40:29
|
*** deadpool has joined #openmrs
|
20:41:59
|
*** r0bby_ has quit IRC
|
20:44:37
|
<MarkG> to people with reporting module experience.... does the reporting module auto-schedule some task to run when it is installed? does this task run frequently?
|
20:44:59
|
*** mseaton has quit IRC
|
20:49:01
|
*** mseaton has joined #openmrs
|
20:49:01
|
*** ChanServ sets mode: +v mseaton
|
20:51:54
|
*** bryq has quit IRC
|
20:54:41
|
<wyclif> mseaton, and djazayeri , code for tagging is committed
|
20:56:58
|
*** deadpool has quit IRC
|
20:57:32
|
*** deadpool has joined #openmrs
|
20:58:34
|
*** lh has quit IRC
|
20:59:24
|
*** sgithens has quit IRC
|
21:14:11
|
*** Felipo has quit IRC
|
21:15:48
|
*** deadpool has quit IRC
|
21:16:35
|
*** deadpool has joined #openmrs
|
21:17:14
|
*** dkayiwa has quit IRC
|
21:21:08
|
*** dkayiwa has joined #openmrs
|
21:21:18
|
*** mseaton has left #openmrs
|
21:47:41
|
*** dkayiwa has quit IRC
|
22:07:57
|
*** MarkG has quit IRC
|
22:12:24
|
*** wyclif has quit IRC
|
22:13:52
|
*** deadpool has quit IRC
|
23:08:28
|
*** sgithens_ has joined #openmrs
|
23:23:56
|
*** mseaton has joined #openmrs
|
23:23:56
|
*** ChanServ sets mode: +v mseaton
|