-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Fix for Postgres 12 #367
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix for Postgres 12 #367
Conversation
Was removed in V12, invalid since V8. See https://www.postgresql.org/docs/8.3/catalog-pg-attrdef.html
Stop using adsrc column
|
When will this be merged into official docker image? |
|
Any updates on this? |
|
Can confirm that commit 717d9e6 allows me to browse tables on postgres 12 (fedora 30) |
|
I would like to see this merged. When will it be merged? |
|
I can also confirm that this patch works, we didn't find any issue with that |
|
Hi @vrana - any idea when you can merge/release this? It would be nice to have this get to the downstream docker stuff I'm using... |
|
HI mikebski, I try your patch for PostgreSQL 12 but don't wort for me in Adminer 4.7.4. I fix it using this in the line has change (note var $Dd instead $identity_column): foreach (get_rows("SELECT a.attname AS field, format_type(a.atttypid, a.atttypmod) AS full_type, pg_get_expr(d.adbin, d.adrelid) AS default, a.attnotnull::int, col_description(c.oid, a.attnum) AS comment, $Dd AS identity |
|
@renderpci - How did you patch it? If you just pasted this in to the compiled file, the variable name looks like it's different because it is minified. In the uncompiled source, the variable is |
|
@mikebski - Yes, we prove it in the compiled file, we don't have Adminer dev version installed... |
|
Merged, thanks. |
https://www.postgresql.org/docs/8.3/catalog-pg-attrdef.html
According to this page, the fix in this PR should work from PG 8.3 on.