Ask a Question / Contact Us code duplication

[torvista]

Should these not be page-specific?

zencart/includes/templates/template_default/templates/tpl_ask_a_question_default.php

Line 41 in 832a820

<?php if ($messageStack->size('contact') > 0) echo $messageStack->output('contact'); ?>

zencart/includes/templates/template_default/templates/tpl_contact_us_default.php

Line 45 in 832a820

<?php if ($messageStack->size('contact') > 0) echo $messageStack->output('contact'); ?>

Not relevant/required for Ask a Question?

zencart/includes/templates/template_default/templates/tpl_ask_a_question_default.php

Lines 48 to 57 in 832a820

	<?php
	// show dropdown if set
	if (!empty(CONTACT_US_LIST)){
	?>
	<label class="inputLabel" for="send-to"><?php echo SEND_TO_TEXT; ?></label>
	<?php echo zen_draw_pull_down_menu('send_to', $send_to_array, 0, 'id="send-to"') . '<span class="alert">' . ENTRY_REQUIRED_SYMBOL . '</span>'; ?>
	<br class="clearBoth">
	<?php
	}
	?>

I supposed the contact form could be moved to a module for contact us, ask a question, back in stock...?

[lat9]

1. While those message-stack pages could be page-specific, I see no harm in the fact that the two pages share the same name.
2. I can see a use-case where questions asked about a product might be associated with shipping vs. product behavior, with the question destination different.