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 > _________________________________________ 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]
