IRC Chat : 2012-05-09 - OpenMRS

01:18:38 *** lh has joined #openmrs
01:18:38 *** lh has joined #openmrs
01:37:21 <bwolfe> dave12, sorry, was afk for dinner, etc. :-)
01:37:29 <bwolfe> dave12, which version of openmrs are you using?
02:37:10 <dave12> bwolfe, are you still around?
02:37:44 <dave12> bwolfe, using Version: 1.8.3 Build 24510
02:50:39 *** robbyoconnor has joined #openmrs
02:50:39 *** ChanServ sets mode: +v robbyoconnor
03:00:26 *** djazayeri1 has joined #openmrs
03:02:18 *** djazayeri has quit IRC
03:07:55 *** maro has quit IRC
03:12:53 *** bwolfe has quit IRC
03:24:19 *** upul` has joined #openmrs
03:24:19 *** ChanServ sets mode: +v upul`
04:20:08 *** djazayeri has joined #openmrs
04:20:08 *** ChanServ sets mode: +o djazayeri
04:20:08 *** djazayeri1 has quit IRC
04:49:38 *** dave12 has quit IRC
05:01:04 *** lh has quit IRC
05:03:06 *** harshadura has joined #openmrs
05:41:50 *** harshadura has quit IRC
06:27:00 *** bryq has joined #openmrs
06:27:00 *** ChanServ sets mode: +v bryq
06:34:21 *** bryq has quit IRC
06:38:53 *** dkayiwa has joined #openmrs
06:51:52 *** Alexis_MSF has joined #openmrs
06:59:18 <Alexis_MSF> Hi everyone ! I have installed OpenMRS on my laptop (version 1.9.0). I'm logged as Super User so I have all the privileges. But on the administration screen, I can't see the link "Manage Global Properties" under Maintenance
06:59:34 <Alexis_MSF> It's the same thing about the link "Manage XForm properties"
07:00:20 <Alexis_MSF> And when I go to the Manage Privileges link, I can see them in the list..
07:00:42 <dkayiwa> Alexis_MSF:
07:00:50 <dkayiwa> it was changed to settings
07:02:15 *** sunbiz has joined #openmrs
07:02:15 *** ChanServ sets mode: +v sunbiz
07:02:32 <Alexis_MSF> Ok, I got it.
07:02:55 <Alexis_MSF> Thanks a lot dkayiwa ! :)
07:03:04 <dkayiwa> Alexis_MSF: you are welcome :)
07:15:35 *** robbyoconnor has quit IRC
07:25:36 <Alexis_MSF> Dkayiwa : I have an issue when I want to create a Static Cohort Query --> "/openmrs/module/reporting/cohorts/%3Cfont%20color=" cannot ne found
07:25:51 <Alexis_MSF> is there a solution or a property ot change anywhere ?
07:26:08 <dkayiwa> Alexis_MSF: is it easily reproducible?
07:27:00 <Alexis_MSF> what do you mean by reproducible ?
07:27:35 <dkayiwa> Alexis_MSF: can you give me the steps for reproducing it?
07:28:56 <dkayiwa> Alexis_MSF: let me get back in 15 minutes
07:29:02 *** dkayiwa has quit IRC
08:01:56 *** Alexis_InMEETING has joined #openmrs
08:02:31 *** Alexis_MSF has quit IRC
08:22:35 *** dkayiwa has joined #openmrs
08:26:54 *** dkayiwa has quit IRC
08:27:01 *** dkayiwa has joined #openmrs
08:58:09 *** sunbiz has quit IRC
08:58:55 *** sunbiz has joined #openmrs
08:58:55 *** ChanServ sets mode: +v sunbiz
09:02:04 *** allen0machary has joined #openmrs
09:03:04 <allen0machary> helo
09:36:05 <allen0machary> hello..!
09:44:03 *** dkayiwa has quit IRC
09:47:24 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Modules: XForms 4.0.9.3 uploaded to OpenMRS Module Repository <https://dev.openmrs.org/modules/view.jsp?module=xforms&ampversion=&amp4.0.9.3>
09:51:42 *** Alexis_InMEETING is now known as Alexis_MSF
10:00:00 *** rafa has joined #openmrs
10:00:00 *** ChanServ sets mode: +v rafa
10:08:19 <Alexis_MSF> Hi everyone, I was talking to dkayiwa but he still not here. My question is that I have an issue when I want to create a Static Cohort Query (Administration Screen -> Manage Report Definitions -> Cohort Queries -> "+" icon next to Static Cohort Query) and I have a java NullPointerException who says "The page "/openmrs/module/reporting/cohorts/%3Cfont%20color=" cannot be found. Check the link and try again. ".
10:08:48 <Alexis_MSF> My current Version of OpenMRS is 1.9.0, Reporting Module is 0.7.1.1 and Reporting Compatibility is 1.5.7.
10:18:46 *** dkayiwa has joined #openmrs
10:30:04 <Alexis_MSF> Dkayiwa : Did you see the problem that I was mentionning ?
10:30:17 <dkayiwa> Alexis_MSF: when?
10:30:40 <Alexis_MSF> Dkayiwa : for creating static cohort query
10:30:54 <dkayiwa> Alexis_MSF: can you ask it again?
10:31:12 <Alexis_MSF> Dkayiwa : My question is that I have an issue when I want to create a Static Cohort Query (Administration Screen -> Manage Report Definitions -> Cohort Queries -> "+" icon next to Static Cohort Query) and I have a java NullPointerException who says "The page "/openmrs/module/reporting/cohorts/%3Cfont%20color=" cannot be found. Check the link and try again. ".
10:31:35 <Alexis_MSF> Dkayiwa : My current Version of OpenMRS is 1.9.0, Reporting Module is 0.7.1.1 and Reporting Compatibility is
10:31:50 <dkayiwa> Alexis_MSF: is that the latest version for each module?
10:31:52 <Alexis_MSF> Dkayiwa : 1.5.7
10:32:39 <Alexis_MSF> Dkayiwa : Yes, when I'm checking for upgrades, there's no modules to be proposed
10:32:52 <dkayiwa> Alexis_MSF: let me check
10:33:44 <Alexis_MSF> Dkayiwa : The latest version on modules.openmrs.org are the same
10:35:03 <dkayiwa> Alexis_MSF: i was lucky to reproduce it
10:38:11 <Alexis_MSF> Dkayiwa : You're able to create this query ?
10:38:47 <dkayiwa> Alexis_MSF: was able to reproduce the bug :)
10:39:16 <Alexis_MSF> Dkayiwa : Ahaha so it's a problem with OpenMRS version ?
10:39:53 *** mayank has joined #openmrs
10:40:04 <dkayiwa> Alexis_MSF: it is a bug in the reporting module. feel free to create a ticket for it
10:40:40 <allen0machary> hey everyone...
10:40:51 *** mayank has quit IRC
10:40:57 <dkayiwa> Alexis_MSF: though it is a very simpler bug to fix (takes like 3 mins :)
10:41:07 <dkayiwa> hey Alexis_MSF
10:41:13 <dkayiwa> hey allen0machary
10:41:30 <allen0machary> hey dkayiwa
10:41:53 <Alexis_MSF> Dkayiwa : How did you fix it ?
10:42:07 <allen0machary> am having a problem on mv clean install
10:42:24 <dkayiwa> Alexis_MSF: i did not fix it. just looked at the stack trace and code and seen that it should be easy to fix
10:42:43 <allen0machary> i pasted the log file here : http://pastebin.com/xUQJ4HYe any help will be much appriciated
10:43:40 *** james_regen has joined #openmrs
10:43:40 *** ChanServ sets mode: +v james_regen
10:43:44 <dkayiwa> allen0machary: is that on trunk?
10:43:56 <Alexis_MSF> Dkayiwa : Ok thanks:)
10:44:05 <allen0machary> yes
10:44:20 <dkayiwa> allen0machary: when did you last update it?
10:44:26 <dkayiwa> allen0machary: svn update
10:44:34 <allen0machary> i pull today..
10:45:14 <dkayiwa> allen0machary: you could either fix the failing unit test, or simply put an @Ignore attribute on it :)
10:46:17 <allen0machary> first i had an fail test on openmrs-api which downgrading jdk 1.7 to jdk 1.6 fixed it
10:46:53 <allen0machary> then now openmrs-web which i have no idea to fix it.
10:47:30 <dkayiwa> allen0machary: then you can just put an @Ignore attribute on the failing test :)
10:48:13 <allen0machary> and how do i do that..?
10:49:12 <dkayiwa> allen0machary: look for the method in org.openmrs.web.dwr.DWRConceptServiceTest
10:51:44 <allen0machary> trying now
10:51:45 *** dkayiwa has quit IRC
10:52:10 *** Alexis_MSF has quit IRC
10:52:46 *** dkayiwa has joined #openmrs
10:59:10 *** Alexis_MSF has joined #openmrs
11:00:57 <Alexis_MSF> Dkayiwa : My last question :) I created an All Patients Query. What's the steps to have finnally this query when I click on the Manage Cohorts link ?
11:02:43 <dkayiwa> Alexis_MSF: did you have a chance to read the reporting module wiki documentation? :)
11:03:37 <Alexis_MSF> Dkayiwa : I'm currently on this page yes
11:04:05 <Alexis_MSF> Dkayiwa : Maybe I haven't see the information that interest me
11:04:35 <dkayiwa> Alexis_MSF: you are safer reading all its non technical info
11:04:45 <dkayiwa> including watching the screen casts :)
11:11:56 <Alexis_MSF> Dkayiwa : Okay
11:18:01 <allen0machary> dkayiwa that worked... much thanks
11:18:12 <dkayiwa> allen0machary: great!!!
11:19:50 *** sunbiz1 has joined #openmrs
11:21:08 *** sunbiz has quit IRC
11:23:06 *** upul` has quit IRC
11:23:17 *** dkayiwa has quit IRC
11:27:02 *** sunbiz has joined #openmrs
11:27:02 *** ChanServ sets mode: +v sunbiz
11:29:26 *** bwolfe has joined #openmrs
11:29:26 *** ChanServ sets mode: +o bwolfe
11:30:45 *** sunbiz1 has quit IRC
11:36:13 *** sunbiz1 has joined #openmrs
11:42:08 *** sunbiz has quit IRC
11:42:58 *** allen0machary has quit IRC
11:57:35 *** wyclif has quit IRC
11:59:39 *** bryq has joined #openmrs
11:59:39 *** ChanServ sets mode: +v bryq
12:01:18 *** dkayiwa has joined #openmrs
12:25:57 *** mvorobey has joined #openmrs
12:26:10 <mvorobey> hi all
12:38:50 *** wyclif has joined #openmrs
12:44:33 *** Alexis_MSF has left #openmrs
12:45:25 *** wyclif has joined #openmrs
12:48:46 *** allen7machary has joined #openmrs
13:06:20 <djazayeri> dkayiwa is showing how to use the XForms module at http://connect.iu.edu/omrsu
13:09:52 *** sgithens__ has joined #openmrs
13:16:45 <wyclif> hi bwolfe
13:18:34 <wyclif> bwolfe, am revisiting EVNT-3 and it seems like the listener is what we need to disable, my question is when i subscribe with a listener instance, do i have to pass in the same instance when unsucribing?
13:19:42 <wyclif> bwolfe, probably not, then don't we need some way to uniquely identify listeners?
13:19:42 <bwolfe> you shouldn't have to. it should match on class name, not on equality
13:20:20 <wyclif> bwolfe, listenr class name?
13:20:29 <bwolfe> because if you install a newer version of atomfeed, its listener will be the same classname as the previous atomfeed, but will have a different equality
13:20:52 <bwolfe> class equality is based on classloader+classname. every new module install has a new classloader
13:21:09 <bwolfe> so classes loaded from different installations of modules will not be equal
13:21:59 <wyclif> bwolfe, i guess my question before all this was supposed to be can't one subscribe with the same listener class but different actions
13:22:35 <wyclif> e.g the way we do Scheduler task where they are of the same types but doing different things
13:22:39 <bwolfe> yes
13:24:19 <wyclif> by yes do you mean you can't or can
13:24:50 <bwolfe> yes, a listener can be subscribed multiples times for several actions
13:31:16 <djazayeri> bwolfe: can you find dawn and see if she can find the raw data for daniel
13:32:23 <bwolfe> sure
13:49:13 *** dawn_ has joined #openmrs
13:54:39 <allen7machary> hello
13:54:42 *** ChanServ sets mode: +v dawn_
13:55:16 *** finbrein has joined #openmrs
13:57:57 <allen7machary> can any one help with failed connection to database server on initial setup
13:58:08 *** lh has joined #openmrs
13:58:19 *** lh has joined #openmrs
13:58:35 <bwolfe> allen7machary, whats the error?
13:58:59 <allen7machary> ???Error executing sql: create database if not exists `?` default character set utf8 - Could not create connection to database server. Attempted reconnect 3 times. Giving up.???
13:59:06 <allen7machary> Unable to create the database. The password might be incorrect or the database is not started.
13:59:28 <allen7machary> am using xampp on window 7
14:00:13 <dawn_> !scrumon dawn
14:00:13 * 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:21 <dawn_> Hi Team!
14:00:49 <dawn_> Today's Order: wyclif , rafa, mvorobey , finbrein, dkayiwa , djazayeri , bwolfe
14:00:54 <rafa> hi
14:01:12 <bwolfe> univ call is still finishing up. give daniel and djazayeri a few minutes
14:01:29 <mvorobey> dawn_, hi
14:01:44 <wyclif> Tuesday:
14:01:45 <wyclif> * EVNT-10 - Implement the ProviderEventAdvice class
14:01:45 <wyclif> * Revisited EVNT-3 (Apparently i had a misconception on this which)
14:01:45 <wyclif> Wednesday:
14:01:45 <wyclif> * EVNT-14 - Change Event to singleton
14:01:45 <wyclif> * design call
14:01:47 <wyclif> * More sprint tickets
14:01:49 <wyclif> Blockers: none
14:01:55 <dawn_> bwolfe: k, thanks
14:03:31 <dawn_> rafa: are you there?
14:03:41 <bwolfe> wyclif, rafa might have ideas on EVNT-14 already...
14:03:54 <bwolfe> (check with him before you start on that)
14:03:56 <wyclif> bwolfe, ok, will ask talk to him
14:04:00 <rafa> bwolfe: correct :)
14:04:38 <rafa> Today:
14:04:38 <rafa> * Testing the standalone with changes from TRUNK-3298: concept_reference_term uuids need to remain consistent across implementations during 1.9 upgrade
14:04:38 <rafa> https://tickets.openmrs.org/browse/TRUNK-3298
14:04:38 <rafa> * Watched Daniel's presentation
14:04:38 <rafa> * Wanted to work on EVNT-14
14:04:38 <rafa> * Will finish EVNT-8: Fix new/updated object detection
14:04:39 <rafa> https://tickets.openmrs.org/browse/EVNT-8
14:04:39 <rafa> * Design call
14:04:40 <rafa> No blockers.
14:05:03 <wyclif> rafa, you can work on EVNT-14 and i work on another
14:05:15 <mvorobey> Re-worked https://tickets.openmrs.org/browse/ATOM-5
14:05:16 <rafa> wyclif: ok
14:05:19 <rafa> thanks
14:05:21 <mvorobey> Gonna pick up the next ticket from Sprint dashboard
14:05:22 <mvorobey> Without blockers
14:05:33 *** r_friedman has joined #openmrs
14:05:36 <finbrein> completed ATOM-10
14:05:41 <rafa> wyclif: you're a click ahead of me ;)
14:05:49 <finbrein> Hopefully it doesn't come back ;)
14:06:04 <finbrein> I'll take another ticket
14:06:06 <finbrein> -- end --
14:06:17 <bwolfe> finbrein, the good ones always come back. :-)
14:06:32 <dkayiwa> Committed: Finish GeneralEventAfterAdvice.getSupportedClasses to support modules - EVNT-9
14:06:33 <dkayiwa> Some IRC and mail
14:06:33 <dkayiwa> Prepared for xforms presentation
14:06:33 <dkayiwa> University Call
14:06:33 <finbrein> :-)
14:06:33 <dkayiwa> Now working on: Finish Event.unsetSubscription - EVNT-4
14:06:34 <dkayiwa> No Blockers
14:06:56 <djazayeri> dkayiwa: read your email about the roles and privileges sprint
14:07:02 <bwolfe> dkayiwa, ditto
14:07:16 <bwolfe> (had typed out some of that, but djazayeri was faster than me)
14:07:21 <djazayeri> dkayiwa: you should put EVNT-4 on hold and deal with that first
14:07:39 <dkayiwa> djazayeri: was waiting on dawn_
14:08:09 <dawn_> djazayeri: that was my fault. just sent the link via bwolfe
14:08:28 <dawn_> djazayeri: time for you updates
14:08:29 <djazayeri> dkayiwa: bwolfe send some more info. also, feel free to discuss with us for inspiration
14:08:40 <djazayeri> Monday 8-May-2012
14:08:41 <djazayeri> * Worked on building an Ubuntu VM dev environment (Eclipse + Maven + SVN really are annoying)
14:08:41 <djazayeri> * Wrote a couple blog posts about the ITECH project (will share when the blog is launched)
14:08:41 <djazayeri> * Started the "ozip" discussion
14:08:41 <djazayeri> Tuesday 9-May-2012
14:08:42 <djazayeri> * Started work on the "distribution" module (i.e. ozip)
14:08:42 <dkayiwa> djazayeri: ok
14:08:44 <bwolfe> djazayeri, send/sent
14:09:00 <djazayeri> Today
14:09:05 <dkayiwa> bwolfe: i thought he meant that the two of us should send some more info :)
14:09:09 <djazayeri> * finish first-pass of "distribution" module
14:09:14 <djazayeri> * university call
14:09:15 <djazayeri> * design call
14:09:20 <djazayeri> * project management call
14:09:22 <djazayeri> No blockers
14:09:40 <bwolfe> dkayiwa, you have all the info now
14:09:46 <dkayiwa> bwolfe: ok
14:10:04 <bwolfe> we will discuss your findings on the design call in 4 hours
14:10:11 <dkayiwa> bwolfe: ok
14:10:19 <bwolfe> Tuesday
14:10:19 <bwolfe> emails
14:10:19 <bwolfe> reviewed EVNT-7, header file
14:10:19 <bwolfe> reviewed ATOM-7, getatomfeedstream
14:10:19 <bwolfe> triaged ERR-312, ERR-314
14:10:19 <bwolfe> reviewed EVNT-3, finish unsubscribe method
14:10:21 <bwolfe> reviewed TRUNK-2895, data model image
14:10:23 <bwolfe> reviewed ATOM-10, test for datetorfc3339
14:10:25 <bwolfe> interview for dev this afternoon
14:10:27 <bwolfe> review more tickets
14:10:29 <bwolfe> set up for design call tomorrow
14:10:31 <bwolfe> Wednesday
14:10:33 <bwolfe> reviewed EVNT-9, annotation for supported classes
14:10:35 <bwolfe> reviewed EVNT-10, provider advice
14:10:37 <bwolfe> reviewed TRUNK-3296, concept map type conversion
14:10:39 <bwolfe> univ call (in background)
14:10:41 <bwolfe> design call
14:10:43 <bwolfe> pm call
14:10:45 <bwolfe> more sprint ticket reviews
14:10:49 <bwolfe> ATOM-12, atom feed jsp page
14:10:51 <bwolfe> no blockers
14:11:20 <dawn_> !scrumoff dawn
14:11:20 * 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:11:44 <dawn_> Discussion Points?
14:12:38 <bwolfe> none that I saw
14:13:15 <bwolfe> allen7machary, is mysql started and running? can you log into it with the username/pw you are providing?
14:13:22 <bwolfe> allen7machary, (or are you using hte openmrs standalone?)
14:13:34 <allen7machary> yes it started and running
14:13:35 *** harshadura has joined #openmrs
14:13:57 <allen7machary> and launch openmrs using jetty
14:14:22 <allen7machary> works okey till the initial setups
14:14:32 <allen7machary> but fails to create database
14:17:19 <bwolfe> allen7machary, strange. which version of openmrs?
14:17:45 <allen7machary> i just pulled the source code today
14:17:52 <bwolfe> oh, so trunk?
14:18:09 <bwolfe> allen7machary, and can you connect to mysql with that username and password and create a table?
14:18:31 <bwolfe> mysql -uUSERNAME -pPASSWORD at a command line will test it
14:20:44 <allen7machary> xampp doesn't add msql on normal terminal unless you use it on the provided sql command line
14:21:08 <allen7machary> but got other project in python that works perfectly..
14:25:45 <r_friedman> rafa, got a minute?
14:26:30 <rafa> r_friedman: hi, yes
14:26:56 <r_friedman> rafa, tks, i have a couple of questions about subresources
14:27:42 <rafa> r_friedman: I'll try, but djazayeri knows more about them :)
14:27:47 <r_friedman> when you void a subresource, does that change the resource's collection property at all?
14:28:07 <djazayeri> r_friedman: if you do:
14:28:32 <djazayeri> GET person/uuid
14:28:32 <djazayeri> DELETE person/uuid/address/uuid
14:28:36 <djazayeri> GET person/uuid
14:28:53 <djazayeri> then the addresses field on person will differ in the first and last call
14:29:33 <r_friedman> tks djazayeri , know you're busy, but i'm in the resource writing the delete method
14:30:01 <r_friedman> do i have to do anything to the collection when i void the subresource?
14:30:17 <djazayeri> you're in the subresource, you mean?
14:30:22 *** sunbiz has joined #openmrs
14:30:22 *** ChanServ sets mode: +v sunbiz
14:30:23 <r_friedman> right sry
14:30:44 <djazayeri> r_friedman: presumably the hibernate mappings take care of that automagically
14:31:12 <sunbiz> hi guys... how is the UUID generated when saving an object?? does that object have to be a subclass of BaseOpenmrsData ??
14:31:41 <sunbiz> or does it also work for Metadata ??
14:31:41 *** finbrein has quit IRC
14:31:49 <r_friedman> djazayeri, for the moment I'm talking about delete, i.e. void/retire
14:31:59 <djazayeri> r_friedman: Rather: voiding a subresource marks something voided. So you don't have to do anything to the collection itself. But you do need to make sure that the collection you're exposing on the parent resource doesn't include voided.
14:32:03 <r_friedman> it's actually done by an update
14:32:39 <djazayeri> r_friedman: e.g. in the patient resource we expose Patient.getActiveIdentifiers() rather than Patient.getIdentifiers() as the "identifiers" collection.
14:32:48 <djazayeri> sunbiz: BaseOpenMrsObject has uuids set
14:32:55 <djazayeri> sunbiz: works for metadata too
14:33:00 <r_friedman> djazayeri, so far so good
14:33:08 *** sunbiz1 has quit IRC
14:33:30 <rafa> djazayeri: did you say every newly created concept map type should be NARROWER-THAN by default?
14:33:34 <r_friedman> so now onto subresource.purge
14:34:16 <r_friedman> we hibernate delete the subresource in the service
14:34:21 <djazayeri> rafa: I think that's what Andy has said
14:34:38 <djazayeri> rafa: sorry, I mean that should happen during the upgrade
14:34:42 <r_friedman> do we have to do anything to refresh the collection in the resource
14:35:15 <rafa> djazayeri: okay, I also thought that only during the upgrade
14:35:16 <r_friedman> in one of the subresources, after the subresource is purged, the parent is retrieved and resaved
14:35:24 <djazayeri> r_friedman: no, because the collection is mapped by hibernate, so just deleting it from the db will mean that next time you fetch its parent, that child won't be there.
14:35:35 <djazayeri> rafa: I can try to look for where andy said this
14:35:57 <r_friedman> also, last question
14:36:30 <djazayeri> rafa: similar to TRUNK-3237, which you did
14:36:35 <rafa> djazayeri: it's also in the recent migrating concept mappings to 1.9 thread
14:36:45 <rafa> !ticket TRUNK-3237
14:36:46 <OpenMRSBot> rafa: [#TRUNK-3237] Change the default value for the default concept map type global property to NARROWER-THAN - OpenMRS JIRA - https://tickets.openmrs.org/browse/TRUNK-3237
14:37:30 <djazayeri> rafa: from some email: I would NOT default to SAME-AS this is a narrower and special status... The more common would be IS-A ornarrower-than. Our concepts in general are usually NARROWER than the concept being mapped to...
14:37:50 <rafa> djazayeri: okay thanks
14:37:53 <r_friedman> when implementing the subresource add method, is it necessary to do anything more than delegate.get<Parent>().get<subresource collection>().add(delegate);
14:37:56 *** allen7machary has quit IRC
14:39:56 <djazayeri> r_friedman: look at PersonAddressResource, PersonNameResource, ConceptNameResource
14:40:26 <djazayeri> r_friedman: they all seem to have extra logic in their save methods
14:40:40 <djazayeri> e.g. // make sure that the name has actually been added to the concept
14:40:57 <wyclif> bwolfe, i still don't get what how you want EVNT-10 to get implemented
14:41:12 <r_friedman> djazayeri, ok
14:41:17 <djazayeri> r_friedman: in general doing parent.getCollection().add(newChild) won't work
14:41:33 <djazayeri> because the hibernate mapping on the parent end is inverse=true
14:41:42 <wyclif> bwolfe, the @Handler annotation doesn't work because the advice to provider service is not registered anywhere
14:42:06 <wyclif> bwolfe, for the rest of the other services it registered from the config.xml file
14:42:33 <djazayeri> r_friedman: in general, just at the java API level, you need to both:
14:42:44 <djazayeri> parent.getCollection().add(child)
14:42:48 <djazayeri> child.setParent(parent)
14:42:57 <wyclif> and if we are to do it programatically from the activator, we still need to get access to the general advice object from the application context
14:43:13 <djazayeri> r_friedman: and it's best to put a convenience method for this on your parent: see Person.addName(PersonName)
14:44:20 <r_friedman> djazayeri, but you don't have to do the reverse on remove?
14:44:35 <wyclif> bwolfe ?
14:46:54 <bwolfe> wyclif, if you change the general "listeners" list to include hte event GeneralAdvice, then ALL services will have the advice
14:47:08 <bwolfe> because in the core spring file the listeners list is applied to all servics
14:47:10 <bwolfe> *services
14:47:37 <mvorobey> bwolfe, hi
14:48:04 <bwolfe> wyclif, so IF we can live with the listeners modification, I like that. its cleanest.
14:48:44 <bwolfe> wyclif, your current solution has nothing to remove the advice when event is unloaded. so I worry about both duplicate advice and advice left over after the module is not used
14:48:47 <bwolfe> hi mvorobey
14:49:29 <wyclif> bwolfe, i can do that in the other method to removed it when the module is stopped
14:49:54 <mvorobey> bwolfe, I just ran couple of tests for ATOM-4 and found out that it does not matter wether you overwrite a whole atom feed header file or just change just the "updated" element from it, in both cases it takes o ms
14:50:12 <bwolfe> 0 ms is not fast enough
14:50:27 <bwolfe> we need negative zero ms
14:50:40 <wyclif> bwolfe, have you looked at the config.xm file, it has advice tags, then why are they necessary if the listeners list of getting changed?
14:50:50 <r_friedman> bwolfe, that's why he said o ms, it's smaller than 0 ms
14:51:29 <mvorobey> bwolfe, ok, will google for it, maybe I need to find somethin about being immortal too ? :)
14:51:42 <bwolfe> wyclif, thats the beauty of my suggested solution: those advice tags won't be necessary then
14:52:04 <bwolfe> mvorobey, yes, def find that. "fountain of youth" is the search term you want.
14:52:19 <mvorobey> r_friedman, you are so attentive :D
14:52:23 <wyclif> bwolfe, will look at your suggestion
14:52:32 *** suranga has joined #openmrs
14:52:32 *** ChanServ sets mode: +v suranga
14:52:44 <mvorobey> bwolfe, :)
14:53:12 <bwolfe> mvorobey, ok, so I guess you can cancel the ticket
14:53:47 <mvorobey> bwolfe, it seems that I do not have permissions for that,
14:53:55 <r_friedman> mvorobey, for a moment i thought u said attractive :D
14:54:44 <mvorobey> r_friedman, of course you attreactive :D
14:55:06 <mvorobey> r_friedman, of course you are attractive :D
14:55:11 <r_friedman> mvorobey, no, that's my alter ego, roxanne.feline
14:55:33 <bwolfe> r_friedman, lol for the o vs 0 comment. :-D
14:55:36 <mvorobey> r_friedman, :D
14:56:49 <mvorobey> bwolfe, I only can comment on ticket about results of performance tests saying that ticket can be canceled as well , ok ?
14:58:00 <bwolfe> mvorobey, too late. I canceled it for you
14:58:05 <sunbiz> djazayeri: thanks
14:58:17 <mvorobey> bwolfe, thanks :)
14:59:25 <suranga> bwolfe, hi, I only just saw your email on Jembi;s work on returning HL7 from REST :-)
14:59:25 *** dkayiwa has quit IRC
14:59:44 <bwolfe> suranga, your email tubes must be clogged! :-p
15:00:10 <suranga> bwolfe, turns out ''ve mistakenly signed out of my openmrs.org email :-)
15:00:51 *** dkayiwa has joined #openmrs
15:01:25 <bwolfe> how does one do that?
15:01:49 <mvorobey> bwolfe, may be we might to add atomfeed-omod/omod/activemq-data/localhost* folder to svn:ignore, do not we ?
15:03:43 <bwolfe> it shows up in the omod project for you?
15:03:49 <bwolfe> it is in my api project.
15:04:04 <bwolfe> I added teh ignore in the api project. you can add it for hte omod project as well
15:04:11 *** dave1212 has joined #openmrs
15:04:51 <mvorobey> bwolfe, apparently it appared after I added and launched tests for omod
15:05:01 <mvorobey> appared = appeared
15:05:30 <mvorobey> bwolfe, ok, will add it
15:07:38 *** harshadura has quit IRC
15:10:45 <wyclif> bwolfe, looks to me like a module can't append its own service interceptor to the list of core interceptors for all services from the application context file
15:11:27 <wyclif> bwolfe, so as rafa said, what he did was for testing purposes
15:11:52 <wyclif> bwolfe, may be programatically
15:12:03 <bwolfe> wyclif, it can't append, only overwrite?
15:12:21 <wyclif> bwolfe, is that what you really want?
15:13:00 <rafa> wyclif: the docs say it's possible to append http://static.springsource.org/spring/docs/2.5.x/reference/xsd-config.html#xsd-config-body-schemas-util-list
15:13:01 <wyclif> bwolfe, i think it might be even more risky
15:13:04 <OpenMRSBot> <http://ln-s.net/$$Y8> (at static.springsource.org)
15:13:14 <rafa> wyclif: can you try that?
15:14:16 <rafa> wyclif: see the last sentence in the section
15:14:27 <rafa> wyclif: Finally, you can also control the merging behavior using the 'merge' attribute of the <util:list/> element; collection merging is described in more detail in the section entitled Section 3.3.2.4.1, “Collection merging”.
15:15:31 <bwolfe> if its possible to append to the core list, I want this impl
15:15:50 <bwolfe> if its not possible to append, I don't like overwriting entirely and we should go with your suggestion wyclif
15:16:55 *** dawn_ has quit IRC
15:18:00 *** sunbiz has quit IRC
15:19:54 *** harshadura has joined #openmrs
15:20:28 <dave1212> bwolfe, did you happen to see my messages last night?
15:20:51 <bwolfe> dave1212, I did go back and look at the logs this morning, yes.
15:21:12 <bwolfe> unfortunately I didn't have a chance to test it. the bug I was thinking of was in an earlier version of openmrs
15:23:04 <r_friedman> !ticket trunk-2767
15:23:05 <OpenMRSBot> r_friedman: [#TRUNK-2767] Concept name searches uses the slower ilike - OpenMRS JIRA - https://tickets.openmrs.org/browse/trunk-2767
15:23:30 *** wyclif has quit IRC
15:23:52 *** wyclif has joined #openmrs
15:24:09 *** wyclif has quit IRC
15:24:21 *** wyclif has joined #openmrs
15:24:54 <bwolfe> dave1212, are you able to debug it to see what the object is that its trying to save?
15:25:41 <bwolfe> gotta run, bbl
15:26:16 <dave1212> bwolfe, ok. It appears that the UserResource overrides the setProperty method, and it doesn't appear to deal with generic collections. the super class does handle this though
15:27:44 <dave1212> bwolfe, alright, thanks
15:30:43 <wyclif> rafa, bwolfe if you read the docs on that merging, it is for parent-child beans definitions
15:31:14 *** cpower14 has quit IRC
15:31:19 <wyclif> rafa, bwolfe i.e if you wish to merge a list of properties in a child bean to those in its parent bean
15:32:51 <rafa> wyclif: ok, so it's not possible with xml?
15:33:25 <rafa> wyclif: you could try injecting that list to some bean and adding an element
15:34:18 <rafa> wyclif: use @Resource(name = "serviceInterceptors") to inject
15:34:22 *** cpower has joined #openmrs
15:35:32 <rafa> wyclif: we could call a method that adds an element in the module activator
15:36:15 <wyclif> rafa, i just don't seem to see how this is possible
15:36:47 <wyclif> rafa, it is just intoruding complexities that might introduce obscure bugs
15:38:25 <rafa> wyclif: hmm it looks plain and simple to me ;)
15:39:29 <wyclif> rafa, and activators are not spring managed beans
15:40:33 <rafa> wyclif: yeah activator is not the best place
15:40:38 <rafa> wyclif: but any bean is
15:41:20 <rafa> wyclif: okay, it's just my thought. Do it as you think is right.
15:43:29 *** sgithens__ has quit IRC
15:53:30 <suranga> !snoppy
15:53:31 <OpenMRSBot> suranga: Error: "snoppy" is not a valid command.
15:53:57 *** cpower has quit IRC
16:00:09 *** dkayiwa has quit IRC
16:05:48 *** cpower has joined #openmrs
16:08:44 <djazayeri> wyclif: I missed that conversation about, but are you talking about registering things at module start-time? and you want to know the best way to do autowiring?
16:10:02 *** mvorobey has quit IRC
16:24:52 *** harshadura has quit IRC
16:38:04 <wyclif> djazayeri, yes
16:38:51 *** harshadura has joined #openmrs
16:39:14 <wyclif> djazayeri, we want to register advice to all service methods basically without having to define it manually ifrom the config file
16:52:08 <djazayeri> wyclif: oh, advice, yeah, I don't know about that
17:04:57 *** apurv has joined #openmrs
17:08:25 *** djazayeri has quit IRC
17:14:10 <bwolfe> rafa, which ticket id did you really mean on https://tickets.openmrs.org/browse/EVNT-8 ?
17:14:34 <rafa> ahh sorry 14
17:15:36 <bwolfe> dave1212, so did you find the fix? Is it something in RESTWS that requires something?
17:17:51 *** r_friedman has quit IRC
17:24:48 <wyclif> hi bwolfe
17:24:59 <bwolfe> hey
17:25:02 <wyclif> am working on EVNT-11 and it doesn't seem clear to me
17:25:09 *** cpower has quit IRC
17:25:20 <wyclif> we need to register the listeners right?
17:25:53 <rafa> anyone working on event please update
17:25:59 <wyclif> so how the @handler come in
17:26:13 <rafa> changes I made may be annoying to merge ;)
17:26:14 <wyclif> bwolfe, so how does the @handler come in?
17:27:12 <bwolfe> rafa, might want to say that explicitly to wyclif, dkayiwa, and mvorbey
17:27:33 <wyclif> i have updated, thanks
17:28:06 <rafa> thx bwolfe ;)
17:28:25 <wyclif> bwolfe, ?
17:28:35 <bwolfe> !ticket EVNT-11
17:28:37 <OpenMRSBot> bwolfe: [#EVNT-11] Add annotation to automatically add event listeners - OpenMRS JIRA - https://tickets.openmrs.org/browse/EVNT-11
17:29:13 <bwolfe> wyclif, what don't you get?
17:29:26 <wyclif> bwolfe, sorry i might just be lost about something i haven't yet come across in the code
17:30:13 <wyclif> bwolfe, as how is the @Handler related to SubscribableEventListener
17:30:41 <bwolfe> @Handler is just an annotation that you could use
17:30:59 <bwolfe> because then its easy to call Context.getHandlers(SubscribableEventListener)
17:31:49 <wyclif> bwolfe, got you
17:31:55 <wyclif> thanks
17:40:10 *** cpower has joined #openmrs
17:40:11 <dave1212> bwolfe, I'm not sure I know what the fix is. I've temporarily copied a few lines of code from BaseDelegatingResource
17:40:39 <dave1212> specifically, lines 560-576
17:41:14 <bwolfe> to where?
17:41:46 <dave1212> I replaced the else block in the UserResource, starting at line 228
17:42:31 *** apurv_ has joined #openmrs
17:45:53 *** apurv has quit IRC
17:45:58 *** apurv_ is now known as apurv
17:47:36 *** sgithens__ has joined #openmrs
17:47:52 <bwolfe> dave1212, how recent is your copy of restws? I don't see that many lines in BaseDelegatingResource
17:49:07 <dave1212> bwolfe, I believe I'm looking at trunk: http://goo.gl/k0kyf
17:51:09 *** dkayiwa has joined #openmrs
17:51:55 <bwolfe> dave1212, ok, yeah, very strange. "svn update" in eclipse was showing no new changes for me. but ran it at command line and now pulled down all the new files
17:53:31 <dave1212> alright
17:54:08 *** cpower has quit IRC
17:55:15 <bwolfe> dave1212, can you not just call super.setProperty(instance.getUser(), propertyName, value) ?
17:56:19 <dave1212> yes I tried to do that, but since the class is parameterized with UserAndPassword, it expects UserAndPassword as the first argument type
18:01:14 *** cpower has joined #openmrs
18:04:11 *** djazayeri has joined #openmrs
18:04:11 *** ChanServ sets mode: +o djazayeri
18:06:23 *** wyclif has quit IRC
18:08:43 *** robbyoconnor has joined #openmrs
18:08:43 *** ChanServ sets mode: +v robbyoconnor
18:08:43 *** dkayiwa has quit IRC
18:09:34 *** dkayiwa has joined #openmrs
18:10:02 <djazayeri> dkayiwa: you got dropped from connect?
18:10:22 <dkayiwa> djazayeri: yes my connection
18:10:28 <dkayiwa> djazayeri: am now back on
18:21:36 *** wyclif has joined #openmrs
18:35:55 *** dkayiwa_ has joined #openmrs
18:37:47 *** dkayiwa has quit IRC
18:37:47 *** dkayiwa_ is now known as dkayiwa
18:39:10 *** cpower14 has joined #openmrs
18:39:41 *** cpower has quit IRC
18:43:42 <dave1212> bwolfe, any ideas?
18:44:41 <bwolfe> dave1212, not sure. can you create a unit test in the restws project for this ?
18:47:15 *** r_friedman has joined #openmrs
18:57:08 *** sgithens__ has quit IRC
18:58:01 *** harshadura has quit IRC
18:58:30 <dave1212> bwolfe, I think I can do that. where should I put the patch?
18:59:00 <bwolfe> dave1212, first, create a ticket about this in jira in the RESTWS project. tickets.openmrs.org
19:01:37 <dave1212> got it, thanks
19:03:03 *** dkayiwa has quit IRC
19:05:21 *** cpower14 has quit IRC
19:23:45 <r_friedman> dave1212, what's the restws issue?
19:26:08 <r_friedman> brb, if you've written a ticket by now, just say the number
19:32:31 *** dkayiwa has joined #openmrs
19:34:03 <dave1212> hi r_friedman, I'm in the process of creating an issue. But I'm getting an exception trying to create a user with roles
19:34:56 <r_friedman> dave1212, is role a subresource of user?
19:35:39 <r_friedman> is there a users collection in roles, or a roles collection in user?
19:35:54 <r_friedman> hold on, I'll answer at least the first if you don't know the answer
19:35:57 <dave1212> r_friedman, no it has the @Resource annotation
19:36:31 <dave1212> User has a Set<Role>
19:37:55 <r_friedman> dave1212, then i would say that you have to create the role (if it's not already created) and then create the user with a collection of uuids of roles.
19:38:07 *** bwolfe has quit IRC
19:38:27 <dave1212> r_friedman, I've done this, and I still get the exception
19:38:46 <r_friedman> dave1212, and what's the exception?
19:39:04 <dave1212> java.lang.ClassCastException: java.lang.String cannot be cast to org.openmrs.OpenmrsObject
19:39:39 <r_friedman> dave1212, and it occurs when?
19:41:13 <dave1212> r_friendman, when I invoke the web service? Here is a patch file that modifies the unit test to create a user and demonstrate the exception: http://pastebin.com/Ra05dvQx
19:43:14 <dave1212> r_friedman, can you tell if I'm invoking with the right json structure?
19:44:37 <r_friedman> it doesn't look like you do, the whole test looks sort of weird
19:45:03 <r_friedman> can you pastebin the actual edited test rather than the patch?
19:45:16 <dave1212> r_friedman, really? ok, I just added the roles property to the json string
19:46:10 <dave1212> r_friendman, just the test: http://pastebin.com/k6ZFKq5k
19:48:02 <r_friedman> the reason it had the @Ignore was because it didn't work :)
19:48:42 *** james_regen has quit IRC
19:49:17 <dave1212> The ignore referenced issue RESTWS-242 which has been closed. If you remove the roles property the test will pass
19:52:30 *** cpower has joined #openmrs
19:54:27 *** dawn_ has joined #openmrs
19:54:27 *** ChanServ sets mode: +v dawn_
19:54:35 <dave1212> r_friedman, i guess I assumed that's what it meant, not sure though
19:55:18 <r_friedman> dave1212, i'm trying to verify whether it's right and if not come up with a good one for you
19:56:08 <dave1212> ok, thanks for the help
19:56:43 *** cpower has quit IRC
19:59:46 <djazayeri> dave1212: that snippet looks right to me
19:59:51 *** cpower has joined #openmrs
20:00:25 <djazayeri> dave1212: can you create a JIRA ticket in the RESTWS project for this, and attach the patch? And also include the json snippet itself?
20:00:34 <djazayeri> dave1212: and the stack trace, of course
20:01:12 *** wyclif has quit IRC
20:01:13 <dave1212> djazayeri, ok I think I can do that
20:01:55 *** apurv has quit IRC
20:07:14 *** dkayiwa has quit IRC
20:11:35 <r_friedman> dave1212, the problem is the variable is name, not username
20:11:55 <r_friedman> but it's still failing with a different error
20:15:45 <dave1212> r_friedman, what do you mean variable? I see a field on the User class with the name 'username'?
20:17:20 *** wyclif has joined #openmrs
20:18:56 <r_friedman> dave1212, there's also one called name
20:19:13 <r_friedman> at the point where it was failing, it was failing on name, not username
20:21:40 <r_friedman> dave1212, how sure are you about the correctness of the person uuid?
20:25:49 *** dawn_ has quit IRC
20:26:01 *** dawn_ has joined #openmrs
20:26:01 *** ChanServ sets mode: +v dawn_
20:26:41 <dave1212> r_friedman, that person uuid was already in test, I assume its right. I searched and found it in the standardTestDatabase.xml file
20:27:35 *** dawn_ has quit IRC
20:27:57 <r_friedman> dave1212, good
20:28:17 <dave1212> the test doesn't fail for me if I remove the roles property, does it fail for you?
20:33:55 <r_friedman> dave1212, ok, this is definitely a bug
20:34:17 *** lh has quit IRC
20:34:17 <r_friedman> dave1212, the problem is that UserAndPassword is not an OpenMRS object
20:34:47 <r_friedman> So user is a property of UserAndPassword, not stored directly
20:36:17 <r_friedman> dave1212, here is a pastebin of the test as I have been running it http://pastebin.com/s1J4zpHK
20:37:04 *** lh has joined #openmrs
20:39:32 *** bryq has left #openmrs
20:39:46 <r_friedman> dave1212, you also have the right exception, it happens at line 141 of UserResource
20:39:55 <dave1212> r_friedman alright. I thought UserAndPassword was an OpenMRS object? It extends BaseOpenmrsMetadata
20:40:33 <r_friedman> yes, but it is not the user object that people want to create, it only contains a user object
20:41:03 <r_friedman> so you would have to create the user, then use its uuid to crete a UserAndPassword
20:41:09 <r_friedman> but there's no way to do that
20:42:05 <r_friedman> dave1212, it's a special object designed for authentication
20:44:11 *** dave1212 has quit IRC
20:45:27 *** dave12 has joined #openmrs
20:47:37 <dave12> r_friendman, I think I got disconnected, i'm not sure if I missed anything you might have said
21:15:46 *** wyclif has quit IRC
21:30:45 *** cpower has quit IRC
21:40:57 *** suranga has quit IRC
21:47:37 *** rafa has quit IRC
22:22:55 *** wyclif has joined #openmrs
22:35:01 *** carloshpf has joined #openmrs
23:22:14 *** lh has quit IRC