+1 - please revert. Cheers, Tim -- Tim Ruppert HotWax Media http://www.hotwaxmedia.com
o:801.649.6594 f:801.649.6595 ----- "David E Jones" <[email protected]> wrote: > I'm going to say that this is a case where the cure is worse than the > > disease. The more I think about it the worse it gets... > > A few implications if we start going in this direction as a solution > > to this problem: > > 1. a new rule that no ID values can start with a number, and in > general that all ID values in the database must follow Java (or UEL) > > variable naming rules > 2. all updates of production systems will require serious data > migration > > We really have to fix this in code and not in the data. This probably > > isn't the only place we'll run into this problem, so it's worth > looking at. > > Anyway, my vote is that we revert this change and leave it broken > until a code-based fix is found. That sort of constraint on data is > just not reasonable. > > -David > > > On Feb 2, 2009, at 3:17 PM, [email protected] wrote: > > > Author: jleroux > > Date: Mon Feb 2 22:17:16 2009 > > New Revision: 740127 > > > > URL: http://svn.apache.org/viewvc?rev=740127&view=rev > > Log: > > A temporary(?) solution to (at least) fix the mini poll bug in > > eCommerce (due UEL rules) > > Maybe a more generic solution will be introduced later... > > > > Modified: > > ofbiz/trunk/applications/ecommerce/data/DemoSurvey.xml > > > > Modified: ofbiz/trunk/applications/ecommerce/data/DemoSurvey.xml > > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/applications/ecommerce/data/DemoSurvey.xml?rev=740127&r1=740126&r2=740127&view=diff > > = > > = > > = > > = > > = > > = > > = > > = > > > ====================================================================== > > --- ofbiz/trunk/applications/ecommerce/data/DemoSurvey.xml > (original) > > +++ ofbiz/trunk/applications/ecommerce/data/DemoSurvey.xml Mon Feb > > > 2 22:17:16 2009 > > @@ -37,72 +37,72 @@ > > <SurveyQuestionCategory surveyQuestionCategoryId="1001" > > description="Shopping Questions"/> > > <SurveyQuestionCategory surveyQuestionCategoryId="1002" > > description="Poll Questions"/> > > > > - <SurveyQuestion surveyQuestionId="1000" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="TEXT_LONG" > > description="senderName" question="Sender's Name:"/> > > - <SurveyQuestion surveyQuestionId="1001" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="TEXT_LONG" > > description="recipientName" question="Recipient's Name:"/> > > - <SurveyQuestion surveyQuestionId="1002" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="EMAIL" > > description="recipientEmail" question="Recipient's Email:"/> > > - <SurveyQuestion surveyQuestionId="1003" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="TEXTAREA" > > description="giftMessage" question="Gift Message:"/> > > - <SurveyQuestion surveyQuestionId="1010" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="GIFT_CARD" > > description="giftCardNumber" question="Gift Card Number:"/> > > - <SurveyQuestion surveyQuestionId="1011" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="TEXT_SHORT" > > description="giftCardPin" question="PIN Number:"/> > > - > > - <SurveyQuestion surveyQuestionId="1030" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="OPTION" > > question="How often to you shop on-line?"/> > > - <SurveyQuestion surveyQuestionId="1031" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="BOOLEAN" > > question="Have you shopped here?"/> > > - <SurveyQuestion surveyQuestionId="1031B" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="BOOLEAN" > > question="Have you shopped here in the last six months?"/> > > - <SurveyQuestion surveyQuestionId="1031C" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="BOOLEAN" > > question="Have you shopped here in the last year?"/> > > - <SurveyQuestion surveyQuestionId="1032" > > surveyQuestionCategoryId="1001" > > surveyQuestionTypeId="SEPERATOR_LINE"/> > > - <SurveyQuestion surveyQuestionId="1033" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="OPTION" > > question="How would you rate this store for products?"/> > > - <SurveyQuestion surveyQuestionId="1034" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="OPTION" > > question="How would you rate this store for on-line > functionality?"/> > > - <SurveyQuestion surveyQuestionId="1035" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="URL" > > question="What is your favorite on-line store? (Enter Valid URL)"/> > > - <SurveyQuestion surveyQuestionId="1036" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="TEXTAREA" > > question="Comments:"/> > > - <SurveyQuestion surveyQuestionId="1037" > > surveyQuestionCategoryId="1001" > > surveyQuestionTypeId="SEPERATOR_TEXT" question="<img > src="http://localhost:8080/images/ofbiz_logo.jpg"> > > ;"/> > > - <SurveyQuestion surveyQuestionId="1038" > > surveyQuestionCategoryId="1001" > > surveyQuestionTypeId="SEPERATOR_LINE"/> > > - <SurveyQuestion surveyQuestionId="1042" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="CONTENT" > > question="Do you have a picture you would like to attach?"/> > > + <SurveyQuestion surveyQuestionId="_1000" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="TEXT_LONG" > > description="senderName" question="Sender's Name:"/> > > + <SurveyQuestion surveyQuestionId="_1001" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="TEXT_LONG" > > description="recipientName" question="Recipient's Name:"/> > > + <SurveyQuestion surveyQuestionId="_1002" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="EMAIL" > > description="recipientEmail" question="Recipient's Email:"/> > > + <SurveyQuestion surveyQuestionId="_1003" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="TEXTAREA" > > description="giftMessage" question="Gift Message:"/> > > + <SurveyQuestion surveyQuestionId="_1010" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="GIFT_CARD" > > description="giftCardNumber" question="Gift Card Number:"/> > > + <SurveyQuestion surveyQuestionId="_1011" > > surveyQuestionCategoryId="1000" surveyQuestionTypeId="TEXT_SHORT" > > description="giftCardPin" question="PIN Number:"/> > > + > > + <SurveyQuestion surveyQuestionId="_1030" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="OPTION" > > question="How often to you shop on-line?"/> > > + <SurveyQuestion surveyQuestionId="_1031" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="BOOLEAN" > > question="Have you shopped here?"/> > > + <SurveyQuestion surveyQuestionId="_1031B" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="BOOLEAN" > > question="Have you shopped here in the last six months?"/> > > + <SurveyQuestion surveyQuestionId="_1031C" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="BOOLEAN" > > question="Have you shopped here in the last year?"/> > > + <SurveyQuestion surveyQuestionId="_1032" > > surveyQuestionCategoryId="1001" > > surveyQuestionTypeId="SEPERATOR_LINE"/> > > + <SurveyQuestion surveyQuestionId="_1033" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="OPTION" > > question="How would you rate this store for products?"/> > > + <SurveyQuestion surveyQuestionId="_1034" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="OPTION" > > question="How would you rate this store for on-line > functionality?"/> > > + <SurveyQuestion surveyQuestionId="_1035" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="URL" > > question="What is your favorite on-line store? (Enter Valid URL)"/> > > + <SurveyQuestion surveyQuestionId="_1036" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="TEXTAREA" > > question="Comments:"/> > > + <SurveyQuestion surveyQuestionId="_1037" > > surveyQuestionCategoryId="1001" > > surveyQuestionTypeId="SEPERATOR_TEXT" question="<img > src="http://localhost:8080/images/ofbiz_logo.jpg"> > > ;"/> > > + <SurveyQuestion surveyQuestionId="_1038" > > surveyQuestionCategoryId="1001" > > surveyQuestionTypeId="SEPERATOR_LINE"/> > > + <SurveyQuestion surveyQuestionId="_1042" > > surveyQuestionCategoryId="1001" surveyQuestionTypeId="CONTENT" > > question="Do you have a picture you would like to attach?"/> > > > > - <SurveyQuestion surveyQuestionId="1040" > > surveyQuestionCategoryId="1002" surveyQuestionTypeId="OPTION" > > question="Which side should the mini-poll box show?"/> > > - <SurveyQuestion surveyQuestionId="1041" > > surveyQuestionCategoryId="1002" surveyQuestionTypeId="OPTION" > > question="Which hand do you use your mouse with?"/> > > + <SurveyQuestion surveyQuestionId="_1040" > > surveyQuestionCategoryId="1002" surveyQuestionTypeId="OPTION" > > question="Which side should the mini-poll box show?"/> > > + <SurveyQuestion surveyQuestionId="_1041" > > surveyQuestionCategoryId="1002" surveyQuestionTypeId="OPTION" > > question="Which hand do you use your mouse with?"/> > > > > - <SurveyQuestionAppl surveyId="1000" surveyQuestionId="1000" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="1"/> > > - <SurveyQuestionAppl surveyId="1000" surveyQuestionId="1001" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="2"/> > > - <SurveyQuestionAppl surveyId="1000" surveyQuestionId="1002" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="3"/> > > - <SurveyQuestionAppl surveyId="1000" surveyQuestionId="1003" > > fromDate="2001-01-01 00:00:00.0" requiredField="N" > sequenceNum="4"/> > > - <SurveyQuestionAppl surveyId="1001" surveyQuestionId="1010" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="1"/> > > - <SurveyQuestionAppl surveyId="1001" surveyQuestionId="1011" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="2"/> > > + <SurveyQuestionAppl surveyId="1000" surveyQuestionId="_1000" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="1"/> > > + <SurveyQuestionAppl surveyId="1000" surveyQuestionId="_1001" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="2"/> > > + <SurveyQuestionAppl surveyId="1000" surveyQuestionId="_1002" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="3"/> > > + <SurveyQuestionAppl surveyId="1000" surveyQuestionId="_1003" > > fromDate="2001-01-01 00:00:00.0" requiredField="N" > sequenceNum="4"/> > > + <SurveyQuestionAppl surveyId="1001" surveyQuestionId="_1010" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="1"/> > > + <SurveyQuestionAppl surveyId="1001" surveyQuestionId="_1011" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="2"/> > > > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1037" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="1" requiredField="N"/> > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1030" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="2" requiredField="Y"/> > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1037" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="1" requiredField="N"/> > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1030" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="2" requiredField="Y"/> > > <!-- NOTE: these questions are associated with different columns > > > to demonstrate the ability to have one question per cell MultiResp; > > > for this to work these must be sequenced properly by > > sequenceNum,surveyMultiRespColId; the first one for a row (set of > > questions) will be used for the row title --> > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1031" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="3" surveyMultiRespId="01" surveyMultiRespColId="01" > > requiredField="Y"/> > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1031B" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="3" surveyMultiRespId="01" surveyMultiRespColId="02" > > requiredField="Y"/> > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1031C" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="3" surveyMultiRespId="01" surveyMultiRespColId="03" > > requiredField="Y"/> > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1032" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="4" requiredField="N"/> > > - > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1033" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="02" > > sequenceNum="5" surveyMultiRespId="01" requiredField="N"/> > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1034" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="02" > > sequenceNum="6" surveyMultiRespId="01" requiredField="Y"/> > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1038" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="02" > > sequenceNum="7" requiredField="N"/> > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1042" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="02" > > sequenceNum="8" requiredField="N"/> > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1035" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="02" > > sequenceNum="9" requiredField="N"/> > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1031" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="3" surveyMultiRespId="01" surveyMultiRespColId="01" > > requiredField="Y"/> > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1031B" > > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="3" surveyMultiRespId="01" surveyMultiRespColId="02" > > requiredField="Y"/> > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1031C" > > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="3" surveyMultiRespId="01" surveyMultiRespColId="03" > > requiredField="Y"/> > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1032" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="01" > > sequenceNum="4" requiredField="N"/> > > + > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1033" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="02" > > sequenceNum="5" surveyMultiRespId="01" requiredField="N"/> > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1034" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="02" > > sequenceNum="6" surveyMultiRespId="01" requiredField="Y"/> > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1038" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="02" > > sequenceNum="7" requiredField="N"/> > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1042" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="02" > > sequenceNum="8" requiredField="N"/> > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1035" > > fromDate="2001-01-01 00:00:00.0" surveyPageSeqId="02" > > sequenceNum="9" requiredField="N"/> > > <!-- this one doesn't have a surveyPageSeqId on purpose so that > > > the default will be used --> > > - <SurveyQuestionAppl surveyId="1002" surveyQuestionId="1036" > > fromDate="2001-01-01 00:00:00.0" sequenceNum="10" > requiredField="N"/> > > + <SurveyQuestionAppl surveyId="1002" surveyQuestionId="_1036" > > fromDate="2001-01-01 00:00:00.0" sequenceNum="10" > requiredField="N"/> > > > > - <SurveyQuestionAppl surveyId="1003" surveyQuestionId="1040" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="1"/> > > + <SurveyQuestionAppl surveyId="1003" surveyQuestionId="_1040" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="1"/> > > > > - <SurveyQuestionAppl surveyId="1004" surveyQuestionId="1041" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="1"/> > > + <SurveyQuestionAppl surveyId="1004" surveyQuestionId="_1041" > > fromDate="2001-01-01 00:00:00.0" requiredField="Y" > sequenceNum="1"/> > > > > - <SurveyQuestionOption surveyQuestionId="1030" > > surveyOptionSeqId="00001" description="1-2 times per year" > > sequenceNum="1"/> > > - <SurveyQuestionOption surveyQuestionId="1030" > > surveyOptionSeqId="00002" description="2-5 times per year" > > sequenceNum="2"/> > > - <SurveyQuestionOption surveyQuestionId="1030" > > surveyOptionSeqId="00003" description="6-10 time per year" > > sequenceNum="3"/> > > - <SurveyQuestionOption surveyQuestionId="1030" > > surveyOptionSeqId="00004" description="more then 10 times per year" > > > sequenceNum="4"/> > > - <SurveyQuestionOption surveyQuestionId="1033" > > surveyOptionSeqId="00001" description="The Best!" sequenceNum="1"/> > > - <SurveyQuestionOption surveyQuestionId="1033" > > surveyOptionSeqId="00002" description="Above Average" > > sequenceNum="2"/> > > - <SurveyQuestionOption surveyQuestionId="1033" > > surveyOptionSeqId="00003" description="Average" sequenceNum="3"/> > > - <SurveyQuestionOption surveyQuestionId="1033" > > surveyOptionSeqId="00004" description="Below Average" > > sequenceNum="4"/> > > - <SurveyQuestionOption surveyQuestionId="1033" > > surveyOptionSeqId="00005" description="Poor" sequenceNum="5"/> > > - <SurveyQuestionOption surveyQuestionId="1034" > > surveyOptionSeqId="00001" description="Excellent Functionality" > > sequenceNum="1"/> > > - <SurveyQuestionOption surveyQuestionId="1034" > > surveyOptionSeqId="00002" description="Good Functionality" > > sequenceNum="2"/> > > - <SurveyQuestionOption surveyQuestionId="1034" > > surveyOptionSeqId="00003" description="Poor Functionality" > > sequenceNum="3"/> > > - <SurveyQuestionOption surveyQuestionId="1040" > > surveyOptionSeqId="00001" description="Right Side" > sequenceNum="1"/> > > - <SurveyQuestionOption surveyQuestionId="1040" > > surveyOptionSeqId="00002" description="Left Side" sequenceNum="2"/> > > - <SurveyQuestionOption surveyQuestionId="1041" > > surveyOptionSeqId="00001" description="Right Hand" > sequenceNum="1"/> > > - <SurveyQuestionOption surveyQuestionId="1041" > > surveyOptionSeqId="00002" description="Left Hand" sequenceNum="2"/> > > + <SurveyQuestionOption surveyQuestionId="_1030" > > surveyOptionSeqId="00001" description="1-2 times per year" > > sequenceNum="1"/> > > + <SurveyQuestionOption surveyQuestionId="_1030" > > surveyOptionSeqId="00002" description="2-5 times per year" > > sequenceNum="2"/> > > + <SurveyQuestionOption surveyQuestionId="_1030" > > surveyOptionSeqId="00003" description="6-10 time per year" > > sequenceNum="3"/> > > + <SurveyQuestionOption surveyQuestionId="_1030" > > surveyOptionSeqId="00004" description="more then 10 times per year" > > > sequenceNum="4"/> > > + <SurveyQuestionOption surveyQuestionId="_1033" > > surveyOptionSeqId="00001" description="The Best!" sequenceNum="1"/> > > + <SurveyQuestionOption surveyQuestionId="_1033" > > surveyOptionSeqId="00002" description="Above Average" > > sequenceNum="2"/> > > + <SurveyQuestionOption surveyQuestionId="_1033" > > surveyOptionSeqId="00003" description="Average" sequenceNum="3"/> > > + <SurveyQuestionOption surveyQuestionId="_1033" > > surveyOptionSeqId="00004" description="Below Average" > > sequenceNum="4"/> > > + <SurveyQuestionOption surveyQuestionId="_1033" > > surveyOptionSeqId="00005" description="Poor" sequenceNum="5"/> > > + <SurveyQuestionOption surveyQuestionId="_1034" > > surveyOptionSeqId="00001" description="Excellent Functionality" > > sequenceNum="1"/> > > + <SurveyQuestionOption surveyQuestionId="_1034" > > surveyOptionSeqId="00002" description="Good Functionality" > > sequenceNum="2"/> > > + <SurveyQuestionOption surveyQuestionId="_1034" > > surveyOptionSeqId="00003" description="Poor Functionality" > > sequenceNum="3"/> > > + <SurveyQuestionOption surveyQuestionId="_1040" > > surveyOptionSeqId="00001" description="Right Side" > sequenceNum="1"/> > > + <SurveyQuestionOption surveyQuestionId="_1040" > > surveyOptionSeqId="00002" description="Left Side" sequenceNum="2"/> > > + <SurveyQuestionOption surveyQuestionId="_1041" > > surveyOptionSeqId="00001" description="Right Hand" > sequenceNum="1"/> > > + <SurveyQuestionOption surveyQuestionId="_1041" > > surveyOptionSeqId="00002" description="Left Hand" sequenceNum="2"/> > > > > <ProductStoreSurveyAppl productStoreSurveyId="1000" > > productStoreId="9000" surveyApplTypeId="CART_ADD" surveyId="1000" > > productCategoryId="GC-101" fromDate="2003-11-24 16:00:02.467" > > surveyTemplate="/applications/content/template/survey/ > > genericsurvey.ftl"/> > > <ProductStoreSurveyAppl productStoreSurveyId="1001" > > productStoreId="9000" surveyApplTypeId="CART_ADD" surveyId="1001" > > productCategoryId="GC-102" fromDate="2003-11-24 16:00:51.693" > > surveyTemplate="/applications/content/template/survey/ > > genericsurvey.ftl"/> > > > >
