Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added basic importer for mbl_newsevent data #15

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Added basic importer for mbl_newsevent data #15

wants to merge 3 commits into from

Conversation

ursbraem
Copy link
Contributor

I've been struggling with roq_newsevent, and then I realized that I don't really need a "newsevent" extension anyway.

If it's an event, I use the datetime field for the event date, the end time is frequently undefinded or stated in the news' bodytext, and the is_event parameter is often handled by pid.

So you can import the start time into "datetime", and the date of the end time into the "archive" field.

This suits my need, but maybe would be helpful to others too - it reduces complexity if you can just use tx_news and don't need additional plugins.

/***************************************************************
* Copyright notice
*
* (c) 2010 Georg Ringer <[email protected]>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add your own name here

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or change it to the TYPO3 copyright notice

* LICENSE.txt file that was distributed with this source code.
*
* The TYPO3 project - inspiring people to share!
*/
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that one valid too?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is okay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants