10 ელ.ფოსტის ვერიფიკაცია #34
Conversation
There was a problem hiding this comment.
@IliaNoz სრულად ვერ გავტესტე, რატომღაც Mailhog - ი არ მუშაობს ჩემთან, თუმცა იქამდე რაც შევამჩნიე კომენტარები დაგიტოვე და გავმართავ ლოკალურად და კიდევ გავტესტავ.
ერთი რაც ვერ მივხვდი, ახალი იუზერი შევქმენი Drush - ით. ელ.ფოსტის და პაროლის გარეშე და პაროლის დარესეტება არ მთხოვა, პირდაპი მეილის ვერიფიკაციას მთხოვს, ოღონდ მეილის ვერიფიკაციასაც ისე ცდილობს რომ მეილს არ მასეტინებს საერთოდ და მიწერს ვერ გაიგზავნაო.
| {% if header_vars.is_email_verified == 0 and header_vars.show_text %} | ||
| <div class="container" id="emailVerContainer"> | ||
| <div class="alert alert-danger" role="alert" > | ||
| გთხოვთ ელ.ფოსტის ვერიფიკაცია გაიაროთ. |
There was a problem hiding this comment.
თუ შეიძლება რომ ეს ცალკე მესიჯი იყოს და ტექსტიც შევცვალოთ. "თქვენ არ გაქვთ გავლილი ელ.ფოსტის ვერიფიკაცია." თუმცა დარჩეს წითელი Error სტატუსით.
| <div class="container" id="emailVerContainer"> | ||
| <div class="alert alert-danger" role="alert" > | ||
| გთხოვთ ელ.ფოსტის ვერიფიკაცია გაიაროთ. | ||
| <a href="#" id="emailVerification">გააგზავნეთ გასააქტიურებელი ლინკი ელ.ფოსტაზე</a> |
There was a problem hiding this comment.
ეს იყოს მეორე მესიჯი რომელიც ზედა წითელის შემდეგ გამოჩნდება: "ელ.ფოსტის ვერიფიკაციის დასაწყებად დააკლიკეთ აქ." ოღონდ მხოლოდ "დააკლიკეთ აქ" იყოს გალინკული და თავად მესიჯს Warning - ის სტატუსი ქონდეს, ყვითლად რომ გამოჩნდეს.
| $message = "ვერ მოხერხდა ელ.ფოსტის გაგზავნა, გთხოვთ ცადოთ თავიდან ან მიმარტოთ ადმინისტრაციას"; | ||
| } | ||
| else { | ||
| $message = "ელ.ფოსტა წამრატებით გაიგზავნა შემდეგ მისამართზე: ${to} გთხოვთ მიჰყვეთ ელ.ფოსტაზე გამოგზავნილ ინსტრუქციას"; |
There was a problem hiding this comment.
@IliaNoz ესეც შევცვალოთ შემდეგით: "ვერიფიკაციის ბმული გამოგზავნილია ${to} - ზე გთხოვთ მიჰყვეთ მას."
| */ | ||
| public function validateForm(array &$form, FormStateInterface $form_state) { | ||
| if (mb_strlen($form_state->getValue('password')) < 6) { | ||
| $form_state->setErrorByName('length', $this->t('პაროლი მინიმუმ 6 სიმბოლოსგან უნდა შედგებოდეს')); |
There was a problem hiding this comment.
@IliaNoz აქ წერტილი აკლია წინადადების ბოლოს.
| $form_state->setErrorByName('length', $this->t('პაროლი მინიმუმ 6 სიმბოლოსგან უნდა შედგებოდეს')); | ||
| } | ||
| if ($form_state->getValue('password') != $form_state->getValue('password_confirm')) { | ||
| $form_state->setErrorByName('mismatch', $this->t('გამეორებული პაროლი არ ემთხვევა მითითებულ პაროლს')); |
There was a problem hiding this comment.
@IliaNoz აქაც წერტილი აკლია.
| add_disable: none | ||
| edit_disable: none | ||
| id: user.user.field_social_auth_password | ||
| field_name: field_social_auth_password |
There was a problem hiding this comment.
@IliaNoz ამ ველს ცოტა დამაბნეველი სახელი აქვს, თუ იმისთვისაა რომ შევამოწმოთ რეალურად პაროლი დასეტილია თუ არა, მგონი მაგისთვის ცალკე ველი არ გვჭირდება.
ზუსტად არ ვიცი მარა სავარაუდოდ ესე შეიძლება შემოწმება რეალურად დასეტილია თუ არა:
$user->get('pass')->existing
checkExistingPassword() მეთოდის იმპლემენტაციას შეგიძლია გადახედო აქ: https://api.drupal.org/api/drupal/core%21modules%21user%21src%21Entity%21User.php/8.2.x
|
|
||
| $result = $this->mailManager->mail($module, $key, $to, $langcode, $params, NULL, $send); | ||
| if ($result['result'] != TRUE) { | ||
| $message = "ვერ მოხერხდა ელ.ფოსტის გაგზავნა, გთხოვთ ცადოთ თავიდან ან მიმარტოთ ადმინისტრაციას"; |
There was a problem hiding this comment.
- ეს ტექსტიც უნდა შევცვალოთ, მწვანედ ჩანს არადა შეცდომაა, ამიტომ წითლად უნდა ჩანდეს, ასევე "მიმარტოთ" წერია, პატარა typo - ა.
- შევქმენი ახალი იუზერი, ელ.ფოსტის გარეშე, თუმცა რომ დავაკლიკე ვერიფიკაციას დამიწერა ეს მესიჯი რომ ვერ მოხდა ელ.ფოსტის გაგზავნაო, როცა უნდა დამიწეროს რომ ელ.ფოსტა მაქვს შესავსები ალბათ.
#10
კონტექსტი
ყველა მომხმარებელს ვისაც რეგისტრაცია გავლილი აქვს Facebook-ით, ლოგინის შემდგომ თუ პაროლი არ აქვს დაყენებული აიძულებს რომ პაროლი დააყენონ.
მომხმარებლებს რომელთაც არ აქვთ ელ.ფოსტა ვერიფიცირებული, ეგზავნებათ ვერიფიკაციის მეილი რის შემდეგაც ხდებიან ვერიფიცირებულები.
თუ მომხმარებელი შეცვლის ელ.პოსტას, ხდება მეილზე ვერიფიკაციის ლინკის თავიდან გაგზავნა.
ინსტალაცია
1 -
drush cim2 -
drush cr