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

Fixed several errors #11

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

Fixed several errors #11

wants to merge 6 commits into from

Conversation

GeraldST
Copy link

Fixed msg_sectcat error: There is no field "msg_sectcat", not in the table nor in the code. Therefore viewing feed threw error notice: Undefined property: stdClass::$msg_sectcat view.raw.php

Added additional default fields for new items: Creating a new feed in Backend threw four error notices: Undefined property: stdClass::$XXY, variables need to be initialized properly.

Changed itemdate: Formerly LastModifiedDate was taken for an item within feed. This causes much problems (as stated from several users) because the date changes with every article modification and item appears again on top of feed. Therefore now changed to Create Date.

There is no field "msg_sectcat", not in the table nor in the code. Therefore viewing feed throws error notice: Undefined property: stdClass::$msg_sectcat view.raw.php
There is no field "msg_sectcat", not in the table nor in the code. Therefore viewing feed throws error notice: Undefined property: stdClass::$msg_sectcat view.raw.php
Creating a new feed in Backend throws four error notices: Undefined property: stdClass::$XXY, variables need to be initialized properly.
Currently LastModifiedDate is taken as an item within feed. This causes much problems (as stated from several users) because the date changes with every modification and item again appears on top of feed. Therefore changed to Create Date.
Doing some cleanup - since Joomla 2.5 assignRef() is no longer needed.
Doing some cleanup - since Joomla 2.5 assignRef() is no longer needed.
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.

1 participant