IRC Chat : 2009-03-10 - OpenMRS

00:08:44 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [7087]: eclipse tools: update version numbers for the 0.3.2 release - #1325 <http://dev.openmrs.org/changeset/7087> || OpenMRS Changesets: Changeset [7086]: eclipse tools: UI glitch in the config.xml editor's text fields - #1326 <http://dev.openmrs.org/changeset/7086>
00:33:48 *** tharanga has joined #openmrs
00:35:37 *** jmiranda has joined #openmrs
00:35:37 *** ChanServ sets mode: +o jmiranda
00:38:53 *** isurundt has joined #openmrs
00:38:58 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Tickets: Ticket #1325 (task closed): Prepare the DevStudio 0.3.2 release <http://dev.openmrs.org/ticket/1325> || OpenMRS Tickets: Ticket #1326 (defect closed): UI glitch in the config.xml editor's text fields <http://dev.openmrs.org/ticket/1326>
00:53:05 *** tharanga has quit IRC
00:53:10 *** isurundt has quit IRC
01:28:33 *** jops87 has quit IRC
01:28:36 *** jops87 has joined #openmrs
02:00:43 * Keelhaul is playing: Dire Straits - Money For Nothing (1985) (.977 The 80s Channel)
03:06:24 *** Keelhaul has quit IRC
05:41:44 *** jmiranda has quit IRC
09:33:35 *** flyankur has joined #openmrs
09:52:02 *** Ankur_ has joined #openmrs
10:06:15 *** flyankur has quit IRC
10:06:56 *** Ankur__ has joined #openmrs
10:20:25 *** Ankur_ has quit IRC
11:31:48 *** james_regen has joined #openmrs
11:31:48 *** ChanServ sets mode: +v james_regen
12:21:54 *** bwolfe has joined #openmrs
12:21:54 *** ChanServ sets mode: +o bwolfe
12:23:28 *** flyankur has joined #openmrs
12:37:32 *** Ankur__ has quit IRC
12:49:43 *** atomicturtle has joined #openmrs
12:49:54 *** atomicturtle has left #openmrs
13:04:25 *** jmiranda has joined #openmrs
13:04:25 *** ChanServ sets mode: +o jmiranda
14:08:54 *** Echidna_ has joined #openmrs
14:22:14 *** Echidna has quit IRC
14:27:28 *** isurundt has joined #openmrs
14:27:34 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [7088]: openmrs-reports: Redesign of the WHO study report with all/most … <http://dev.openmrs.org/changeset/7088>
14:30:58 *** atomicturtle has joined #openmrs
14:31:20 *** tharanga has joined #openmrs
14:37:53 <tharanga> hi bwolfe
14:38:38 <bwolfe> hey tharanga
14:40:04 <tharanga> i'm a student expecting to participate in gsoc 2009. I would like to contribute to a project of openmrs. specially i'm interested in the LDAP module project.
14:40:55 *** isurundt has left #openmrs
14:41:32 <bwolfe> tharanga: ok, sounds great!
14:41:47 <bwolfe> tharanga: please note that we haven't been accepted yet...and there is a chance we won't be.
14:42:18 <bwolfe> tharanga: however, if you want to assume that we will, now is the time to get started with some smaller projects in openmrs to get to know the code
14:42:19 <bwolfe> :-)
14:42:31 <tharanga> ok. is anyone currently working on this project at the moment?
14:43:18 <bwolfe> not that I know of
14:43:53 <bwolfe> I would suggest starting with a smaller project though
14:44:00 <bwolfe> perhaps one from one of these:
14:44:00 <tharanga> ok.I also have used LDAP based authentication in one of the project i have done
14:44:03 <bwolfe> !trivialtickets
14:44:03 <OpenMRSBot> bwolfe: "trivialtickets" --- http://dev.openmrs.org/report/16
14:44:06 <bwolfe> or
14:44:12 <bwolfe> !introtickets
14:44:12 <OpenMRSBot> bwolfe: "introtickets" --- http://dev.openmrs.org/report/20
14:44:31 *** atomicturtle has left #openmrs
14:44:32 <bwolfe> tharanga: ah, cool
14:44:58 <tharanga> but it is a very simple one. but can understand the concept.
14:49:46 <tharanga> is there a possibility of using an open source library like spring LDAP for this
14:54:30 <bwolfe> tharanga: I'm not sure
14:54:40 *** Keelhaul has joined #openmrs
14:54:40 *** ChanServ sets mode: +v Keelhaul
14:54:49 <bwolfe> you're open to using any open source libraries
14:56:01 <tharanga> ok.
14:56:45 <Keelhaul> hi
14:59:32 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Tickets: Ticket #1323 (defect closed): Edit this Patient gives error when changing the tribe <http://dev.openmrs.org/ticket/1323#comment:3> || OpenMRS Changesets: Changeset [7090]: Fixing large patient edit screen editing of attributes - #1323 <http://dev.openmrs.org/changeset/7090> || OpenMRS Changesets: Changeset [7089]: 1.4.x: Fixing large patient edit screen editing of attributes - #1323 <http://dev.openmrs.org/changeset/7089>
14:59:47 <bwolfe> was that a general hello to everyone Keelhaul ? :-)
15:01:49 <Keelhaul> bwolfe: lol yes, why =o
15:02:09 <bwolfe> couldn't tell if you were just saying that to tharanga
15:02:30 <bwolfe> so in response, I say "hello Keelhaul "
15:02:57 <Keelhaul> heh
15:03:06 <Keelhaul> if i come in here and say "hi", why wouldnt it not be to everyone
15:04:12 <bwolfe> Keelhaul: because it was 1/2 second after tharanga said something?
15:04:15 <tharanga> :)
15:31:35 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [7091]: openmrs-logic:Change the columns from tinyint(4) to tinyint(1) <http://dev.openmrs.org/changeset/7091>
15:51:39 *** tharanga has quit IRC
15:57:20 <Keelhaul> bwolfe: you suggested to use PersonAttribute rather than UserProperty for marking a user as location-restricted
15:57:32 <Keelhaul> but i cant find a way to deliberately alter the values for either from the webapp
16:13:11 <Keelhaul> ah hm not true
16:13:37 <Keelhaul> one a userproperty is in, it can be set in the user form
16:24:19 <Keelhaul> ok i have to back out of my claim about PersonAttribute as well
16:24:28 <Keelhaul> it appears in the user form as well
16:24:29 <Keelhaul> =)
16:47:35 <Keelhaul> bwolfe: about ticked 936, when i "accepted" it, the ownership got transferred to me. will bmckown even get notified of updates to that ticket now?
16:48:22 <bwolfe> yeah, he will get notified because he is the one that created it
16:52:15 <Keelhaul> ok
16:52:46 <Keelhaul> bwolfe: if i add a PersonAttribute of type Boolean, the edit person form wont be able to save the values correctly
16:52:55 <Keelhaul> <input type="radio" name="8" id="8_f" value="f"/> false&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="radio" name="8" id="8_t" value="t"/> true&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="radio" name="8" id="8_u" value="u"/> unknown<!-- Default View for FieldGen module -->
16:53:02 <Keelhaul> it uses "t" and "f" for values
16:53:09 <Keelhaul> which dont pass the validator
16:53:46 <bwolfe> odd
16:54:22 <Keelhaul> i just changed it to Strong for now
16:54:25 <Keelhaul> no errors
17:02:11 *** tharanga has joined #openmrs
17:06:54 *** jmiranda has quit IRC
17:22:57 *** jmiranda has joined #openmrs
17:22:57 *** ChanServ sets mode: +o jmiranda
17:27:28 *** tharanga has quit IRC
17:37:45 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [7092]: openmrs-reports: Made some final changes to the indicators for this … <http://dev.openmrs.org/changeset/7092>
17:44:03 <Keelhaul> bwolfe: i need help =(
17:44:11 <bwolfe> uh oh
17:44:34 <Keelhaul> i have a UserMap object with User and Set<Department>
17:45:28 <Keelhaul> i need hibernate to get me all UserMap records by a Collection<Department>
17:45:49 <Keelhaul> as in, all departments in the criteria collection must be in the member collection
17:51:54 <bwolfe> interesting
17:52:05 <bwolfe> not sure how to do that in sql either
17:53:07 <Keelhaul> i guess i'll have to fetch all and then filter them in java
18:09:47 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Tickets: Ticket #1322 (defect closed): Found Similar People yields excessive results when adding a new person <http://dev.openmrs.org/ticket/1322#comment:3> || OpenMRS Changesets: Changeset [7094]: 1.4.x: Tightened up the similar person search algorithm - #1322 Fixed the … <http://dev.openmrs.org/changeset/7094> || OpenMRS Changesets: Changeset [7093]: Tightened up the similar person search algorithm - #1322 Fixed the total … <http://dev.openmrs.org/changeset/7093>
18:39:16 *** nribeka has quit IRC
18:55:15 <Keelhaul> bwolfe: can you explain a mystery to me =o
18:55:27 <Keelhaul> http://rafb.net/p/A9KgsY58.html
18:55:29 <Keelhaul> this piece of code
18:55:36 <Keelhaul> causes this: http://rafb.net/p/DsHBo671.html
18:55:46 <Keelhaul> they are in no way interconnected
18:55:55 <Keelhaul> cept that they are in the same form
18:56:09 <Keelhaul> if i remove that piece of code, the patientNumber works
18:56:24 <Keelhaul> if i remove the call to patientNumber, the other thing works
18:56:31 <Keelhaul> but when they're in together, i get the exception
19:01:51 <Keelhaul> maybe it's the patient/user problem again
19:02:17 <Keelhaul> because getPatientNumber iterates over encounters and extracts the patient
19:02:35 <Keelhaul> and the code in the controller fetches users
19:02:40 <Keelhaul> one of my patients is also a user
19:19:26 <bwolfe> Keelhaul: I think you're right
19:19:39 <Keelhaul> bwolfe: why cant i comment out jsp code
19:19:46 <Keelhaul> it still executes and throws the error
19:20:08 <bwolfe> "IllegalArgumentException occurred while calling setter of org.openmrs.Encounter.patient" means you're probably sending a user to the setPatient method
19:20:30 <Keelhaul> yea, the bottom lines says incompatible types or smt
19:20:31 <bwolfe> you have to use <%-- asdfsdf --%> (or similar)
19:20:36 <Keelhaul> ah ok
19:21:16 <Keelhaul> thx
19:47:15 *** james_regen has quit IRC
19:48:17 <Keelhaul> bwolfe: how do the privileges get updated in a role? ListPicker returns a string, which is never processed in RoleFormController
19:48:47 <bwolfe> Keelhaul: theres an onBind method there
19:48:48 <bwolfe> or something
19:48:49 <Keelhaul> there's no spring binding for them anywhere either
19:49:04 <bwolfe> property editors
19:49:04 <Keelhaul> bwolfe: nope
19:49:13 <bwolfe> I think there are some prop editors registered
19:49:26 *** Keelhaul has quit IRC
19:49:32 *** Keelhaul has joined #openmrs
19:49:32 *** ChanServ sets mode: +v Keelhaul
19:51:45 <bwolfe> Keelhaul: just stumbled across this: binder.registerCustomEditor(Boolean.class, new CustomBooleanEditor("t", "f", true));
19:52:06 <bwolfe> that needs to be in the PersonFormController for the boolean person attributes to work correctly
19:52:17 <Keelhaul> can you add it?
19:53:57 <bwolfe> nah, I'll torture you
19:54:10 <Keelhaul> lol
19:54:22 <Keelhaul> it's the end user who will be tortured
19:55:21 <bwolfe> can you create a ticket for it ?
19:55:34 <bwolfe> and maybe a patch thats been proven on your local system? ;-)
19:56:12 <Keelhaul> =/
19:56:17 <Keelhaul> why does everything have to be so formal
19:56:30 <Keelhaul> an extra editor wont break a form
19:57:06 <bwolfe> Keelhaul: but its nice to have a record of it :-)
20:08:56 <Keelhaul> bwolfe: digging through dozens of closed tickets when searching for keyword can be tedious too
20:10:01 <bwolfe> which means you should put as many keywords into it as possible then :-)
20:10:08 <bwolfe> or maybe less keywords
20:18:15 <Keelhaul> bwolfe: hmm i take it back, this particular editor does break the entire controller lol
20:19:35 <bwolfe> is it killing the voided checkboxes ?
20:20:10 <Keelhaul> no
20:20:28 <Keelhaul> http://rafb.net/p/KEYhig54.html
20:20:41 <Keelhaul> in the editor binding above (date) it causes an exception for some reason
20:20:42 <Keelhaul> hold on
20:23:38 <r0bby> Keelhaul: those Fully Qualified Class Names piss me for some reason
20:23:40 <r0bby> not sure why
20:23:52 <r0bby> Integer.class will resolve just fine ;)
20:24:43 <Keelhaul> why
20:26:14 <Keelhaul> hmm: the date-related nullpointerexception is gone now for some reason
20:26:28 <Keelhaul> but it still wont accept t/f
20:34:24 <r0bby> Keelhaul: I hate spring :)
20:34:56 <Keelhaul> heh
20:35:06 <Keelhaul> i havent used anything else
20:35:10 <Keelhaul> so i cant compare
20:35:20 * r0bby made a PropertyEditor for his custom class for GroovyForms :)
20:35:42 <r0bby> Rod Johnson is a warlock
20:48:17 <Keelhaul> bwolfe: the editor didnt fix it, is there more i have to do?
20:49:48 <bwolfe> I dont think so
20:49:56 <bwolfe> unless you need a differnet fix
20:49:57 <bwolfe> one sec
20:58:35 <bwolfe> Keelhaul: just committed a fix that Frank found
21:00:22 <Keelhaul> who's frank =o
21:00:43 <bwolfe> Frank Knoll
21:00:48 <bwolfe> he's a fellow german :-)\
21:01:36 <Keelhaul> oh
21:03:05 <Keelhaul> bwolfe: hmm
21:03:28 <Keelhaul> you sure that's enough?
21:03:34 <Keelhaul> i still get the same error
21:03:42 <bwolfe> no, not sure thats enough :-p
21:03:44 <Keelhaul> or does it have to be used in conjunction with the boolean editor
21:03:50 <bwolfe> whats the error ?
21:03:55 <Keelhaul> i reverted that class back to default
21:04:00 <Keelhaul> Invalid value for Location-Restricted: 'f'
21:04:04 <bwolfe> oh, yeah, use the boolean editor too
21:04:08 <Keelhaul> hehe
21:04:30 <Keelhaul> bwolfe: i'd prefer a checkbox for booleans, tbqh
21:04:34 <Keelhaul> instead of 3 radio buttons
21:06:08 * r0bby needs to figure out a way to somehow re-write fieldgen
21:06:20 <r0bby> I don't wanna do JSP anymore for the view
21:07:42 <Keelhaul> bwolfe: nope, still error
21:14:56 <bwolfe> I'm out of ideas
21:15:25 <Keelhaul> damn
21:15:32 <Keelhaul> ok i'll leave it as string for now
21:15:45 <Keelhaul> bwolfe: but i'm still puzzled over the role-priv mapping
21:15:59 <bwolfe> property editors
21:16:08 <Keelhaul> the property editors are basic
21:16:11 <Keelhaul> like all other editors
21:16:28 <bwolfe> binder.registerCustomEditor(Privilege.class, new PrivilegeEditor());
21:18:05 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [7095]: Fixing boolean fieldgen tags Author: Knoll_Frank <http://dev.openmrs.org/changeset/7095>
21:18:50 <Keelhaul> bwolfe: i dont see anything there
21:19:04 <bwolfe> anything where ?
21:19:14 <Keelhaul> what exactly binds the returned string from ListPicker to role.privileges
21:19:45 <bwolfe> the spring bind
21:19:57 <bwolfe> it sets the name of the input to privilege
21:20:21 <bwolfe> so when that comes through, spring matches the name "privilege" with the list of privileges on the role object
21:21:44 <Keelhaul> bwolfe: yes, thats usually done via spring:bind tags
21:21:54 <Keelhaul> there is no tag in this case though
21:21:57 <Keelhaul> at least i cant find one
21:22:37 <Keelhaul> http://rafb.net/p/DCQ1sg39.html
21:23:51 <bwolfe> its just the input name
21:23:57 <bwolfe> spring:bind isn't really necessary
21:24:10 <Keelhaul> oh i see
21:24:22 <Keelhaul> i guess i'll have to parse that string somehow
21:24:29 <Keelhaul> beecause i dont bind that to anything
21:24:33 <Keelhaul> there's no object member
22:18:58 <Keelhaul> bwolfe: if you had to map allowed departments to users, would it make more sense to select a user and map departments to him or select a department and add members?
22:20:33 <bwolfe> both :-)
22:20:51 <bwolfe> but if you were only going to do one, I would say departments and select members
22:21:00 <Keelhaul> damn
22:21:11 <bwolfe> not the answer you wanted? :-p
22:21:14 <Keelhaul> no
22:21:20 <bwolfe> sorry :-/
22:21:26 <bwolfe> I meant to say users and select departments
22:21:31 <bwolfe> hehe
22:21:33 <bwolfe> hows that ?
22:21:36 <Keelhaul> because restricging a user to departments is optional
22:21:40 <Keelhaul> only if you flag a user as s uch
22:43:41 *** atomicturtle has joined #openmrs
22:52:11 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Tickets: Ticket #962 (enhancement closed): Drug Regimen Start and End Dates Not Ordered <http://dev.openmrs.org/ticket/962#comment:8> || OpenMRS Changesets: Changeset [7096]: Fixed Orders to validate start/end dates - #962 Added ValidateUtil Author: … <http://dev.openmrs.org/changeset/7096>