IRC log for #uphpu on 20151009

04:15.30*** join/#uphpu clintberry2 (~clintberr@170-72-15-42.ut.vivintwireless.net)
04:39.44*** join/#uphpu stevecoug (~stevecoug@oalug/member/stevecoug)
04:39.45*** mode/#UPHPU [+v stevecoug] by ChanServ
10:18.58*** join/#uphpu trave (sid38181@gateway/web/irccloud.com/x-vjvaubbfasghnijp)
12:34.28*** join/#uphpu eggyknap (~jtolley@unaffiliated/eggyknap)
12:34.28*** mode/#UPHPU [+v eggyknap] by ChanServ
13:55.58*** join/#uphpu bbankes (~bbankes@50-200-100-242-static.hfc.comcastbusiness.net)
14:01.48*** join/#uphpu clintberry2 (~clintberr@170-72-15-42.ut.vivintwireless.net)
14:26.03*** join/#uphpu stevecoug (~stevecoug@oalug/member/stevecoug)
14:26.03*** mode/#UPHPU [+v stevecoug] by ChanServ
14:52.19*** join/#uphpu josephscott (~josephsco@192.0.125.251)
15:10.41*** join/#uphpu ninnypants (~ninnypant@unaffiliated/ninnypants)
15:10.41*** mode/#UPHPU [+v ninnypants] by ChanServ
15:10.53ninnypantsmgeary: I can take a look if you still need help
15:11.04mgearyhey there
15:11.06mgearyi do
15:11.10mgearylemme spin things up a bit
15:11.25mgearyi’m trying to fix an apparent bug in an off-the-shelf theme
15:16.20mgearyok, so i have a custom post here that is being output in a widget, “sh_testimonial”. There is a VisualComposer widget (that was also broken), that allows you to specify a category to filter on when outputting . I believe i managed to fix that VC widget correctly, but it still doesn’t run the query correctly.
15:16.34mgearythe SQL is generating a “Where (0 = 1)” clause
15:16.38mgearyso i get zero results
15:16.53mgearylemme pastbin stuff
15:16.59ninnypantsinteresting
15:17.24mgearyhttp://pastebin.com/sScx0AMG
15:18.07mgearyso, line 24 there is the original code. If that is enabled, i get zero results. But without it, i don’t get results filtered by category
15:18.38mgearyi’ll post the before and after SQL
15:19.32mgearyhttp://pastebin.com/wt6jmMrp
15:19.54mgearyfurthermore, enabling that “cat” filter doesn’t even reflect anywhere in the SQL, but for the (0=1)
15:20.14mgearyit appears to be related to this: http://stackoverflow.com/questions/11148983/using-tax-query-creates-a-0-1-in-wp-query-request, but i’m not sure that’s the same fix
15:22.13ninnypantsmgeary: what's the value returned by `sh_set( $instance, 'cat' )`
15:23.29*** join/#uphpu stevecoug (~stevecoug@oalug/member/stevecoug)
15:23.29*** mode/#UPHPU [+v stevecoug] by ChanServ
15:23.44mgeary“39”
15:23.54mgearywhich *should* be the category ID i’m filtering on
15:24.32mgearybut i don’t know enough about the taxonomies in WP to know if that isn’t the “wrong kind” of category. It does appear in the custom post admin edit screen, so i think that jives
15:26.02ninnypantsis it a custom taxonomy or is it in the default post category taxonomy?
15:26.49ninnypantsa lot of the time if WP_Tax_Query runs into an error it'll just return 0=1
15:27.36ninnypantsif it's a custom taxonomy then you're probably running into it because 39 isn't a term id in that taxonomy
15:34.02mgearysorry. got called away
15:34.29mgearyhow do i know what kind of taxonomy it is?
15:36.13ninnypantsbest way would probably be to just look where you found the ID in the admin
15:36.26ninnypantsor are they giving you a dropdown of categories?
15:37.15mgearythey do give a dropdown of categories. I inspected the <option> to confirm that it had an ID of 39
15:38.25mgearybut i’m not sure if that’s what you’re really asking
15:38.56ninnypantsdid you add lines 20 and 21 in this http://pastebin.com/sScx0AMG
15:39.07mgearyyes
15:39.19mgearythat was me trying to get solutions from StackOverflow
15:39.27mgearyhad no positive impact
15:40.19*** join/#uphpu clintberry2 (~clintberr@205.197.215.90)
15:40.24mgearyi can confirm that the “Testimonial” custom post appears to have its own set of Categories, as opposed to using the categories available on a normal “Post”. Doest that mean it’s a custom taxonomy?
15:40.35ninnypantshmm can you post the code for the form method of that class
15:40.45ninnypantsyeah
15:42.48mgearyare you talking about the form where the admin selects the category
15:42.49mgeary?
15:43.21ninnypantsyeah and that should be in the form method
15:43.36mgearyyep
15:44.54mgearyhttp://pastebin.com/q02gSBgx
15:45.02mgearyso, i did change this.
15:45.24mgearybecause while it was presenting a list of Testimonial categories, selecting one of them and submitting didn’t “stick"
15:46.21ninnypantsok that looks good I wanted to make sure you were using the taxonomy parameter for `wp_dropdown_categories()`
15:46.28ninnypantswasn't sure how they'd set it up
15:46.36mgearyi modified fields 22 & 23
15:47.59mgearybefore that was hardcoding the field name there as “cat"
15:49.12ninnypantsmgeary: try this http://pastebin.com/p9SH1LsS
15:51.10mgearyno joy
15:51.15mgearysql produced is:
15:51.16mgearySELECT SQL_CALC_FOUND_ROWS wp_posts.ID FROM wp_posts WHERE 1=1 AND ( 0 = 1 ) AND wp_posts.post_type = 'sh_testimonial' AND (wp_posts.post_status = 'publish' OR wp_posts.post_status = 'private') GROUP BY wp_posts.ID ORDER BY wp_posts.post_date DESC LIMIT 0, 4
15:51.29mgearystill not putting that category id in the query, and still showing that 0 = 1
15:55.00ninnypantshmm that tax query should be good
15:55.30ninnypantsmaybe try force casting term in that array to an int
16:02.27ninnypantsmgeary: not sure if forcing it to int would really help you could also add the field paramater to that array and set it to 'term_id'
16:03.08mgearyyeah, casting as int doesn’t change anything
16:03.40mgearyyou mean like this: 'field'     => "term_id",
16:03.48mgearythat doesn’t change anything
16:05.31ninnypantsyou could try adding the 'include_children' => false, but I wouldn't think it'd make a big difference
16:05.57mgearynope
16:06.56mgearyany thoughts on a workaround? the theme has a shortcode that does appear to be filtering correctly on category
16:07.05mgearyanything i can try to leverage there?
16:08.23ninnypantswhat do you get if you just run var_dump( new WP_Query( $args ) );
16:08.36ninnypantsright before their call to query_posts
16:09.19mgearyi get a big hairy object dump
16:09.33mgearybut the “request” on that object still has the 0 = 1 in it
16:10.01mgearyhttp://pastebin.com/xPccBPZ5
16:10.22DexterTheDragonjosephscott: both the github links on your post goto the same place
16:12.05josephscottDexterTheDragon: thanks, fixed
16:12.21josephscottcopy and paste error :-|
16:16.20ninnypantsmgeary: can you have it just dump tax_query so var_dump( WP_Query( $args )->query_vars['tax_query'] );
16:17.56mgearyhere are all the query_vars
16:17.57mgearyhttp://pastebin.com/CHdxZ28R
16:21.04ninnypantsmgeary: I'm stumped that all looks correct to me
16:21.11ninnypantsactually term should be terms
16:21.52mgearyoh ho
16:21.57mgearythat did the trick
16:21.59mgearysweet!
16:22.14mgearynice job, dude
16:25.39*** join/#uphpu romanovic (~romanovic@puritynine.com)
16:25.39*** mode/#UPHPU [+v romanovic] by ChanServ
16:29.05ninnypantsmgeary: if only I hadn't mistyped the first time eh? :)
16:29.15mgearyjeeze, seriously. You moron!
16:29.18mgeary;)
17:09.46carmonyAlright.... so... I have something if people want to check it out :P
17:10.34carmonyhttps://utos-slack.herokuapp.com/
17:10.55DexterTheDragongross
17:15.01ninnypantscarmony: http://d.pr/i/1gZb5
17:36.16josephscottslack really needs to fix multiple accounts
17:36.36josephscottthis make the 432,534,987,511,783 slack account for me
17:36.46josephscottall with two factor auth :-(
17:40.27carmony1Password saves my backon
17:40.29carmonybacon
17:40.45carmonyand I'm planning on setting up a bot to bridge the channels if we want
17:40.54carmonybut it is just an experiment :P
17:43.34romanovicyeah, i wonder if slack anticipated the ways people have started using it
17:43.41josephscott1password doesn't solve polluting Google Authenticator with 2fa codes
17:44.15romanovicie fragmenting user communities grouped on federated irc servers into a bunch of smaller slack teams
17:44.21josephscotta separate account for everyone that wants to use slack is making it less useful
17:45.22ninnypantsjosephscott: I agree I just want one slack account that lets me join all the teams I want
17:45.50romanovicyeah. well, they could do away with registration for slack teams who want to be open, but then they'd be an irc server
17:47.57DexterTheDragongiven that what they sell is logging and integrations, they should just be an irc server for open communities
17:50.49carmonyWhat slack should have done is create freenode.slack.com :P
17:56.26*** join/#uphpu Heartsbane (tsharpe@unaffiliated/heartsbane)
17:56.26*** mode/#UPHPU [+v Heartsbane] by ChanServ
18:00.51DexterTheDragonmindjuju: ditto. I think I'm going to move on
18:01.04mindjujuany alternatives?
18:01.28DexterTheDragonI'm debating switching to this http://www.passwordstore.org/
18:05.16fungusmindjuju, what's this about lastpass?  Bought by who?
18:05.32mindjujulogmein
18:05.37mindjujuwhich is log me in
18:05.41mindjujuand not some kind of noodle
18:05.48DexterTheDragona wild fungus appears
18:06.14mindjujuevery time i see logmein , i think noodles
18:06.30mindjujusurely i'm not the only one?
18:06.37fungusI think you are.
18:06.56mindjuju:(
18:07.43mindjujucourse though, now you're going to see log mein
18:11.31mindjujuthe internet is a fantastical place where you can see marvels like this! https://www.facebook.com/hojukandojo/videos/721336247968610/
18:11.44mindjujui PROMISE and guarantee this is NOT a rick roll fungus
18:11.48mindjujuit's safe to watch!!
18:12.04mindjujuoh and it's CERTAINLY NOT charlie the unicorn
18:12.12mindjujubut equally amazing
18:14.25*** join/#uphpu stevecoug (~stevecoug@oalug/member/stevecoug)
18:14.26*** mode/#UPHPU [+v stevecoug] by ChanServ
19:25.22*** join/#uphpu josephscott (~josephsco@192.0.125.251)
20:10.40*** join/#uphpu stevecoug (~stevecoug@oalug/member/stevecoug)
20:10.40*** mode/#UPHPU [+v stevecoug] by ChanServ
20:24.20mgearyyou are wrong, mindjuju
20:24.29mgearythat is *not* as amazing as Charlie the Unicorn
21:26.53mindjuju:)
22:04.56*** join/#uphpu ninnypants (~ninnypant@unaffiliated/ninnypants)
22:04.56*** mode/#UPHPU [+v ninnypants] by ChanServ
22:26.53*** join/#uphpu bbankes (~bbankes@2601:680:c402:3225:84cf:ba66:e327:6d02)
22:44.19*** join/#uphpu ninnypants (~ninnypant@unaffiliated/ninnypants)
22:44.19*** mode/#UPHPU [+v ninnypants] by ChanServ

Generated by irclog2html.pl Modified by Tim Riker to work with infobot.