| Time |
Nick |
Message |
| 08:31 |
|
mmorgan joined #evergreen |
| 08:34 |
|
Rogan joined #evergreen |
| 10:08 |
|
mantis1 joined #evergreen |
| 10:30 |
|
mantis1 left #evergreen |
| 10:41 |
abneiman |
FYI merge pause this evening starting at midnight ET / 9pm PT |
| 11:07 |
|
Christineb joined #evergreen |
| 11:09 |
csharp_ |
abneiman++ |
| 11:10 |
|
Dyrcona joined #evergreen |
| 11:30 |
Bmagic |
does anyone know off hand if the Evergreen SIP 64 message can return the barcodes of holds that are ready for pickup, and not* those that are in transit or waiting for capture? |
| 11:32 |
Bmagic |
I found that the 10-character fixed-width section in the 63 message will elicit a different response depending on which flags you "Y". Example "Y " returns a list of all of the targeted holds. and " Y " will return all of the checked out items |
| 11:32 |
|
jihpringle joined #evergreen |
| 11:35 |
Bmagic |
I thinkI found it: " Y " |
| 11:41 |
csharp_ |
following up on bug 2137020 (thank you gmcharlt)... |
| 11:41 |
pinesol |
Launchpad bug 2137020 in Evergreen "In-query PCRUD permission testing can fail of classname is an SQL reserved word" [High,Fix committed] https://launchpad.net/bugs/2137020 |
| 11:42 |
csharp_ |
looks like we need more quoting to accommodate UPDATE and DELETE SQL |
| 11:42 |
csharp_ |
either that or change the class name from "asc" to something non-sql-reserved? |
| 11:43 |
csharp_ |
(the latter might be easier, but it's probably Bad™ to allow the reserved-word-in-sql landmine to remain |
| 11:43 |
csharp_ |
) |
| 11:49 |
csharp_ |
or maybe there's a way to create (or use) a C function that properly quotes the SQL string before sending it to the server? |
| 11:51 |
csharp_ |
meh, I guess just one-at-a-time backslash quotes are probably best |
| 11:52 |
csharp_ |
thank you Evergreen ducks |
| 11:56 |
berick |
*quack* |
| 12:00 |
csharp_ |
AFLACKK |
| 12:12 |
csharp_ |
okay, got it working after some tweak-rebuild-tweak-rebuild cycles |
| 12:18 |
Bmagic |
csharp_++ |
| 12:25 |
csharp_ |
https://bugs.launchpad.net/evergreen/+bug/2138747 - if someone can review it can make it in before the freeze |
| 12:26 |
pinesol |
Launchpad bug 2138747 in Evergreen "Generated SQL with unquoted class names" [High,New] - Assigned to Chris Sharp (chrissharp123) |
| 12:26 |
* csharp_ |
glances at gmcharlt in case he can look |
| 12:27 |
gmcharlt |
csharp_: I'll take a look at it today before the freeze |
| 14:01 |
csharp_ |
gmcharlt++ |
| 14:31 |
|
jihpringle joined #evergreen |
| 15:58 |
|
eby joined #evergreen |
| 15:58 |
|
briank joined #evergreen |
| 16:37 |
|
jihpringle joined #evergreen |
| 16:47 |
gmcharlt |
csharp_++ |
| 16:48 |
gmcharlt |
csharp_: I've tested and pushed your patch |
| 16:55 |
Dyrcona |
csharp_++ gmcharlt++ |
| 17:04 |
|
mmorgan left #evergreen |
| 17:05 |
|
jihpringle joined #evergreen |
| 17:08 |
pinesol |
News from commits: LP#2138747: update SQL-generation to properly quote strings <http://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=34c04e5a8fdebcc77708688db3f3b030ac0052f6> |
| 23:26 |
|
JBoyer joined #evergreen |
