Welcome, Guest
Username: Password: Remember me
  • Page:
  • 1

TOPIC: Group Member Registration - city written to state

Group Member Registration - city written to state 16 years 6 months ago #1722

  • JulieL
  • JulieL's Avatar Topic Author
  • Offline
  • Fresh Boarder
  • Fresh Boarder
  • Posts: 10
  • Thank you received: 0
I downloaded DT Register on 6/14/08.

From the Registration Records feature, it looked like a the address was written to the state field for a group registration. I double checked this by looking at the SQL table.

I looked through the code for the main component and in 3-4 places I found....
if($m_s_address)
		{
			$arrMemberFields[]=\'address\';
			$arrMemberValues[]=$arrAddress[$i];
		}
		if($m_s_city)
		{
			$arrMemberFields[]=\'city\';
			$arrMemberValues[]=$arrAddress[$i];
		}

I corrected the last line to read
$arrMemberValues[]=$arrCity[$i];

uploaded and now that problem is fixed.

Question - I have experience with a number of programming languages, but none with PHP. (Until yesterday, my experience was limited to writing about 3 lines!) In the main component, I am seeing many lines of code repeated rather than contained in functions. Is this normal for PHP?

Julie

Please Log in or Create an account to join the conversation.

Group Member Registration - city written to state 16 years 6 months ago #1733

  • JulieL
  • JulieL's Avatar Topic Author
  • Offline
  • Fresh Boarder
  • Fresh Boarder
  • Posts: 10
  • Thank you received: 0
Make the same change in the code in admin.dtregister.php so that inputting group registrations on the admin side will work.

Please Log in or Create an account to join the conversation.

Group Member Registration - city written to state 16 years 6 months ago #1739

  • dthadmin
  • dthadmin's Avatar
  • Offline
  • Administrator
  • Administrator
  • Posts: 5470
  • Thank you received: 3
Thanks Julie. The current code is not ideal and we are aware. This component has been in a constant state of development for about 2 years and has come a long way with TONS more features than it started with. As a result of so many changes, the code is in need or reorganization which is why we are working on a rewrite now to clean things up and make some improvements. Thanks.

Please Log in or Create an account to join the conversation.

  • Page:
  • 1
Time to create page: 0.638 seconds