A Subscription Event should be able to tell from where it is triggered

Answers

• mohana_cse06 Member Posts: 5,504

  2016-10-26

  yes please. at present we have only RunTrigger as parameter

  
  -Mohana
  http://mohana-dynamicsnav.blogspot.in/
  https://www.facebook.com/MohanaDynamicsNav

  0
• binilabraham Member Posts: 45

  2016-10-26 edited 2016-10-26

  If I'm not wrong, the subscriber code will execute whenever a sales line deleted

  Software Developer,
  Archerpoint India Pvt. Ltd,Chennai.

  0
• Kishorm Member Posts: 921

  2016-10-26

  @Remco_Reinking - I don't think this is possible. Even if you tried the Interop stuff in Vjeko's blog as I would have thought that would identify the object that the subscription function was in (or possibly the Sales Line table itself) - this is just a hunch though so it's worth trying out.

  0
• Remco_Reinking Member Posts: 74

  2016-10-27 edited 2016-10-27

  @binilabraham : yes, and that is just the problem. I dont want to do run some code if the delete is triggered from CU80.
  @Kishorm : It would be possible, but yes I agree, it is not easy.
  This is an example of the stacktrace:
  
  
  Best guess would be to look through the stacktrace, and to check if there is a line with CodeUnit80 OnRun.
  
  If I gather the stacktrace and show it in the Error Message it shows:
  
  
  So yes, maybe I'll give it a try.
  Code would be something like this:
  
  
  
  
  
  

  StackTrace OnDelete while posting.png 6.8K

  StackTrace in ErrorMessage.png 17.8K

  StackTraceTest.png 22.5K

  0
• Kishorm Member Posts: 921

  2016-10-27 Answer ✓

  @Remco_Reinking Great thinking about the stack trace - I agree that would be the way to go. You can create a function...
  

  ExistsInStackTrace(ObjectType,ObjectID,FunctionName)
  

  ...and then you could use it for other stuff if you needed to

  5
• Remco_Reinking Member Posts: 74

  2016-10-28

  But I wonder what is the impact on performance, because it would be triggered per line deleted. Anyway, for now this is the only solution I guess.

  0
• Kishorm Member Posts: 921

  2016-10-28

  I wouldn't expect a significant impact on performance as it's all CPU based code - no access to SQL tables.
  
  You could also check the RunTrigger parameter first - so if this is TRUE then you know already that it's not being called from CU 80 and don't need to check the StackTrace

  0
• Marije_Brummel Member, Moderators Design Patterns Posts: 4,262

  2016-10-28

  Guys, this is not what events are intended to be used for. The idea about events is to have loosely coupled code from Microsoft that in theory would allow MSFT to replace Codeunit 80 with something else without breaking your code.
  
  Remco, you need to rethink your business case.

  Marije Brummel | Freelance Business Central Specialist

  0
• Sog Member Posts: 1,023

  2016-10-28

  How about you check if a sales invoice line exists related to the deleted sales line. Then you shouldn't have to (ab)use the stacktrace.

  |Pressing F1 is so much faster than opening your browser|
  |To-Increase|

  0
• Remco_Reinking Member Posts: 74

  2016-10-28

  @Mark_Brummel IF for whatever reason one wants to use this OnDeleteSalesLine event it should also be possible to skip it if it is triggered from the posting routine. That is my opinion. Loosely coupled, yes, but always with the version in mind were it is supposed to run on. So if Microsoft comes up with another posting routine, the events should also be updated according to this.
  
  @Sog That is a good option, but it would definitely have an impact on the performance.

  0
• Kishorm Member Posts: 921

  2016-10-28

  @Remco_Reinking wants to...
  
  1) ...do something different depending on whether the line is being deleted from the posting codeunit or not - not unreasonable (standard NAV does some stuff slightly differently in CU 80 depending on whether it's called from the Inventory Pick/Put-away or not)
  
  2) ...try to achieve this by only having code in an event, I.e without making changes to standard NAV objects such as codeunit 80 - again not unreasonable (this is the way we are being to told to do things these days)
  
  Yes, he has come up with a creative solution - but is this really a bad thing?
  
  Until someone comes up with a better solution, I think what has been suggested is quite reasonable

  0
• Kishorm Member Posts: 921

  2016-10-29

  @Remco_Reinking - I think in pretty much most other cases the Sales Line DELETE method will be called with the trigger set to true, so would it work for your situation to just check the RunTrigger parameter and only run your code if that is TRUE?

  0