Agreed that usage information for a new concept doesn't make sense; however, if the sidebar contains links to similar concepts and/or resources (like lab websites for definitions), those were specifically meant to be useful during concept creation – e.g., after you enter the new concept's name and click on similar concepts, you find that you have already defined the concept; when adding a new concept for a lab test, you use the link to look up a good definition for it; etc.
-Burke On Fri, May 11, 2012 at 1:12 PM, Darius Jazayeri <djazayeri+...@gmail.com>wrote: > Go ahead and remove it for the new concept case, but leave it for editing > an existing concept. > > (Not having looked at it, I expect this to be ugly in the UI, but I'll > reserve judgment until I've looked.) > > -Darius > > On Fri, May 11, 2012 at 9:36 AM, Andrew Kanter <andy_kan...@yahoo.com>wrote: > >> It seems that it would be relevant to know how the concept was being used >> BEFORE editing it. It makes no sense to have this with New concepts that >> aren't used anywhere and haven't even been submitted.... >> >> Andy >> >> *-------------------- >> Andrew S. Kanter, MD MPH >> >> * >> Asst. Prof. of Clinical Biomedical Informatics and Clinical Epidemiology >> Columbia University >> Email: andrew.kan...@dbmi.columbia.edu >> Mobile: +1 (646) 469-2421 >> Office: +1 (212) 305-4842 >> Skype: akanter-ippnw >> Yahoo: andy_kanter >> >> ------------------------------ >> *From:* Darius Jazayeri <djazayeri+...@gmail.com> >> *To:* openmrs-deve...@listserv.iupui.edu >> *Sent:* Friday, May 11, 2012 12:23 PM >> *Subject:* Re: [OPENMRS-DEV] Exception occurs when trying to create a >> new concept >> >> I'd have thought that the sidebar with usage information is only for >> viewing the concept, not for editing it... >> >> -Darius >> >> On Fri, May 11, 2012 at 8:39 AM, Steven Githens <sgith...@iupui.edu>wrote: >> >> This may have appeared in trunk-3044 that I added last week. It looks >> like I left off a few Transaction = readonly attributes on some new methods >> in ProgramWorkFlowService, but even after adding them, I'm still seeing >> this. ( and from Marks comment below, it appears it's actually coming from >> an existing ConceptService method. ) >> >> I'm wondering if the sidebar with the concept usage information should >> just be left out in the JSP conditionally if this is a new concept, and >> left in if you're editing an existing concept? >> >> -Steve >> >> >> On 05/11/2012 10:36 AM, Mark Goodrich wrote: >> >> Fyi, it looks like this is the offending part of the code:**** >> ** ** >> at >> org.openmrs.web.controller.ConceptFormController$ConceptFormBackingObject.getNumberOfObsUsingThisConcept(ConceptFormController.java:913) >> **** >> ** ** >> Assumedly, something within this method call is causing a flush, and the >> Concept is transient as this point, because it is new. It looks like the >> page is trying to read some statistical data about a concept, which is >> really only relevant when viewing/editing an existing concept.**** >> ** ** >> Mark**** >> ** ** >> ** ** >> ** ** >> *From:* dev@openmrs.org [mailto:dev@openmrs.org <dev@openmrs.org>] *On >> Behalf Of *Ishara Premadasa >> *Sent:* Friday, May 11, 2012 10:02 AM >> *To:* openmrs-deve...@listserv.iupui.edu >> *Subject:* [OPENMRS-DEV] Exception occurs when trying to create a new >> concept**** >> ** ** >> Hi,**** >> I was trying to create a new concept with the latest trunk today for my >> gsoc project related work and it throws the following exception and page is >> not fully loaded.**** >> An error has occurred! >> The following error happened somewhere on this page: >> javax.el.ELException: org.hibernate.TransientObjectException: object >> references an unsaved transient instance - save the transient instance >> before flushing: org.openmrs.Concept**** >> (The full error stack trace output is in the source of this page.) **** >> I have created a ticket for this in >> https://tickets.openmrs.org/browse/TRUNK-3345 . Can someone please have >> a look and help me?**** >> ** ** >> -- >> Ishara Premadasa, >> Google Summer of Code Intern 2012, >> Dept. of Computer Science and Engineering, >> University of Moratuwa, >> Sri Lanka.**** >> ------------------------------ >> Click here to >> unsubscribe<lists...@listserv.iupui.edu?body=SIGNOFF%20openmrs-devel-l>from >> OpenMRS Developers' mailing list >> **** >> ------------------------------ >> Click here to >> unsubscribe<lists...@listserv.iupui.edu?body=SIGNOFF%20openmrs-devel-l>from >> OpenMRS Developers' mailing list >> >> >> ------------------------------ >> Click here to >> unsubscribe<lists...@listserv.iupui.edu?body=SIGNOFF%20openmrs-devel-l>from >> OpenMRS Developers' mailing list >> >> >> ------------------------------ >> Click here to >> unsubscribe<lists...@listserv.iupui.edu?body=SIGNOFF%20openmrs-devel-l>from >> OpenMRS Developers' mailing list >> >> >> ------------------------------ >> Click here to >> unsubscribe<lists...@listserv.iupui.edu?body=SIGNOFF%20openmrs-devel-l>from >> OpenMRS Developers' mailing list >> > > ------------------------------ > Click here to > unsubscribe<lists...@listserv.iupui.edu?body=SIGNOFF%20openmrs-devel-l>from > OpenMRS Developers' mailing list > _________________________________________ To unsubscribe from OpenMRS Developers' mailing list, send an e-mail to lists...@listserv.iupui.edu with "SIGNOFF openmrs-devel-l" in the body (not the subject) of your e-mail. [mailto:lists...@listserv.iupui.edu?body=SIGNOFF%20openmrs-devel-l]
