Post by Cyberdude » Tue Feb 16, 2010 5:57 pm

Hi,

I have a problem.

I had a customer who told me there whas an error when registering at my site when language is set to Dutch.
(only Dutch (nederlands))

When they register, and submit application, they see an error on top of screen.

I have made a printscreen, and it is in attachement.

What i don't understand, is, that, it's only in Dutch language. I have registered a test user in the other languages, and no problems found.

Can someone check?

the error is:

Warning: sprintf() [function.sprintf]: Too few arguments in /home/xxxxxx/public_html/catalog/controller/account/success.php on line 31

what can i do to fix?

And, strange thing is, user is registered, and even recieves welcome mail.

I need this fixed, because Dutch is main language of my site

Attachments

errorsuccess.jpg

error - errorsuccess.jpg (53.32 KiB) Viewed 2006 times

Last edited by i2Paq on Tue Feb 16, 2010 6:42 pm, edited 1 time in total.
Reason: Title fix

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by burrito » Tue Feb 16, 2010 6:03 pm

please post /catalog/controller/account/success.php line 31 so we can have a look?

Image
Opencart specialist | Our website | Our modules


User avatar
Active Member

Posts

Joined
Tue Dec 15, 2009 6:10 pm
Location - Amsterdam, The Netherlands

Post by Cyberdude » Tue Feb 16, 2010 6:06 pm

Code: Select all

<?php 
class ControllerAccountSuccess extends Controller {  
	public function index() {
    	$this->language->load('account/success');
  
    	$this->document->title = $this->language->get('heading_title');

		$this->document->breadcrumbs = array();

      	$this->document->breadcrumbs[] = array(
        	'href'      => $this->url->http('common/home'),
        	'text'      => $this->language->get('text_home'),
        	'separator' => FALSE
      	); 

      	$this->document->breadcrumbs[] = array(
        	'href'      => $this->url->http('account/account'),
        	'text'      => $this->language->get('text_account'),
        	'separator' => $this->language->get('text_separator')
      	);

      	$this->document->breadcrumbs[] = array(
        	'href'      => $this->url->http('account/success'),
        	'text'      => $this->language->get('text_success'),
        	'separator' => $this->language->get('text_separator')
      	);

    	$this->data['heading_title'] = $this->language->get('heading_title');

		if (!$this->config->get('config_customer_approval')) {
    		$this->data['text_message'] = sprintf($this->language->get('text_message'), $this->url->http('information/contact'));
		} else {
			$this->data['text_message'] = sprintf($this->language->get('text_approval'), $this->config->get('config_store'), $this->url->http('information/contact'));
		}
		
    	$this->data['button_continue'] = $this->language->get('button_continue');
		
		if ($this->cart->hasProducts()) {
			$this->data['continue'] = $this->url->http('checkout/cart');
		} else {
			$this->data['continue'] = $this->url->http('account/account');
		}
		
		if (file_exists(DIR_TEMPLATE . $this->config->get('config_template') . '/template/common/success.tpl')) {
			$this->template = $this->config->get('config_template') . '/template/common/success.tpl';
		} else {
			$this->template = 'default/template/common/success.tpl';
		}
		
		$this->children = array(
			'common/header',
			'common/footer',
			'common/column_left',
			'common/column_right'
		);
		
		$this->response->setOutput($this->render(TRUE), $this->config->get('config_compression'));				
  	}
}
?>

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by Cyberdude » Tue Feb 16, 2010 6:08 pm

but, i don't think it is this file, because, other languages also use it i think.

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by burrito » Tue Feb 16, 2010 6:13 pm

see if your languages/dutch/account/succes.php has this line:

Code: Select all

$_['text_message']  = '<p>Uw account is aangemaakt!</p> <p>U kunt nu gebruik maken van uw lidprivileges om het online winkelen nog gemakkelijker te maken.</p> <p>Indien u vragen heeft over deze online winkel, aarzel dan niet en neem <a href="%s%">contact</a> met ons op.</p> <p>Een bevestiging van uw registratie is naar uw e-mailadres gestuurd. Indien u deze niet binnen een uur ontvangt, neemt u dan <a href="%s%">contact</a> met ons op.</p>';

Image
Opencart specialist | Our website | Our modules


User avatar
Active Member

Posts

Joined
Tue Dec 15, 2009 6:10 pm
Location - Amsterdam, The Netherlands

Post by Cyberdude » Tue Feb 16, 2010 6:14 pm

I don't know if this may help, but i also notice that, when account is created,
there is no text displayed on screen

normaly you should see, your account has been created, if any problem please contact us etc.....

then click continue.

but this text is not showed...

don't know if this helps...

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by Cyberdude » Tue Feb 16, 2010 6:15 pm

burrito wrote:see if your languages/dutch/account/succes.php has this line:

Code: Select all

$_['text_message']  = '<p>Uw account is aangemaakt!</p> <p>U kunt nu gebruik maken van uw lidprivileges om het online winkelen nog gemakkelijker te maken.</p> <p>Indien u vragen heeft over deze online winkel, aarzel dan niet en neem <a href="%s%">contact</a> met ons op.</p> <p>Een bevestiging van uw registratie is naar uw e-mailadres gestuurd. Indien u deze niet binnen een uur ontvangt, neemt u dan <a href="%s%">contact</a> met ons op.</p>';
yes but i now copy your text, because, the text i have is different...

hold, i test right now

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by Cyberdude » Tue Feb 16, 2010 6:19 pm

still not working.

same error

this is in dutch success.php :

Code: Select all

<?php
// Heading
$_['heading_title'] = 'Uw account is aangemaakt!';

// Text
$_['text_message']  = '<p>Uw account is aangemaakt!</p> <p>U kunt nu gebruik maken van uw lidprivileges om het online winkelen nog gemakkelijker te maken.</p> <p>Indien u vragen heeft over deze online winkel, aarzel dan niet en neem <a href="%s%">contact</a> met ons op.</p> <p>Een bevestiging van uw registratie is naar uw e-mailadres gestuurd. Indien u deze niet binnen een uur ontvangt, neemt u dan <a href="%s%">contact</a> met ons op.</p>';
$_['text_approval'] = '<p>Bedankt voor uw registratie bij %s!</p><p>U krijg een email zodra uw account is geactiveerd door de winkeleigenaar.</p><p>Als u enige vragen heeft over de werking van deze online shop, neemt u dan <a href="%s%">contact</a> op met de winkeleigenaar.</p>';
$_['text_account']  = 'Account';
$_['text_success']  = 'Succes';
?>

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by burrito » Tue Feb 16, 2010 6:35 pm

I am at a loss... my files are exactly the same and are working fine...

Image
Opencart specialist | Our website | Our modules


User avatar
Active Member

Posts

Joined
Tue Dec 15, 2009 6:10 pm
Location - Amsterdam, The Netherlands

Post by i2Paq » Tue Feb 16, 2010 6:41 pm

You are using the wrong Dutch translation for 1.4.0.

Norman in 't Veldt
Moderator OpenCart Forums

_________________ READ and Search BEFORE POSTING _________________

Our FREE search: Find your answer FAST!.

[How to] BTW + Verzend + betaal setup.


User avatar
Global Moderator

Posts

Joined
Mon Nov 09, 2009 7:00 pm
Location - Winkel - The Netherlands

Post by Cyberdude » Tue Feb 16, 2010 6:45 pm

euh??

i'm using :

opencart-v1.4.0e-Dutch

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by Cyberdude » Tue Feb 16, 2010 6:54 pm

Maybe delete whole Dutch? and recopy?

(problem is, i have mods installed)

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by i2Paq » Tue Feb 16, 2010 7:31 pm

stefke1974 wrote:euh??

i'm using :

opencart-v1.4.0e-Dutch
Die tekst komt, bij mijn weten, niet meer voor in die versie. Hij struikeld namelijk over 2x contact.
Hier.

Norman in 't Veldt
Moderator OpenCart Forums

_________________ READ and Search BEFORE POSTING _________________

Our FREE search: Find your answer FAST!.

[How to] BTW + Verzend + betaal setup.


User avatar
Global Moderator

Posts

Joined
Mon Nov 09, 2009 7:00 pm
Location - Winkel - The Netherlands

Post by Cyberdude » Tue Feb 16, 2010 7:51 pm

maar is versie E de laatste dan?

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by i2Paq » Tue Feb 16, 2010 8:07 pm

stefke1974 wrote:maar is versie E de laatste dan?
Nee, heb je mijn linkje gevolgd?

Norman in 't Veldt
Moderator OpenCart Forums

_________________ READ and Search BEFORE POSTING _________________

Our FREE search: Find your answer FAST!.

[How to] BTW + Verzend + betaal setup.


User avatar
Global Moderator

Posts

Joined
Mon Nov 09, 2009 7:00 pm
Location - Winkel - The Netherlands

Post by Cyberdude » Tue Feb 16, 2010 8:20 pm

bedoel je dit gedeelte te verangen?

Code: Select all

$_['text_message']  = '<p>Uw account is aangemaakt!</p> <p>U kunt nu gebruik maken van de privileges die u als geregistreerd klant heeft om het online winkelen nog gemakkelijker te maken.</p><p>Een bevestiging van uw registratie is naar uw e-mailadres gestuurd.<p>Indien u vragen heeft over deze online winkel, of u ontvangt het naar u verstuurde e-mail niet binnen een uur, neem u dan direct <a href="%s">contact</a> met ons op.</p>';
Lag dus aan die 2x %S ?

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by Cyberdude » Tue Feb 16, 2010 8:25 pm

Hehe,

it is working.

thanks guys.

By the way. Is it possible to force new customer to fill in Postcode?

Now, it is not obligated.

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by i2Paq » Tue Feb 16, 2010 8:28 pm

stefke1974 wrote:bedoel je dit gedeelte te verangen?

Code: Select all

$_['text_message']  = '<p>Uw account is aangemaakt!</p> <p>U kunt nu gebruik maken van de privileges die u als geregistreerd klant heeft om het online winkelen nog gemakkelijker te maken.</p><p>Een bevestiging van uw registratie is naar uw e-mailadres gestuurd.<p>Indien u vragen heeft over deze online winkel, of u ontvangt het naar u verstuurde e-mail niet binnen een uur, neem u dan direct <a href="%s">contact</a> met ons op.</p>';
Lag dus aan die 2x %S ?
Helemaal goed, daar lag het aan ;)

Bij mij gaf hij tijdens registratie een blanco scherm.

The issue is the 2x <a href="%s">contact</a> in that text, after removing and reducing it to 1x it worked well.

This is solved in the latest Dutch language pack.

Norman in 't Veldt
Moderator OpenCart Forums

_________________ READ and Search BEFORE POSTING _________________

Our FREE search: Find your answer FAST!.

[How to] BTW + Verzend + betaal setup.


User avatar
Global Moderator

Posts

Joined
Mon Nov 09, 2009 7:00 pm
Location - Winkel - The Netherlands

Post by Cyberdude » Tue Feb 16, 2010 8:46 pm

Toch raar, want ik dacht dat ik laatste versie had.

1.4.0E

maar, daar staat het nog steeds slecht in.

groeten

Logohunt.Net, Get Amazing custom logo designs for your business in just a few Hours! Start a online logo design contest for only $29 and watch designer logos roll in the same day!


User avatar
Active Member

Posts

Joined
Wed Feb 03, 2010 10:39 pm
Location - Belgium

Post by zrxraver » Tue Feb 16, 2010 9:11 pm

De 2de %s is de ' boosdoener' in de vertaling, in conctroller/account/create.php wordt die regel uit het taalbestand gevuld met $subject = sprintf($this->language->get('mail_subject'), $this->config->get('config_store'));
Hier http://nl2.php.net/manual/en/function.sprintf.php kan je zien dat sprintf dus ook 2 argumenten nodig heeft om alle 2 de %s te vullen.

Active Member

Posts

Joined
Fri Oct 30, 2009 5:36 am
Who is online

Users browsing this forum: No registered users and 10 guests