1.0.8.21 Questions / Comments / Concerns

[heatherbell]

The breaking of "create_account.php"

create_account.php is not broken, just change the configurations as already described.

[Scratchilito]

Hello,
For me, create_account.php notifies an error on line 49. And I have a 2x6 display rather than a 12 display on my 1.0.8.20 online version.

Here is the old code for line 50 (on prod v20):

Code: Select all

foreach ($modules as $module) {
      $module->display_input($customer_details);
    }

And the new line 49 (in test v21):

Code: Select all

   <div class="<?= $customer_data_group['customer_data_groups_width'] ?>">

Concerning Localization>Customer Data Groups, are both the 6 form entries to be changed to col-sm-12 instead of 12? (I just did it without result).

The only Module/Header tags I found to uninstall was 'Datepicker jQuery', not 'ht Datepicker jQuery'

[heatherbell]

are both the 6 form entries to be changed to col-sm-12 instead of 12?

Already answered in previous post - viewtopic.php?p=15306#p15306
Ensure you have run SQL - https://github.com/CE-PhoenixCart/Updat ... update.sql
It is includes/modules/header_tags/ht_datepicker_jquery.php https://github.com/CE-PhoenixCart/Phoen ... jquery.php

[burt]

You can just change them to whatever, then looking at the create_account page to make sure you are happy with the layout. It should not be difficult.

If you have a 6 - 6 layout, then change to col-sm-6
If you have 8, change to col-sm-8
If you have 12, change to col-sm-12

Get used to it, because there will be a lot more of such changes in the future going from (eg) 9 to (eg) col-sm-9 and so on.

[Scratchilito]

Thanks for help.

are both the 6 form entries to be changed to col-sm-12 instead of 12?

Already answered in previous post - viewtopic.php?p=15306#p15306

Ok for all entries, I had seen the answer and my question was subsequently desired to understand having a doubt because I've never entered a value like this before. Thanks Burt for the confirmation and information.

Ensure you have run SQL - https://github.com/CE-PhoenixCart/Updat ... update.sql

Yes I did it.

It is includes/modules/header_tags/ht_datepicker_jquery.php https://github.com/CE-PhoenixCart/Phoen ... jquery.php

By cross-checking your instructions with the json or readme.md

Code: Select all

    "required_modules_remove": [
	"header_tags/ht_datepicker_jquery.php"
    ],
    "required_modules_install": [],
    "optional_modules_install": [],
    "optional_files_delete": [
    "includes/modules/header_tags/ht_datepicker_jquery.php",
	"includes/languages/english/modules/header_tags/ht_datepicker_jquery.php"
    ],
    "required_files_delete": []

• I have updated mysql (phpmyadmin)

• I've uninstalled the modules/header/datepicker_jquery

• I deleted the ht_datepicker_jquery.php files (modules and languages)

The module/header/datepicker_jquery is well no longer available for installation.

• I've set all the widths of the customers datas groups to col-sm-12

Unfortunately, I still get the error on line 49 of create_account.php which is repeated for each block (personal infos, address, contact, password) and the display changes between Firefox and Google chrome.

[heatherbell]

Unfortunately, I still get the error on line 49 of create_account.php which is repeated for each block (personal infos, address, contact, password) and the display changes between Firefox and Google chrome.

Post the error

[Scratchilito]

No further information other than on :

Firefox:


and

Chrome:

[bonbec]

Hello,
Remove the "> at the end of the line 49 in template/default/includes/pages/create_account.php
I have this too in my 1.0.8.21 full version

Bonjour, un autre français, c'est cool
Retire "> à la fin de la ligne 49 dans template/default/includes/pages/create_account.php
J'ai çà aussi dans ma version 1.0.8.21 complète.

Fred (France)

[ecartz]

Remove the "> at the end of the line 49 in template/default/includes/pages/create_account.php

That is not the correct fix. The correct fix would be to remove "> from the width in the database. The "> at the end of that line should be there.

[burt]

Hello,
Remove the "> at the end of the line 49 in template/default/includes/pages/create_account.php
I have this too in my 1.0.8.21 full version

Bonjour, un autre français, c'est cool
Retire "> à la fin de la ligne 49 dans template/default/includes/pages/create_account.php
J'ai çà aussi dans ma version 1.0.8.21 complète.

https://github.com/CE-PhoenixCart/Phoen ... nt.php#L49

is correct.

No to remove the HTML "> please
The extra "> is coming from somewhere else, not L49.

Ne supprimez pas le code HTML "> de cette ligne 49.
Le code HTML "> supplémentaire vient d'ailleurs.