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

new class for interacting with posts #153

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

Conversation

TamaroWalter
Copy link
Contributor

@TamaroWalter TamaroWalter commented Jun 19, 2023

Goal of the redesign is a new structure that makes the post.php more readable and that encapsulates functions that work with posts that were previously in the locallib.

The redesign is structured as followed:

  • There are new classes post and discussion, that represent their component and provide functions to read and manipulate them.
  • There is a new class post_control, that administrates the workflow of the post.php. The Controller detects the interaction (create/edit/answer/delete), builds the post_form and executes the interaction while securing that everything is being correctly executed.
    The post.php now only calls the post_control and the controller handles everything.

Copy link
Contributor

@NinaHerrmann NinaHerrmann left a comment

Choose a reason for hiding this comment

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

Intermediate review

classes/post/post.control.php Outdated Show resolved Hide resolved
classes/post/post.control.php Outdated Show resolved Hide resolved
classes/post/post.control.php Outdated Show resolved Hide resolved
classes/post/post.control.php Outdated Show resolved Hide resolved
classes/post/post.control.php Outdated Show resolved Hide resolved
classes/post/post.control.php Outdated Show resolved Hide resolved
classes/post/post.control.php Outdated Show resolved Hide resolved
classes/post/post.php Outdated Show resolved Hide resolved
@TamaroWalter TamaroWalter self-assigned this Apr 26, 2024
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