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

Working on implementing SwiftMailer #3265

Conversation

euantorano
Copy link
Member

@euantorano euantorano commented Jun 16, 2018

Fixes #2909

This removes the old mail handler class and substitutes SwiftMailer for sending emails.

This code is not yet tested, but I will be testing it during tomorrow.

Mark Vincent and others added 30 commits September 28, 2017 17:18
* Adding a composer configuration file for the project, with no defined dependencies.

* Rquire autoload file

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Removing global.php from files to autoload
* Adding Laravel container

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Updating doc comment for app() function

Signed-off-by: Euan Torano <euantorano@gmail.com>
* Initial admin routing setup

* Adding a basic layout for the ACP
* Starting work on moving front end to Twig

This marks the start of integrating Twig into the front end. There are two global variables shared with all views - mybb and lang.

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Starting the long process of template translation

Not much to show yet, but this is a first step towards seeing that key variables such as $mybb, $lang and $theme can be accessed.

Need to sort out loading of stylesheets still to make things look decent.

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Changing how language is registered

There is a new LangExtension that provides a `trans` function along with the `lang` global.

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Showing drafts by adding drafts to an array

Moved setting of theme global to theme extension.

Need to fix the use of |raw in some places - we should generate the HTML inside the views themselves instead…

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Finished the majority of work on the drafts page

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Add return type hints

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Fixing trow usage currently following 1.8 standard

Signed-off-by: Euan Torano <euantorano@gmail.com>
mybb#2975)

* Working on mybb#2972 Template Conversion to Twig Format (warnings.php)
Part 1 - User warning log

* Coding standards

* Working on mybb#2972 Template Conversion to Twig Format (warnings.php)
Part 2 - Warning details

* Working on mybb#2972 Template Conversion to Twig Format (warnings.php)
Part 3 - Warning a user
…hp) (mybb#2983)

* Working on mybb#2972 Template Conversion to Twig Format (sendthread.php)

* Removed double escaping
* Starting work on translatig subscribed threads

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Finish subscriptions page in UCP

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Use `&amp;` rather than `&` in URL.

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Move HTML out of PHP into view

Also use `threadprefix` as `raw` as it can contain HTML.
* For mybb#2972

* added whitespace

* remove htmlspecialchars_uni
mybb#2984)

* Working on mybb#2972 Template Conversion to Twig Format (calendar.php)
Part 1 - Adding an event

* Working on mybb#2972 Template Conversion to Twig Format (calendar.php)
Part 2 - Editing an event

* Working on mybb#2972 Template Conversion to Twig Format (calendar.php)
Part 3 - Moving event

* Working on mybb#2972 Template Conversion to Twig Format (calendar.php)
Part 4 - Viewing an event

* Working on mybb#2972 Template Conversion to Twig Format (calendar.php)
Part 5 - Day view

* Working on mybb#2972 Template Conversion to Twig Format (calendar.php)
Part 6 - Week view

* Working on mybb#2972 Template Conversion to Twig Format (calendar.php)
Part 7 - Calendar

* Moved gobutton to inline

* Code updates
-Added selected and checked HTML to templates
-Move day select options to array

* More code updates
- Moved month select options to array
- Split bottom calendar jump/month jump into seperate Twig file
- Converted user stars to Twig
…ing on PHP 7.2 (mybb#2995)

* [FIX] string to array conversation causing Illegal string offset warning
on PHP 7.2

* Use 4 spaces instead of tab
Signed-off-by: Euan Torano <euantorano@gmail.com>
…hp) (mybb#2998)

* Working on mybb#2972 Template Conversion to Twig Format (memberlist.php)
Part 1 - Member list

* Working on mybb#2972 Template Conversion to Twig Format (memberlist.php)
Part 2 - Member search

* Coding improvements

* Updated formatting
…mybb#2991)

* Working on mybb#2972 Template Conversion to Twig Format (contact.php)

* Indented template
* Postbit — For mybb#2972

[WIP]

* removed horizontal postbit

* postbit_groupimage

* postbit_userstar

* Online Status

postbit_online, postbit_away. postbit_offline

* postbit_avatar

* postbit_find

* postbit_pm

* postbit_rep_button

* postbit_www

* postbit_email

* User Details

postbit_author_user, postbit_author_guest

* postbit_reputation

* Warning Info

postbit_warninglevel, postbit_warn

* postbit_purgespammer

* Custom Profile Fields

postbit_profilefield, postbit_profilefield_multiselect,
postbit_profilefield_multiselect_value

* Private Messages

postbit_reply_pm, postbit_replyall_pm, postbit_forward_pm,

* Edit Message

postbit_editedby, postbit_editedby_editreason

* Post Edit

postbit_editreason, postbit_edit

* Quick Delete/Restore

postbit_quickdelete, postbit_quickrestore

* Post Links/Inline Moderation

postbit_posturl, postbit_inlinecheck

* postbit_quote

* postbit_multiquote

* postbit_report

* Announcements

announcement_edit, announcement_quickdelete

* Post IP Display

postbit_iplogged_show, postbit_iplogged_hiden

* postbit_status

* postbit_signature

* postbit_icon

* postbit_deleted

* postbit_ignored

* postbit_deleted_member

* Attachments

postbit_attachments, postbit_attachments_thumbnails,
postbit_attachments_thumbnails_thumbnail, postbit_attachments_images,
postbit_attachments_images_image, postbit_attachments_attachment,
postbit_attachments_attachment_unapproved

* Code Cleanuo

* More Code Cleanup
…php) (mybb#3018)

* Working on mybb#2972 Template Conversion to Twig Format (managegroup.php)

* Coding improvements
* Initial Cleanup and Main Template Conversion [WIP]

* report_thanks

* report_thanks

* Errors

report_error_nomodal,

* report_duplicate

* Report Reason

report_reason, report_reasons

* Separated `nomodal` Content

...and code cleanup, refactoring info sent to Twig, and fixing an error
where a legacy language string was left in the `report_reasons.twig`
template.

* Adjust Alignment

* Remove unused assignment

* Remove Template Cache List
…ybb#3032)

* Working on mybb#2972 Template Conversion to Twig Format (member.php)

* Added label to inputs
…ybb#3019)

* Working on mybb#2972 Template Conversion to Twig Format (portal.php)

* Updated attachments

* Coding standards
mybb#3039)

* Working on mybb#2972 Template Conversion to Twig Format (editpost.php)

* Code updates
Shade- and others added 12 commits June 13, 2018 10:14
…er) (mybb#3259)

* Converted modcp finduser

* Removed redundant condition
mybb#3252)

* Converted smilieinsert

* Unlocked smilie inclusion

* Fixed indent, moved HTML from php to twig

* Minor fixes
…b#3253)

* Converted footer

* Added missing post_code_string

* Fixed indent
…r) (mybb#3242)

* Converted usercp_avatar

* Fixed indenting and code style

* Fixed bburl -> bbname
…bb#3249)

* Converted mycodes

* Deleted old parser/video.twig template

* Restored onclick and extraclass for smilie template

* Moved HTML from php to twig
…ns) (mybb#3243)

* Converted usercp options

* Converted themeselector and lang fixes to templates

* Fixes indent and minor syntax change
* Merge remote-tracking branch 'upstream/develop/1.9' into fix-2976-lang_parse

* Fixes Copy Paste Mistake
This removes the old mail handler class and substitutes Swiftmailer for sending emails.

This code is not yet tested, but I will be testing it during tomorrow.

Signed-off-by: Euan Torano <euantorano@gmail.com>
@euantorano euantorano added t:enhancement Type: Enhancement. Contains minor improvements b:1.9 Branch: 1.9.x labels Jun 16, 2018
@euantorano euantorano added this to the 1.9.0 milestone Jun 16, 2018
Signed-off-by: Euan Torano <euantorano@gmail.com>
@euantorano euantorano force-pushed the develop/1.9-add-swiftmailer-for-mail branch from c48a2c4 to bab7b3f Compare June 16, 2018 22:12
Signed-off-by: Euan Torano <euantorano@gmail.com>
Signed-off-by: Euan Torano <euantorano@gmail.com>
Signed-off-by: Euan Torano <euantorano@gmail.com>
@Shade-
Copy link
Contributor

Shade- commented Jun 23, 2018

Two issues found:

  • errors are not handled properly, internal Exceptions are displayed;
  • send_mail_queue is registered as a shutdown function, but it seems shutdown functions do not fire in 1.9. I don't know if it's an issue with my local install or if it's screwed for everyone, but a test might be worth. I don't know to debug shutdown functions unfortunately.

@effone effone added the s:conflict Status: Conflict. Solution incompatible with current code label Jul 22, 2018
@zygzagZ
Copy link
Contributor

zygzagZ commented Jul 31, 2018

send_mail_queue shutdown function fixed in #3371

@euantorano
Copy link
Member Author

Yep, I just need to rebase the develop/1.9 branch @zygzagZ, which I've been meaning to do for ages.

@euantorano euantorano closed this May 31, 2019
@euantorano euantorano deleted the develop/1.9-add-swiftmailer-for-mail branch May 31, 2019 17:57
@rajat315315
Copy link
Contributor

I can't understand, there are too many commits for this SwiftMail feature.

@euantorano
Copy link
Member Author

euantorano commented Jul 24, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
b:1.9 Branch: 1.9.x s:conflict Status: Conflict. Solution incompatible with current code t:enhancement Type: Enhancement. Contains minor improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants