Project

General

Profile

Actions

Feature #15371

closed

In FEediting the creation of new records in other pages than the current should be possible

Added by Martin Klaus almost 19 years ago. Updated over 18 years ago.

Status:
Closed
Priority:
Should have
Category:
Communication
Target version:
-
Start date:
2006-01-07
Due date:
% Done:

0%

Estimated time:
PHP Version:
4
Tags:
Complexity:
Sprint Focus:

Description

In FEediting the creation of new records in other pages than the current should be possible.

With this feature you will be able to create new records i.e. in a sysfolder.

Add a new TS config option to select the target folder where the newly created records should be saved.

i.e.: "editpanel.newRecordInPid"

(issue imported from #M2200)


Files

class.tslib_content.php.diff.txt (1.41 KB) class.tslib_content.php.diff.txt Administrator Admin, 2006-01-16 23:32
class.tslib_content.php.diff-u.txt (4.35 KB) class.tslib_content.php.diff-u.txt Administrator Admin, 2006-01-17 01:10
class.tslib_content.php.diff-u (stdWrap).txt (4.94 KB) class.tslib_content.php.diff-u (stdWrap).txt Administrator Admin, 2006-01-17 02:09
class.tslib_content.php.diff-ub (stdWrap).diff (3.35 KB) class.tslib_content.php.diff-ub (stdWrap).diff Administrator Admin, 2006-01-17 10:32
bug_2200.diff (3.28 KB) bug_2200.diff Administrator Admin, 2006-02-05 22:57
Actions #1

Updated by Martin Klaus almost 19 years ago

"editpanel.newRecordInPid" is correct.
The new stdWrap option has to be documented in TSref of course

Actions #2

Updated by Sebastian Kurfuerst almost 19 years ago

Hi,
what do you need that for? To create for example news records in a sysfolder?
What happens when setting the general record storage page? I assume this doesn't change anything, but it is worth a try.
Greets, Sebastian

Actions #3

Updated by Martin Klaus almost 19 years ago

hi Sebastian,

That's exactly why i need it. But not limited to news records. It would be useful for any type of records wich are stored in other pages than the current (mostly Sysfolders)

The problem is that sysfolders are not accessible through frontend editing. It's only possible to edit already created records in those folders wich are displayed by a plugin content element (EditIcons for the records have to be enabled).

Unfortunatly, it not possible to create records in those pages, if the user would have access to that pages in backend.

I have not found a way to activate the use of the general record storage page. (or any other page than the current page)

regards,
Martin

Actions #4

Updated by Martin Klaus almost 19 years ago

hi Sebastian,

That's exactly why i need it. But not limited to news records. It would be useful for any type of records wich are stored in other pages than the current (mostly Sysfolders)

The problem is that sysfolders are not accessible through frontend editing. It's only possible to edit already created records in those folders wich are displayed by a plugin content element (EditIcons for the records have to be enabled).

Unfortunatly, it not possible to create records in those pages, if the user would have access to that pages in backend.

Even looking at the code I have not seen a way to activate the use of the general record storage page. (or any other page than the current page)

regards,
Martin

Actions #5

Updated by Sebastian Kurfuerst almost 19 years ago

Hi,
is it possible for you to implement this feature and supply a patch?

Greets, Sebastian

Actions #6

Updated by Martin Klaus almost 19 years ago

As promised in chat, here comes the bugfix for 2200 as diff.

Actions #7

Updated by Michael Stucki almost 19 years ago

Hi Martin, thanks so far but could you please create the patch again by giving the "-u" parameter to the diff command? That way we will see the context of your changes. Thanks in advance, Michael.

Actions #8

Updated by Martin Klaus almost 19 years ago

Mi Michael,
Now I've added the patch in unified format. (I've used a windows diff tool, so maybe there's a difference to reports created from unix diff)
regards, Martin

Actions #9

Updated by Michael Stucki almost 19 years ago

Hmm, that still looks wrong. Isn't there any "unified" option?
For example, this is how your patch should look like (it's a plain text file):
http://bugs.typo3.org/view.php?id=1199 (the last file which was attached)

Actions #10

Updated by Martin Klaus almost 19 years ago

Now with stdWrap Properties for "newRecordInPid"

Actions #11

Updated by Martin Klaus almost 19 years ago

Hi Michael,
Thanks for your hint. This time i've used unix diff with command-option -ub against cvs
I hope it works now.
regards

Actions #12

Updated by Martin Klaus almost 19 years ago

Hi Michael,
Was the latest diff -file okay for you now?

Actions #13

Updated by Michael Stucki almost 19 years ago

I removed the stdWrap property again because I don't think it is useful here. The patch is sent for review by the core team and will be committed shortly.

Actions #14

Updated by Michael Stucki almost 19 years ago

Hi Martin,
I'm not sure anymore if this patch is really needed. Made a test using the quickstart side and the mininews listing.

Now let's store the news on a different page and enable the editPanel instead of editIcons (needs to be change in class.tx_mininews_pi1.php).

I can now create new records based on an existing one, and it is stored on the same page like this record.

So what's the problem after all?

Actions #15

Updated by Martin Klaus almost 19 years ago

Hi Michael,
I hope you had a nice week in fiesch :)

Thank's for digging into this issue!
The Problem is, that you need at least one record of this type in the page/sysfolder.

If someone for example deletes the records on the page/sysfolder, then there is no way to create records on this page/sysfolder with frontend-editing again.

This is a very common case for i.e. calender records, or any other records wich get often deleted over time.

From this point of view the patch is definitly needed.

regards,
Martin

Actions #16

Updated by Michael Stucki almost 19 years ago

OK, got it. Committed in CVS.

Actions

Also available in: Atom PDF