Re: PDO\Sqlite: Add transaction mode attribute

From: Date: Wed, 03 Sep 2025 22:38:27 +0000
Subject: Re: PDO\Sqlite: Add transaction mode attribute
References: 1  Groups: php.internals 
Request: Send a blank email to internals+get-128621@lists.php.net to get a copy of this message
Hi, Is there anything I could do here to get this reviewed before the 8.5 release? As mentioned earlier the proposed implementation should not be subject to the feature freeze. <https://github.com/php/php-src/pull/19317> Thank you. Samuel On August 12, 2025, "Samuel Štancl" <samuel@archte.ch> wrote: > By enums I'm assuming you mean PHP enums. That could work and would > solve my issue of trying to represent the values of the attribute > (only thing I'm not sure about is Pdo\SqliteTransactionMode being > directly in the Pdo namespace). I've included a sample implementation > in this comment <https://github.com/php/php- > src/pull/19317#issuecomment-3177353751> > > On August 12, 2025, Morgan <weedpacket@varteg.nz> wrote: > > On 2025-08-12 07:35, Samuel Štancl wrote: > > > > > > Hi, I've opened a PR adding a new attribute to PDO\Sqlite: > > https:// > > > github.com/php/php-src/pull/19317 > > > <<https://github.com/php/php- > > src/>  > > > pull/19317?  > > > > > notification_referrer_id=NT_kwDOAfgLhrQxNzkxMjcwMjYzOTozMzAzMzA5NA#issuecomment- > > 3176177745> > > >  > > As far as the form the attribute takes goes, an Enum with the > > values  > > DEFERRED/IMMEDIATE/EXCLUSIVE would do the job. > > > > (As for calling it an attribute, I thought at first that what was > > being  > > suggested was an Attribute, rather than a connection option.) >

« previous php.internals (#128621) next »