00:27:56
|
<Keelhaul> hey bwolfe, do you remember which modules can generate a pdf?
|
00:43:03
|
<OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Tickets: Ticket #1285 (enhancement created): The idea of Form Entry needs to be in core <http://dev.openmrs.org/ticket/1285>
|
01:05:31
|
<bwolfe> r0bby_: nice
|
01:05:37
|
<bwolfe> Keelhaul: clinicalsummary
|
01:06:00
|
<Keelhaul> thx
|
01:08:31
|
<r0bby_> do you have any clue how it works
|
01:08:43
|
<r0bby_> setAsText doesn't expose anything -- the variable is local to the method
|
01:08:58
|
<r0bby_> :-/
|
01:10:32
|
<r0bby_> I dont understand it :(
|
01:12:53
|
<r0bby_> :(
|
01:14:57
|
<bwolfe> r0bby_: I haven't looked at htmlformentry, so I don't know how it works either :-/
|
01:18:25
|
<r0bby_> the property editors
|
01:18:41
|
<r0bby_> it uses the openmrs ConceptEditor, LocationEditor, etc
|
01:18:52
|
<r0bby_> I like the way he went about it
|
01:18:57
|
<r0bby_> it beats what I have
|
01:19:03
|
<r0bby_> hands down since it you know, WORKS!
|
01:19:40
|
<Keelhaul> bwolfe: when are you going to review the updated location patch?
|
01:20:06
|
<bwolfe> Keelhaul: hmm, soon hopefully :-)
|
01:25:18
|
<r0bby_> Keelhaul: ask again in 10 years
|
01:25:22
|
* r0bby_ snickers
|
01:25:32
|
* r0bby_ goes back to writing this proof or disproof
|
01:26:01
|
<Keelhaul> zingue
|
01:27:25
|
<Keelhaul> bwolfe: how about putting i-text into trunk? =P
|
01:27:33
|
<Keelhaul> it's a whole meg =/
|
01:28:24
|
<r0bby_> ...why
|
01:29:05
|
<bwolfe> Keelhaul: isn't it there already ?
|
01:29:13
|
<Keelhaul> i dont think so
|
01:29:19
|
<bwolfe> Keelhaul: I think its in the /nealreporting one
|
01:29:22
|
<bwolfe> or /report
|
01:29:23
|
<bwolfe> or something
|
01:29:29
|
<bwolfe> /lib/report
|
01:30:04
|
<Keelhaul> /neal-reporting
|
01:30:09
|
<Keelhaul> hm
|
01:30:37
|
<Keelhaul> thx
|
01:30:46
|
<Keelhaul> who's neal
|
01:30:57
|
<Keelhaul> and can i rely on the jar to stay there for long
|
01:48:02
|
<bwolfe> Keelhaul: probably not. :-p as soon as the nealreporting hack is pulled out it will probably be too
|
01:48:24
|
<bwolfe> Keelhaul: does it have to be in core? or will it work in putting it in /lib in your module ?
|
01:48:35
|
<Keelhaul> yes it will
|
01:48:41
|
<Keelhaul> it will also expand the jar by 1mb lol
|
01:48:52
|
<Keelhaul> trying to keep the clutter in other people's basement =P
|
01:56:54
|
<Keelhaul> bwolfe: i forgot, what am i not supposed to do when catching exceptions in openmrs?
|
01:56:59
|
<Keelhaul> so that the stack doesnt get lost
|
02:02:16
|
<r0bby_> Keelhaul: keep the clutter in your own god damn basement
|
02:05:54
|
<bwolfe> Keelhaul: not sure what you're referring to
|
02:06:10
|
<bwolfe> Keelhaul: as long as you don't catch it and don't "throw(e)" it on
|
02:06:23
|
<Keelhaul> bwolfe: i remember reading something on a wiki page
|
02:06:37
|
<Keelhaul> i have to catch this exception
|
02:06:43
|
<Keelhaul> can i leave the catch block blank?
|
02:20:56
|
<bwolfe> Keelhaul: that is how you keep it from getting recorded
|
02:21:20
|
<bwolfe> you can put a log.debug("this happened because something went wrong", e); or a log.warn("this...", e)
|
02:21:34
|
<Keelhaul> what if i want to show it on the page
|
02:22:47
|
<bwolfe> Keelhaul: just throw it
|
02:22:49
|
<bwolfe> or don't catch it
|
02:23:03
|
<bwolfe> put a "throws Exception" on the method signature
|
02:23:07
|
<Keelhaul> yes
|
02:23:23
|
<Keelhaul> and if i call that method from another method, eclipse says a catch block is required
|
02:25:02
|
<bwolfe> you have to put "throws ___" in each method
|
02:25:08
|
<bwolfe> err "on" each method
|
02:25:48
|
<Keelhaul> i tried that too
|
02:25:49
|
<Keelhaul> Exception DocumentException is not compatible with throws clause in HttpServlet.doGet(HttpServletRequest, HttpServletResponse)
|
02:26:32
|
<bwolfe> ah, so catch it and do a "throw new ServletException("something bad happened", e);"
|
02:27:03
|
<Keelhaul> catch (DocumentException e) {
|
02:27:04
|
<Keelhaul> throw new ServletException(e);
|
02:27:06
|
<Keelhaul> will that do?
|
02:27:36
|
<bwolfe> Keelhaul: no, because that actually calls the ServletException(String) method and it does an e.toString()
|
02:27:52
|
<bwolfe> Keelhaul: you have to call ServletException("some explanation", e)
|
02:27:58
|
<Keelhaul> hm
|
02:28:18
|
<Keelhaul> ic
|
02:28:20
|
<Keelhaul> thx
|
03:30:42
|
*** bwolfe has quit IRC
|
03:39:29
|
<Keelhaul> gn
|
03:39:35
|
*** Keelhaul has quit IRC
|
07:05:17
|
*** jops87 has quit IRC
|
07:06:10
|
*** jops87 has joined #openmrs
|
11:31:05
|
*** atomicturtle has joined #openmrs
|
11:31:20
|
*** atomicturtle has left #openmrs
|
11:53:36
|
*** nribeka has joined #openmrs
|
12:05:37
|
*** nribeka3 has quit IRC
|
13:04:15
|
*** nribeka1 has joined #openmrs
|
13:08:01
|
*** james_regen has joined #openmrs
|
13:08:01
|
*** ChanServ sets mode: +v james_regen
|
13:15:17
|
*** nribeka has quit IRC
|
13:34:13
|
*** nribeka has joined #openmrs
|
13:49:56
|
*** bwolfe has joined #openmrs
|
13:49:56
|
*** ChanServ sets mode: +o bwolfe
|
13:52:45
|
*** nribeka1 has quit IRC
|
14:04:00
|
*** atomicturtle has joined #openmrs
|
14:04:25
|
*** atomicturtle has left #openmrs
|
14:04:36
|
<OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Forum: Re: Google Summer Of Code 2009 <http://forum.openmrs.org/viewtopic.php?f=11&t=421#p1474>
|
14:58:12
|
*** nribeka has quit IRC
|
16:03:08
|
*** nribeka has joined #openmrs
|
16:12:12
|
<OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Tickets: Ticket #965 (task closed): Mongolian cyrillic characters don't entirely work <http://dev.openmrs.org/ticket/965#comment:14>
|
16:13:19
|
<nribeka> bwolfe, i think there are two test failing with the current trunk
|
16:13:27
|
<nribeka> could you please confirm it?
|
16:13:30
|
<bwolfe> uh oh
|
16:13:45
|
<bwolfe> http://ci.openmrs.org/ will be reporting it soon if there are
|
16:13:54
|
<nribeka> i was about to upload hibernate and sha-512
|
16:13:58
|
<nribeka> hm
|
16:14:36
|
<nribeka> lemme check again :D
|
16:34:35
|
<nribeka> bwolfe, the build version is diff
|
16:34:46
|
<nribeka> in the CI: 6963
|
16:34:53
|
<nribeka> in the trunk: 6965
|
16:36:25
|
<nribeka> nvm bwolfe
|
16:50:53
|
*** Keelhaul has joined #openmrs
|
16:50:53
|
*** ChanServ sets mode: +v Keelhaul
|
17:05:56
|
*** Echidna_ has joined #openmrs
|
17:07:48
|
*** Echidna has quit IRC
|
17:12:01
|
<bwolfe> nribeka: what was the problem with the tests ? did you figure it out ?
|
17:13:03
|
<nribeka> i was re-running it
|
17:13:06
|
<nribeka> org.hibernate.NonUniqueObjectException: a different object with the same identifier value was already associated with the session: [org.openmrs.GlobalProperty#minSearchCharacters]
|
17:14:56
|
<nribeka> getPatients_shouldForceSearchStringToBeGreaterThanMinsearchcharactersGlobalProperty
|
17:16:26
|
<bwolfe> nribeka: so its working now ?
|
17:16:38
|
<nribeka> no :P
|
17:16:49
|
<nribeka> i was checking the ci
|
17:32:39
|
<Keelhaul> bwolfe: turns out clinicalsummary doesnt even use i-text
|
17:32:50
|
<Keelhaul> it uses FOP because it builds out of XSL-FO
|
17:34:02
|
<Keelhaul> arght, it's 2.15megs
|
17:34:05
|
<Keelhaul> damn
|
17:35:59
|
<bwolfe> Keelhaul: right
|
17:36:17
|
<Keelhaul> bwolfe: maybe you can give me a hint
|
17:36:25
|
<Keelhaul> i have an xml doc that i need to get to the servlet somehow
|
17:36:38
|
<Keelhaul> the workflow is: the user clicks on the pdf icon and gets a new window with the pdf inside
|
17:36:51
|
<Keelhaul> i thought i can send the xml via POST
|
17:37:05
|
<Keelhaul> that jsp already contains a POST form
|
17:37:12
|
<Keelhaul> thats for different purposes
|
17:40:01
|
<Keelhaul> i dont wanna serialize again
|
17:40:02
|
<Keelhaul> or use the db
|
17:42:41
|
*** nribeka has quit IRC
|
17:46:23
|
<OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [6967]: Pruning the ticket974-liquibase branch as it has been merged into trunk ⦠<http://dev.openmrs.org/changeset/6967> || OpenMRS Changesets: Changeset [6966]: foodprogram: * upgraded module to openmrs-1.4 * in process of ⦠<http://dev.openmrs.org/changeset/6966>
|
17:55:51
|
<Keelhaul> bwolfe: also, it seems i can only use the person search tags if my command object is defined in hibernate or something, right?
|
17:55:53
|
<Keelhaul> <openmrs_tag:personField formFieldName="personId"
|
17:56:22
|
<bwolfe> Keelhaul: you can add another <form> tag with a different post address in it
|
17:56:56
|
<bwolfe> Keelhaul: no, not right. you should be able to use those tags even if you don't have your object in hibernate
|
17:57:46
|
<Keelhaul> hmm
|
17:57:56
|
<Keelhaul> it doesnt save the person
|
17:57:57
|
<Keelhaul> weird
|
18:02:34
|
<bwolfe> Keelhaul: um, you would have to call savePerson in your controller to have that happen
|
18:03:24
|
<Keelhaul> save as in myobject.person = person i selected
|
18:03:39
|
<Keelhaul> after i send the form, the box is empty again
|
18:05:52
|
<bwolfe> not sure what you mean Keelhaul
|
18:06:02
|
<bwolfe> perhaps go through your entire workflow ?
|
18:06:05
|
<bwolfe> brb
|
18:06:07
|
<Keelhaul> ok
|
18:06:35
|
<Keelhaul> my non-persistent object has a member called "recipient" which is an org.openmrs.Person
|
18:06:48
|
<Keelhaul> and i have a form for that object that has a person select tag
|
18:07:12
|
<Keelhaul> and i need it to be assigned to recipient after i send the form
|
18:07:14
|
<Keelhaul> but it doesnt happen
|
18:07:55
|
*** bwolfe has quit IRC
|
18:09:45
|
*** bwolfe has joined #openmrs
|
18:09:45
|
*** ChanServ sets mode: +o bwolfe
|
18:11:22
|
<Keelhaul> wb
|
18:11:26
|
<Keelhaul> [19:06:13] <+Keelhaul> my non-persistent object has a member called "recipient" which is an org.openmrs.Person
|
18:11:26
|
<Keelhaul> [19:06:26] <+Keelhaul> and i have a form for that object that has a person select tag
|
18:11:26
|
<Keelhaul> [19:06:49] <+Keelhaul> and i need it to be assigned to recipient after i send the form
|
18:11:26
|
<Keelhaul> [19:06:51] <+Keelhaul> but it doesnt happen
|
18:14:03
|
<bwolfe> ok, so you just want keelhaulObject.recipient to equal the person id that they choose ?
|
18:14:13
|
<Keelhaul> yes
|
18:14:29
|
<Keelhaul> spring.bind is set correctly around the tag
|
18:14:32
|
<bwolfe> that should work just fine
|
18:14:34
|
<Keelhaul> err spring:bind
|
18:14:42
|
<bwolfe> whats your tag look like ?
|
18:15:15
|
<Keelhaul> pm
|
18:15:45
|
<bwolfe> set formFieldName to ${status.expression}
|
18:17:43
|
<Keelhaul> ah seems to work now
|
18:17:45
|
<Keelhaul> thanks
|
18:18:06
|
<Keelhaul> why is formFieldName usually explicitly set to a name in other forms
|
18:18:12
|
<Keelhaul> like providerId
|
18:18:15
|
<Keelhaul> or patientId
|
18:19:00
|
<bwolfe> dunno
|
18:19:17
|
<bwolfe> people ${status.expression} really turns out to be providerId or patientId ? :-)
|
18:20:02
|
<Keelhaul> also, i cant bind to input type="checkbox"?
|
18:22:52
|
<bwolfe> yeah, you should be able to
|
18:22:58
|
<bwolfe> you bind to the whole set of them though
|
18:23:17
|
<bwolfe> Keelhaul: or you can use the cool spring 1.5 <spring:form> tags
|
18:23:37
|
<bwolfe> they do all the binding, etc magic with a lot of the form elements
|
18:23:44
|
<Keelhaul> rly
|
18:23:45
|
<Keelhaul> hm
|
18:25:37
|
<Keelhaul> lol you try to conform to xhtml and use checked="value" instead of just checked
|
18:25:48
|
<Keelhaul> and then checked="false" is interpreted as true
|
18:25:53
|
<Keelhaul> how counterintuitive is that
|
18:42:30
|
<Keelhaul> ok i see why my checkboxes update when i check them but not when i uncheck them
|
18:42:39
|
<Keelhaul> there needs to be a second input type="hidden"
|
18:42:48
|
*** nribeka has joined #openmrs
|
18:43:36
|
<bwolfe> Keelhaul: yeah, <input type="hidden" name="_${status.expression}" value="" />
|
18:43:45
|
<bwolfe> spring understands the underscore
|
18:43:51
|
<Keelhaul> is value really needed?
|
18:44:23
|
<Keelhaul> but i cant find example of the new spring 2.0 tags anywhere
|
18:44:26
|
<Keelhaul> +s
|
18:45:40
|
<nribeka> bwolfe, the CI works fine
|
18:45:51
|
<nribeka> why it's failing on me :(
|
18:48:14
|
<bwolfe> nribeka: did you try a clean run of hte test s?
|
18:48:21
|
<OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Tickets: Ticket #1256 (task closed): Fix Dojo Javascript Paging <http://dev.openmrs.org/ticket/1256#comment:3> || OpenMRS Changesets: Changeset [6968]: Removed javascript "smart" paging and instead just display results 20 at a ⦠<http://dev.openmrs.org/changeset/6968>
|
18:48:28
|
<bwolfe> nribeka: are you using the junit-report ? or right clicking on test/api ?
|
18:48:29
|
<nribeka> running it now. clean test
|
18:48:34
|
<nribeka> right click
|
18:55:15
|
<bwolfe> nribeka: yeah, you can use the eclipse junit environment to run tests on everything just as you do for one test
|
18:55:29
|
<bwolfe> just right click on the base package and choose run as --> unit test
|
18:59:47
|
<nribeka> BUILD SUCCESSFUL using the ant junit-test
|
18:59:53
|
<nribeka> but fail on right click
|
19:19:27
|
<Keelhaul> bwolfe: for (PatientIdentifier pi : getEncounter().getPatient().getIdentifiers()) {
|
19:19:34
|
<Keelhaul> why does this give me a lazyinitiationexception
|
19:19:49
|
<Keelhaul> err initialization
|
19:24:22
|
<bwolfe> Keelhaul: because you're not in the session ?
|
19:24:27
|
<bwolfe> are you in a jsp apge ?
|
19:24:31
|
<bwolfe> or a controller ?
|
19:24:39
|
<bwolfe> is @Transactional on your service(s) ?
|
19:24:47
|
<bwolfe> (the interfaces)
|
19:25:21
|
<Keelhaul> no services
|
19:25:33
|
<Keelhaul> it happens within that object we talked about before
|
19:25:39
|
<Keelhaul> it has no db table or services
|
19:25:53
|
<Keelhaul> it has an Encounter as its member
|
19:26:02
|
<Keelhaul> and pulls a lot of information via that
|
19:37:41
|
*** Echidna has joined #openmrs
|
19:41:19
|
*** Echidna_ has quit IRC
|
19:50:36
|
<bwolfe> Keelhaul: as long as that encounter is being used across a submit you should be fine
|
19:50:52
|
<bwolfe> you can't use "sessionForm=true" in the spring application context for that controller
|
19:52:42
|
<Keelhaul> it is a sessionForm
|
19:53:00
|
<Keelhaul> i need to keep the object over a submit
|
19:53:06
|
<Keelhaul> so
|
19:53:32
|
<Keelhaul> the first formBackingObject call has a ?encounterId=n
|
19:54:21
|
<Keelhaul> and formBackingObject constructs the command object and sets object.encounter = es.getEncounter(encounterId);
|
19:54:52
|
<Keelhaul> and that part of the object remains unchanged over a submit
|
20:05:22
|
<Keelhaul> bwolfe: how about this
|
20:05:23
|
<Keelhaul> Patient p = Context.getPatientService().getPatient(getEncounter().getPatientId());
|
20:05:23
|
<Keelhaul> for (PatientIdentifier pi : p.getIdentifiers()) {
|
20:08:35
|
<bwolfe> Keelhaul: yeah, if you fetch the patient object again after the submit you shouldn't have a problem
|
20:08:48
|
<bwolfe> its when you try to keep that same patient object that things break
|
20:08:52
|
<bwolfe> brb
|
20:08:55
|
*** bwolfe has quit IRC
|
20:11:08
|
*** bwolfe has joined #openmrs
|
20:11:08
|
*** ChanServ sets mode: +o bwolfe
|
20:31:07
|
<Keelhaul> bwolfe: is there any way to record allergies besides obs?
|
20:31:26
|
<bwolfe> Keelhaul: not right now
|
20:31:46
|
<bwolfe> Keelhaul: there is a ticket for us to make an "active list" type of data storage, but we haven't gotten to it yet
|
20:31:55
|
<bwolfe> Keelhaul: it would probably involve a new table
|
20:32:07
|
<bwolfe> and would be similar to how the current orders are kept
|
20:32:16
|
<Keelhaul> do orders work?
|
20:32:18
|
<bwolfe> Keelhaul: want to take that one on ? :-D
|
20:32:21
|
<bwolfe> Keelhaul: yeah, they should !
|
20:32:23
|
<Keelhaul> not right now =/
|
20:32:24
|
<nribeka> bwolfe, if i attach a patch to ticket, is it going to send any email? :P
|
20:32:26
|
<bwolfe> the pih guys are using them
|
20:32:30
|
<bwolfe> nribeka: nope
|
20:32:37
|
<bwolfe> nribeka: you have to comment on it to send the email
|
20:32:42
|
*** Keelhaul has quit IRC
|
20:32:48
|
*** Keelhaul has joined #openmrs
|
20:32:48
|
*** ChanServ sets mode: +v Keelhaul
|
20:32:52
|
<bwolfe> nribeka: (Which is quite annoying)
|
20:33:22
|
<nribeka> ooo ic ic. i attach patch to the hibernate
|
20:33:26
|
<nribeka> i will put a comment :)
|
21:24:27
|
*** james_regen has quit IRC
|
21:31:11
|
<basic`> bwolfe: re your email, im about to leave for a meeting, but i think that is a good idea and dev2 wont be hard
|
21:31:27
|
<bwolfe> basic`: sweet
|
21:31:28
|
<basic`> the only issue is that there is no official 0.11 rpm for centos, we have a custom one but it hasn't had much testing
|
21:31:47
|
<bwolfe> basic`: I thought you've upgraded 5-10 or so other people
|
21:31:57
|
<basic`> i'll start working on that this week
|
21:32:06
|
<basic`> bwolfe: yes, but those were on a gentoo VM
|
21:32:16
|
<bwolfe> oh
|
21:32:19
|
<basic`> i *think* our rpm works, i just need to test it out
|
21:32:23
|
<bwolfe> thats unfortunate
|
21:32:24
|
<bwolfe> heh
|
21:32:25
|
<basic`> if it doesnt, i can make it go :)
|
21:32:31
|
<bwolfe> well, all the more reason to have a dev2 then ! :-)
|
21:32:35
|
<basic`> exactly
|
21:33:22
|
<basic`> i've also got a dedicated postgresql server setup, so far it's been pretty quick for trac (faster than sqlite)
|
21:33:55
|
<bwolfe> basic`: the only reason I asked as I did is because there will probably be tickets, comments, etc between when you start the upgrade and when we finally "approve" it, so it won't be as easy as just changing dev2 to dev
|
21:34:07
|
<bwolfe> basic`: nice
|
21:34:21
|
<basic`> anyways, i'll get an email back to you and start working on that, hopefully i can have it ready to play with by friday
|
21:34:44
|
<basic`> and ya, we'll have to take a slight downtime to get things synced up, but that shouldn't take very long
|
21:35:06
|
<basic`> gotta run, ttyl
|
21:35:10
|
<bwolfe> cya
|
21:36:58
|
<Keelhaul> wooo
|
21:37:01
|
<Keelhaul> cardio ftw
|
21:40:39
|
*** nribeka1 has joined #openmrs
|
21:58:11
|
*** nribeka has quit IRC
|
22:13:38
|
*** [mharrison] has quit IRC
|
22:21:32
|
<Keelhaul> bwolfe: what does jmiranda mean by you're going to make the changes from the review?
|
22:22:20
|
<bwolfe> Keelhaul: there was really only one change, so before I commit it to trunk, I'm going to make the simple change.
|
22:22:28
|
<Keelhaul> ok
|
22:22:33
|
<bwolfe> Keelhaul: basically it means: "don't worry about a thing". :-)
|
22:22:36
|
<Keelhaul> and about that many to many mapping
|
22:22:47
|
<bwolfe> still under discussion
|
22:22:50
|
<Keelhaul> you sure it can be done later w/o losing existing hierarchies?
|
22:26:57
|
<bwolfe> Keelhaul: yeah, it would just be moving the parent_location column out into a map type of table
|
22:27:15
|
<Keelhaul> yea i guess
|
22:27:17
|
*** atomicturtle has joined #openmrs
|
22:27:22
|
<Keelhaul> since all have the same types
|
22:27:39
|
<Keelhaul> i'm not sure how to do the transition with my current hierarchy
|
22:27:51
|
<Keelhaul> since each is an own table atm with own running ids
|
22:27:57
|
*** atomicturtle has left #openmrs
|
22:32:26
|
*** nribeka has joined #openmrs
|
22:38:20
|
*** nribeka1 has quit IRC
|
22:42:39
|
*** Echidna_ has joined #openmrs
|
22:47:28
|
*** Echidna has quit IRC
|
23:07:20
|
<bwolfe> Keelhaul: you'll have to convert each different "layer" in the hierarchy to a normal location and they all get new location ids ?
|
23:30:08
|
*** bwolfe has quit IRC
|