#dev | Logs for 2017-06-07

« return
[01:38:54] * Bytram shambles in and reads the backscroll
[01:39:36] <Bytram> FatPhil: Saw a callout for assistance with 'Featured Stories'... would love to help, but not sure where that is on the site? Linky please?
[05:27:43] <FatPhil> Bytram: sure, there are some bits of the site that are extracted from the database which are malformed HTML and are not fixed before squirting them out:
[05:27:48] <FatPhil> <div class="content" id="features-content"> 
[05:27:48] <FatPhil> <ul>
[05:27:48] <FatPhil> <li><a href=http://soylntnews.org/article.pl?sid=14/02/12/0715245>Welcome to SoylentNews!</a></li>
[05:27:51] <FatPhil> <li><a href=http://dev.soylentnews.org/article.pl?sid=14/02/13/0412219>Reworking Moderation Access</a></li>
[05:28:09] <FatPhil> ^^ href's URLs not in quotes
[05:37:35] <FatPhil> Hmmm that touche's a fun one
[05:37:46] <FatPhil> $ git grep -h 'reason_select.To' | od -c -tx1
[05:37:53] <FatPhil> 0000020 o n _ s e l e c t . T o u c h � │······················ 6f 6e 5f 73 65 6c 65 63 74 2e 54 6f 75 63 68 c3 │······················
[05:37:57] <FatPhil> 0000040 � % ] \t < / t d > \n │······················ a9 20 25 5d 09 3c 2f 74 64 3e 0a │······················
[05:38:46] <FatPhil> U+00E9 0xE9 éÃ© %C3 %A9
[05:49:57] <FatPhil> so it's not utf8 being mangled, that's valid utf8
[05:50:20] <FatPhil> it's perl not accepting utf8 in the file it reads?
[05:57:23] <FatPhil> commit 896ba751 "Two exceedingly minor bugs fixed" Fri Feb 3 14:25:56 2017 +0000
[05:59:33] <FatPhil> This is even odder:
[05:59:46] <FatPhil> $ git grep -h "UPDATE modreasons SET name = 'To" | od -c -tx1
[05:59:54] <FatPhil> 0000040 u c h � ' W H E R E i d = │······················ 75 63 68 e9 27 20 57 48 45 52 45 20 69 64 20 3d │······················
[06:00:17] <FatPhil> That *ain't* utf8, which seems inconsistent
[06:04:15] <FatPhil> e9 is the latin-1 code point for eacute, obviously
[10:26:22] <TheMightyBuzzard> FatPhil, perl by default does not accept unicode in source files. you need "use utf8" at the top of the file to make it do so.
[10:27:47] <TheMightyBuzzard> also, we don't go back and modify prior db setup statements, we amend them.
[10:32:03] <TheMightyBuzzard> in the upgradeCoreDB sub of Slash/DB/MySQL/MySQL.pm in the case of base changes and in the upgradeDB sub of plugins/<whatever>/Upgrade/MySQL.pm in the case of plugin changes
[10:34:49] <TheMightyBuzzard> :q
[10:37:02] -!- cosurgi has quit [Ping timeout: 276 seconds]
[10:37:52] <TheMightyBuzzard> ooooh, you changed a variable name to unicode when you changed reason_select.Touché
[10:38:43] <TheMightyBuzzard> or someone did anyway
[10:41:21] <TheMightyBuzzard> no... that shit has been working for quite some time...
[10:45:51] <TheMightyBuzzard> paulej72, FatPhil, after testing: not seeing that error on my end. https://dev.soylentnews.org renders peachy keen.
[10:47:53] <TheMightyBuzzard> and appears to have been working fine since february
[10:53:31] <TheMightyBuzzard> yeah, that file is the same version as on prod right now and it works fine there and on dev. chalk that up to random weirdness?
[11:05:35] -!- cosurgi [cosurgi!~cosurgi@qylizv7.bl.pg.gda.pl] has joined #dev
[11:06:34] cosurgi is now known as SoyGuest52409
[11:20:06] <FatPhil> $ git blame "themes/default/templates/editComm;users;default" | grep \\.Tou
[11:20:06] <FatPhil> 896ba751 themes/default/templates/editComm;users;default (TheMightyBuzzard 2017-02-03 14:25:56 +0000 192) <td> [% reason_select.Touché %] </td>
[11:21:05] <TheMightyBuzzard> FatPhil, yes but it ain't been erroring since february. no idea why it decided to yesterday.
[11:21:21] <FatPhil> perl upgrade causing crankiness?
[11:21:46] <TheMightyBuzzard> nah, we don't use system perl. we roll our own and rarely upgrade.
[11:22:13] <TheMightyBuzzard> random wonkiness caused by cosmic rays flipping a bit in ram?
[11:22:17] <FatPhil> bizarro - working fine here too
[11:23:52] <TheMightyBuzzard> "use utf8" is supposed to allow inclusion of utf8 in source files but we try quite hard to not do so because underlying modules may not be utf8-safe.
[11:24:13] <FatPhil> use utf8 is in every .pl file, and many .pm files
[11:24:55] <TheMightyBuzzard> ya. it should be in most all of em but the modules we use but didn't write are the problem. they could very well handle text in a utf8-unsafe way.
[11:25:08] <TheMightyBuzzard> which is still possible to do even with "use utf8"
[11:26:01] <TheMightyBuzzard> perl and utf8 are an enormously complex and headache-inducing minefield
[11:29:12] <FatPhil> if using utf isn't enough, then use double-utf!
[11:30:08] <TheMightyBuzzard> use utf16?
[11:30:54] <FatPhil> if the world didn't have enough evidence that the unicode consortium is a bunch of arse, the existence of utf-16 seals the deal.
[11:31:52] <TheMightyBuzzard> indeed
[11:33:09] * TheMightyBuzzard debates getting any SN work done today
[11:33:27] <TheMightyBuzzard> you wanted two strip subs written, didn't ya?
[11:33:53] <FatPhil> no rush, I've had a pile of work dumped on me from a great height
[11:34:22] <FatPhil> I think the next thing I should do is just write a to-do list wiki page for myself, as I keep uncovering other things I want to look at.
[11:35:22] <TheMightyBuzzard> bender there has a !todo function but the wiki is prolly less cumbersome
[11:36:49] * TheMightyBuzzard writes up a quick submission
[11:57:19] <Bytram> FatPhil: trying to follow along, but not able to find what you are referencing. A URL would be nice and/or, if it's a template, the baz;bar;foo name, if you would be so kind? No rush as I need to be AT work in an hour so not gonna get much done on it until later today
[12:09:27] <FatPhil> Bytram: this looks like a dev-only thing
[12:10:37] <Bytram> Oh? Ohhhh.... that 'splains why I could not find it on prod!
[18:09:26] -!- FatPhil has quit [Ping timeout: 276 seconds]
[18:09:29] -!- FatPhil [FatPhil!phil@azdr.ee] has joined #dev
[18:10:28] FatPhil is now known as SoyGuest18916
[20:16:13] SoyGuest52409 is now known as cosurgi
[20:16:15] -!- cosurgi has quit [Changing host]
[20:16:15] -!- cosurgi [cosurgi!~cosurgi@Soylent/Staff/Misc/cosurgi] has joined #dev