eXpandFramework / eXpand

DevExpress XAF (eXpressApp) extension framework. 𝗹𝗶𝗻𝗸𝗲𝗱𝗶𝗻.𝗲𝘅𝗽𝗮𝗻𝗱𝗳𝗿𝗮𝗺𝗲𝘄𝗼𝗿𝗸.𝗰𝗼𝗺, 𝘆𝗼𝘂𝘁𝘂𝗯𝗲.𝗲𝘅𝗽𝗮𝗻𝗱𝗳𝗿𝗮𝗺𝗲𝘄𝗼𝗿𝗸.𝗰𝗼𝗺 and 𝘁𝘄𝗶𝘁𝘁𝗲𝗿 @𝗲𝘅𝗽𝗮𝗻𝗱𝗳𝗿𝗮𝗺𝗲𝘄𝗼𝗿𝗸 and or simply 𝗦𝘁𝗮𝗿/𝘄𝗮𝘁𝗰𝗵 this repository and get notified from 𝗚𝗶𝘁𝗛𝘂𝗯
http://expand.expandframework.com
Microsoft Public License
220 stars 114 forks source link

SequenceGenerator XAF Middle Tier Tenant not working #1066

Open yunits opened 1 month ago

yunits commented 1 month ago

Hi,

SequenceGenerator is not working when using Middle Tier Security.

I've attachted a simple project. -Login as admin@company1.com -Navigate to Sequence Storage -try to add new for "DomainObject1" and member "number". => can't save

Second test: -Navigate to Sequence Storage -press "test" action and see error message (test tries to get sequence storage to create in code "ObjectSpace.GetSequenceStorage(typeof(DomainObject1));"

DXApplication1.zip

apobekiaris commented 1 month ago

I will push a tenant support but for MiddleTier I am still consulting with the Xaf team if it is possible with the current implementation or it requires a totally different approach

apobekiaris commented 1 month ago

The pre-release 4.241.3.0 in the Reactive.XAF lab branch includes commits that relate to this task:

To minimize version conflicts we recommend that you use the Xpand.XAF.Core.All, Xpand.XAF.Win.All, Xpand.XAF.Web.All packages. Doing so, all packages will be at your disposal and .NET will add a dependecy only to those packages that you actually use and not to all (see the Modules installation-registrations youtube video).

Released packages: No packages released.

Please update the related Nuget packages and test if issues is addressed. These are nightly nuget packages available only from our NugetServer.

If you do not use these packages directly but through a module of the main eXpandFramework project, please wait for the bot to notify you again when integration is finished or update the related packages manually.

Thanks a lot for your contribution.

yunits commented 1 month ago

Thanks for your time, I will test later without Middle Tier. I let this ticket open as memory, until you remove your assignment.

apobekiaris commented 4 weeks ago

currently the Xaf team is occupied with 24.2, they will probably revisit the case later as this module is actually a user friendly encapulation of https://github.com/DevExpress-Examples/XAF_generate-a-sequential-number-for-a-persistent-object-within-a-database-transaction/tree/22.2.4%2B/CS/XPO/SequenceGenerator

apobekiaris commented 4 weeks ago

speaking about the MiddleTier part

apobekiaris commented 2 weeks ago

@yunits looking forward to see it u tested this one?

yunits commented 2 weeks ago

It looks like my issue has been fixed but since 4.241.3.1 I have a new issue: when I use another connection string than the default one, I get error messages when I try to login as super admin. "Login failed for user 'sa'.", "Unable to open database." and so on, but my connection string has not been changed and is still working with other applications.

working connection string: <add name="ConnectionString" connectionString="Integrated Security=SSPI;Pooling=false;Data Source=(localdb)\mssqllocaldb;Initial Catalog=SequenceGenerator_Service" />

not working connection string: <add name="ConnectionString" connectionString="Pooling=false;Data Source=SERVER;User ID=sa;Password=*************;TrustServerCertificate=True;Initial Catalog=SequenceGenerator_Service" />

I've created a sample without Middle Tier to test. SequenceGenerator.zip

Steps to reproduce:

  1. modify connection string
  2. login as admin@company1.com and see it works
  3. login as admin and see error messages
apobekiaris commented 2 weeks ago

The pre-release 4.241.3.0 in the Reactive.XAF lab branch includes commits that relate to this task:

To minimize version conflicts we recommend that you use the Xpand.XAF.Core.All, Xpand.XAF.Win.All, Xpand.XAF.Web.All packages. Doing so, all packages will be at your disposal and .NET will add a dependecy only to those packages that you actually use and not to all (see the Modules installation-registrations youtube video).

Released packages: No packages released.

Please update the related Nuget packages and test if issues is addressed. These are nightly nuget packages available only from our NugetServer.

If you do not use these packages directly but through a module of the main eXpandFramework project, please wait for the bot to notify you again when integration is finished or update the related packages manually.

Thanks a lot for your contribution.

yunits commented 2 weeks ago

thx, I tested this without Middle Tier and it is working now!

apobekiaris commented 1 week ago

do u impply that fails with MiddleTier?

yunits commented 6 days ago

I just didn't test this with Middle Tier as you said before that you have to consult with XAF team.

I could test it at the end of next week at the earliest.

apobekiaris commented 6 days ago

Middle Tier

apologies my bad I meant to say Multi Tenant so I assumed that case is tested?

yunits commented 6 days ago

Tenant support has been tested using GUI with 2 tenants and is working.

I have not tested sequence creation in code, if it matters.

apobekiaris commented 6 days ago

ok thnks in any case let me know will push in public soon