Please let me know the replacement functions

Open to all! Ask other shopowners for help.
Post Reply
radhavallabh
Senior Contributor
Posts: 466
Joined: Tue Oct 27, 2020 4:09 am
Phoenix Version: 1.1.0.6
Has thanked: 29 times
Been thanked: 3 times

Please let me know the replacement functions

Post by radhavallabh »

Hi dear;
It would be really helpful if I could know the replacement of functions below as I am not able to locate the replacement for the same....
tep_draw_separator function

Thank you in advance;
Warm Regds./
radhavallabh


Join The Code Co-op to get access to your library in the Code Co-op Forum
LeeFoster
Contributor
Posts: 263
Joined: Sun Feb 28, 2021 9:41 pm
Phoenix Version: v1.0.8.20
Has thanked: 1 time
Been thanked: 5 times

Re: Please let me know the replacement functions

Post by LeeFoster »

There's a spread sheet for it on google sheets.

https://docs.google.com/spreadsheets/d/ ... edit#gid=0
radhavallabh
Senior Contributor
Posts: 466
Joined: Tue Oct 27, 2020 4:09 am
Phoenix Version: 1.1.0.6
Has thanked: 29 times
Been thanked: 3 times

Re: Please let me know the replacement functions

Post by radhavallabh »

LeeFoster wrote: Thu Feb 17, 2022 9:02 am There's a spread sheet for it on google sheets.

https://docs.google.com/spreadsheets/d/ ... edit#gid=0
I already checked here dear before writing the post, I could not locate that function in it so asked for help....
ecartz
Core Team
Posts: 3084
Joined: Tue Nov 05, 2019 6:02 pm
Phoenix Version:
Has thanked: 4 times
Been thanked: 208 times

Re: Please let me know the replacement functions

Post by ecartz »

There is no replacement for tep_draw_separator, as we no longer use it. When @burt does his design lectures, he will presumably discuss alternatives.

It's possible that if you approached this from the other side, people might be able to give more useful advice. I.e. if you show code that currently uses tep_draw_separator, someone might be able to suggest an alternative.

For a concrete example of what I mean, you might consider replacing

Code: Select all

<?= tep_draw_separator() ?>
with

Code: Select all

<hr>
radhavallabh
Senior Contributor
Posts: 466
Joined: Tue Oct 27, 2020 4:09 am
Phoenix Version: 1.1.0.6
Has thanked: 29 times
Been thanked: 3 times

Re: Please let me know the replacement functions

Post by radhavallabh »

ecartz wrote: Thu Feb 17, 2022 10:01 am There is no replacement for tep_draw_separator, as we no longer use it. When @burt does his design lectures, he will presumably discuss alternatives.

It's possible that if you approached this from the other side, people might be able to give more useful advice. I.e. if you show code that currently uses tep_draw_separator, someone might be able to suggest an alternative.

For a concrete example of what I mean, you might consider replacing

Code: Select all

<?= tep_draw_separator() ?>
with

Code: Select all

<hr>
Thank you dear;
I will make sure to be more precise on future... :)
I eagerly await the design lectures... Hope they start soon.. :)
Thank you again.
Very warm Regds./
radhavallabh
User avatar
burt
Core Team
Posts: 4551
Joined: Tue Oct 29, 2019 9:37 am
Phoenix Version: v1.1.0.8
Has thanked: 252 times
Been thanked: 412 times

Re: Please let me know the replacement functions

Post by burt »

tep_draw_seperator is not (as far as I recall) used anywhere in the core code. In old osCommerce it was mainly used in the admin area for placing a "black line" in a table. The old osC admin was a tables based design.

https://github.com/CE-PhoenixCart/Phoen ... _separator

As per Matts post above, what we did in the admin area redesign was to not use the function and instead either do away with the "black line" and just have nothing, or place a horizontal rule instead; <hr> which does the same job in a better way.

Addons may still use it, which is why the function still exists but triggers an error. If an addon does use it, it means that addon is (most likely) not updated to match the current admin layout.

HTH
I am not here to build for you.
I am here to build with you. Let's help each other.
ecartz
Core Team
Posts: 3084
Joined: Tue Nov 05, 2019 6:02 pm
Phoenix Version:
Has thanked: 4 times
Been thanked: 208 times

Re: Please let me know the replacement functions

Post by ecartz »

burt wrote: Mon Feb 21, 2022 1:33 pm In old osCommerce it was mainly used in the admin area for placing a "black line" in a table.
Also, at one point, it was used as a spacer, with pixel_trans.gif (want space between things, just put an invisible element there!). Really old add-ons still have that appearing.


Join The Code Co-op to get access to your library in the Code Co-op Forum
Post Reply