Project

General

Profile

Actions

Bug #24117

closed

Rename table be_layouts to backend_layout

Added by Ingo Renner over 13 years ago. Updated about 13 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
-
Target version:
-
Start date:
2010-11-18
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Rename table be_layouts to backend_layout.
Also rename the fields be_layout and be_layout_next in table to not use abbreviations.

(issue imported from #M16458)


Files

16458.diff (22.7 KB) 16458.diff Administrator Admin, 2011-01-20 17:56
Actions #1

Updated by Jo Hasenau over 13 years ago

Any reason why?

We've got pages, fe_users, fe_groups, fe_sessions etc.

Or would you want to rename each of these as well?

Actions #2

Updated by Ingo Renner over 13 years ago

Of course we can not rename the old ones for the obvious backwards compatibility reasons. However, along with database lessons and practices tables should be named in singular form.

You may also think in ways of ORMs where the class name matches the table name, then you would have a class called BeLayouts for example, where it should be BeLayout though.

Also I'm not sure whether it might not be just "layout".

It's not a big deal, I'll take care of it later, I just added the issue as a reminder...

Actions #3

Updated by Jo Hasenau over 13 years ago

"be_layout" would be ok, but "layout" wouldn't fit here, since there already is a "layout" field in the pages table and the naming would confuse people.

Actions #4

Updated by Jo Hasenau over 13 years ago

but maybe then we should use "backend_layout" to get rid of acronyms as well

Actions #5

Updated by Ingo Renner over 13 years ago

sounds good to me

Actions #6

Updated by Steffen Gebert over 13 years ago

So... do it now or never?

Actions #7

Updated by Ingo Renner over 13 years ago

now ;)
I'll prepare something

Actions #8

Updated by Jo Hasenau over 13 years ago

do you have something ready so that I can base upcoming bugfixes on the new naming - or should I just provide a patch with the current names?

Actions #9

Updated by Ingo Renner over 13 years ago

committed to trunk (4.5) in r10200

Actions #10

Updated by Susanne Moog about 13 years ago

  • Target version deleted (4.5.0)
Actions

Also available in: Atom PDF