Ticket #38 (new Defekt) — at Version 3

Opened 15 years ago

Last modified 11 years ago

Problem med pp-mail om medlem byter namn

Reported by: jorgen.lindell@… Owned by: Todi
Priority: major Milestone:
Component: PirateWeb Keywords:
Cc:

Description (last modified by Todi) (diff)

Mailadresser tilldelas av ett Bot-jobb, men såvitt jag kan se så tilldelar det ingen ny (== lägger upp mailkonto) om det redan finns en assignad i databasfältet (Person.PartyEmail) för detta sedan innan.

Däremot så används en _annan_ metod (Person.PPMailAddress) på en del ställen som räknar fram en adress från namnet, vilket innebär att det ser ut att skickas mail med en avsändare som inte har nåt motsvarande konto upplagt, om jag fattar korrekt.

En korrekt lösning bör vara att den senare metoden använder den lagrade adressen om sådan finns.

För att hantera önskemål om ny pp-adress i fall av namnbyte så bör
PersonBasicDetails.ascx.cs modifieras så att den detekterar namnbyte och ställer fråga om man vill ändra sin pp-mailadress i samband med detta, och ett nytt event postas till boten som då lägger upp det nya kontot, helst kopplat till det gamla på nåt sätt.

Problemet detekterades i samband med:
[20:58:24] Andreas Käiväräinen: dn: medlem 3975 har bytt efternamn och vill således ha sin @pp.se-adress med sitt nya efternamn ist för det gamla

Change History

comment:1 Changed 15 years ago by Jörgen Lindell <jorgen.lindell@…>

Jag har inte nån databas så jag kan inte testa. Annars hade jag skrivit en patch...

comment:2 Changed 15 years ago by Jörgen Lindell <jorgen.lindell@…>

Kollade lite till i koden: Genom att PPMailAddress returnerar en adress som inte finns så kommer det inte heller att gå att skicka till medlemmen som bytt namn om det ska gå till pp-mailen.

comment:3 Changed 15 years ago by Todi

  • Description modified (diff)

/ <summary>
/ This is an ugly and temporary hack, need to replace with a property or a PirateWeb mail address
/ </summary>
public string PPMailAddress

'nuff said kanske :)

Förslag på lösning:

  1. Förändring i BasicPerson.PPMailAddress så att den inte försöker räkna fram en mailadress (utom i fallet med vissa specialfall) och istället då går på Person.PartyEmail, i de fall den finns. Lite osäker på vad det bästa är att göra om det inte finns någon PartyEmail.. använda no-reply@…?
  1. När byte av namn görs så ställs fråga om konto ska ändras till det nya namnet, och det gamla mailkontot kopplas till den nya adressen, alternativt nytt konto skapas om det inte går att koppla det gamla (men vi siktar på att koppla det gamla iaf).
Note: See TracTickets for help on using tickets.