Mandantentrennung NAV und SWS

10. Januar 2020 11:33

Hallo Zusammen,

Bei uns konnte offenbar in SwissSalary von Mandant B(Neu seperate Firma) in Mandant A Daten abändern.
Dies sollte weder im SwS noch im Standard möglich sein bei uns.
Was muss noch angepast werden?

Nach meinem Kenntnisstand sollten die Mandaten doch so weit ich weiss abgetrennt sein.


Danke für eure Hilfe und gruss

Re: Mandantentrennung NAV und SWS

10. Januar 2020 11:52

Hallo,
Nach meinem Kenntnisstand sollten die Mandaten doch so weit ich weiss abgetrennt sein.


das kann man nur mit einem eindeutigen JEIN beantworten.

In NAV kann es Mandanten- übergreifende Tabellen geben. Welche aus dem Standard und welche die der Kunde selbst als solche deklariert hat. (was man besser lassen sollte, aber manchmal geht es nicht anders :wink: ).

Dann kann jeder Mandant natürlich Code enthalten der per CHANGECOMPANY auf die Daten des anderen Mandanten zugreift. Auch damit können Daten in anderen Mandanten geändert werden.

Und dann gibt es da noch die EventSubscriber, die wenn sie nicht sauber programmiert sind, u.U. Daten im falschen Mandanten ändern.

Gruß Fiddi

Re: Mandantentrennung NAV und SWS

13. Januar 2020 10:50

fiddi hat geschrieben:Und dann gibt es da noch die EventSubscriber, die wenn sie nicht sauber programmiert sind, u.U. Daten im falschen Mandanten ändern.

:shock: Hast du dafür mal ein Beispiel, wie man es nicht programmieren sollte, bzw. was man tun muss, um solch ein "unerwartetes Programmverhalten" zu vermeiden?

Re: Mandantentrennung NAV und SWS

13. Januar 2020 11:30

Hallo,
war dir das noch nicht bekannt?

Die meisten Eventsubscriber prüfen bei OnInsert,OnModify auf Rec.ISTEMPORARY. Was sie aber nicht tun, ist zu prüfen, ob sie auch im richtigen Mandanten sind. Geschweige denn, dass Sie das richtige tun (können).

Machst du ein Rec.CHANGECOMPANY und dann ein Rec.MODIFY(egal ob TRUE oder FALSE) werden auch die Eventsubscriber aufgerufen.
Lokale Record-Variablen der Subscriber oder auch davon aufgerufene Codeunits bleiben aber im aufrufenden Mandanten, was schon mal zu Komplikationen führen kann. :roll:

Das war im alten System auch nicht anders, wenn du ein MODIFY(true) aufgerufen hast, weshalb man das tunlichst vermieden hat. Mit den Events kannst du das aber nicht mehr verhindern.

Gruß Fiddi

Re: Mandantentrennung NAV und SWS

13. Januar 2020 16:35

Die Problematik mit CHANGECOMPANY ist mir aus den alten Zeiten noch bekannt.
Aber wie kann ich herausfinden, in welcher "Mandanten-Instanz" ein Record bzw. ein RecordRef ausgeführt wird?

Bisher fangen alle meine EventSubscriber mit folgenden Zeilen an:
Code:
IF (COMPANYNAME = '') OR RecRef.ISTEMPORARY THEN BEGIN
  EXIT;
END;


[Edit] Gefunden! Rec.CURRENTCOMPANY bzw. RecRef.CURRENTCOMPANY zeigt an, in welchem Mandanten die Variable gerade läuft.
Ich glaube, ich muss da mal ein paar EventSubscriber auf den aktuellen Stand der Wissenschaft bringen. ;-)

Re: Mandantentrennung NAV und SWS

13. Januar 2020 16:44

Hallo,

da bist du dann aber einer der wenigen, die damit umgehen könnten. MS bekommt das selbst nicht auf die Reihe, und ob andere Extensions die du nutzt das tun, weißt du nicht.

Daher kann die Antwort eigentlich nur lauten: Finger Weg! oder eine Tabelle benutzen, von der du weißt, das sie niemand anderes benutzt.

Gruß Fiddi

Re: Mandantentrennung NAV und SWS

13. Januar 2020 16:55

Streng genommen kann ich keiner Record-Variablen mehr vertrauen, denn sie könnte ja in einer vorherigen Instanz direkt oder indirekt durch einen EventSubscriber im Kontext eines anderen Mandanten laufen.
Somit müsste ich jedesmal, wenn ich von einer Record-Variablen ausgehend die Daten in anderen Tabellen ändern möchte, immer zuerst die andere Record-Variable mit CHANGECOMPANY auf CURRENTCOMPANY der übergeordneten Variable setzen.

In "unseren" EventSubscribern werde ich mich jetzt mit dem Thema intensiver auseinandersetzen und prüfen, wie ich in dem Fall reagiere.
a) EventSubscriber mit EXIT verlassen
b) EventSubscriber mit ERROR abbrechen
c) NewRecord.CHANGECOMPANY(OldRecord.CurrentCompany);
d) OldRecord.CurrentCompany per Parameter an die aufgerufene Funktion übergeben und dort mit a)-d) darauf reagieren.

Re: Mandantentrennung NAV und SWS

13. Januar 2020 17:21

Hallo,

spätestens bei einer Codeunit, die von deinem Eventsubscriber aufgerufen wird, oder den lokalen Variablen von Sales Line die du in deinem Subscriber ist dann aber Schluss mit lustig.
Das kannst du für deine Subscriber von Fall zu Fall tun, MS wirdsich darum wohl nicht kümmern.

Gruß Fiddi