Skip to content

Selection insert fix - #102

Open
Unlink wants to merge 1 commit into
nette:masterfrom
Unlink:selection-insert-fix
Open

Selection insert fix#102
Unlink wants to merge 1 commit into
nette:masterfrom
Unlink:selection-insert-fix

Conversation

@Unlink

@Unlink Unlink commented Sep 27, 2015

Copy link
Copy Markdown
Contributor

Fixes #80, #41
https://forum.nette.org/cs/24391-standartni-html-formular-a-zpracovani-post-pozadavku-v-presenteru-jine-stranky#p163128

In Structure there is autoincrement information in columns which i need in insert method but i'm not sure if i can extend it interface so i add method for this only into Selection.

Any ideas how to get this information in Selection better way?

I also changed return value when inserting multiple values. So insert can returns ActiveRow or number of affected rows.

@Unlink
Unlink force-pushed the selection-insert-fix branch from 071f47c to bf899b4 Compare September 27, 2015 18:33
@dg
dg force-pushed the master branch 3 times, most recently from 6c43c50 to 6690dde Compare November 5, 2015 00:42
@dg
dg force-pushed the master branch 4 times, most recently from 0771c2f to 425ad23 Compare April 21, 2016 13:03
@dg
dg force-pushed the master branch 2 times, most recently from 59a6661 to b660054 Compare May 30, 2016 14:28
@Unlink
Unlink force-pushed the selection-insert-fix branch from aa5a5f2 to 338fa52 Compare June 2, 2016 14:08
@Unlink

Unlink commented Jun 10, 2016

Copy link
Copy Markdown
Contributor Author

@dg i've updated this, so what do you think? or #105 way should be better?

@dg
dg force-pushed the master branch 2 times, most recently from 5c87d56 to d9fd67a Compare June 27, 2016 23:11
@dg dg mentioned this pull request Jun 28, 2016
@dg
dg force-pushed the master branch 11 times, most recently from 221f038 to 276f489 Compare October 18, 2016 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant