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

Combine gedcom_news & user_blog modules #43

Closed
wants to merge 1 commit into from
Closed

Combine gedcom_news & user_blog modules #43

wants to merge 1 commit into from

Conversation

ddrury
Copy link
Contributor

@ddrury ddrury commented Nov 27, 2013

Hi Greg,

When I was looking at fixing the CKEditor problem with editnews.php, I noticed a TODO in there about moving
its functions into the module

So I've done it. The gedcom_news & user_journal modules are both now incorporated in the blog module.

The add/edit form is now a modal dialog and whereas previously the user_journal module had a hard
coded option for deciding what happened if the contents overflowed the window (scroll bar), now both news &
journal have configurable options.

The downside is I've changed the configuration settings that are saved [class constants (integers) instead of
text strings], so if this module is adopted as a replacement either the user will have to redo his preferences
or a migration will be needed. New translations will also be needed.

I also incorporated the csrf stuff for saving or deleting articles and this raised a problem for the delete
function, previously the anchor had an onclick function that popped up a confirm dialog and if accepted just
called the delete routine. To incorporated the csrf stuff I had to create a modal dialog with a form (ugly!)
so that I could post the csrf token. Maybe a need here for a generic confirmation dialog?

Another thing I noticed was the addNews, deleteNews, getUserNews & getGedcomNews functions are in
authentication.php - shouldn't they be in functions_db.php?

The gedcom_news, user_journal & now the blog module call getNewsItem, getUserNews or getGedcomNews as
appropriate - it should be simple to convert these to one function - just tell it the type of id we're giving
it.

Phew, its taken longer writing this that do the coding!

Uses modal dialogs for add & edit functions, editnews.php is now
redundant. Each user has a set of configuration parameters (previously
config was hard coded)
@ddrury
Copy link
Contributor Author

ddrury commented Jan 3, 2014

Withdrawn pending conversion to markdown

@ddrury ddrury closed this Jan 3, 2014
@kexuebin kexuebin mentioned this pull request May 20, 2019
@ricklach ricklach mentioned this pull request Jul 11, 2019
@makitso makitso mentioned this pull request Aug 11, 2019
@vytux-com vytux-com mentioned this pull request Oct 23, 2019
@ramu98 ramu98 mentioned this pull request Apr 7, 2020
@ghost ghost mentioned this pull request Aug 9, 2020
@webtrees-pesz webtrees-pesz mentioned this pull request Aug 18, 2023
@makitso makitso mentioned this pull request Oct 21, 2023
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.

None yet

1 participant