Change logic of "don't create a status post for this photo upload" and turn it into a bona-fide on/off toggle. This takes up one more vertical line, but we have it to spare, and the single button (while concise visually) provided no feedback of the current state, or indicate whether a second click toggled it. Some addons use this setting so addons will need to be pulled when this is pulled in order to remain in sync.

2 files changed, 2 insertions, 2 deletions

diff --git a/mod/photos.php b/mod/photos.php
index 297790f6e..58a38f3f3 100644
--- a/mod/photos.php
+++ b/mod/photos.php
@@ -553,7 +553,7 @@ function photos_content(&$a) {
 			'$nickname' => $a->data['channel']['channel_address'],
 			'$newalbum_label' => t('Enter a new album name'),
 			'$newalbum_placeholder' => t('or select an existing one (doubleclick)'),
-			'$nosharetext' => t('Do not show a status post for this upload'),
+			'$visible' => array('visible', t('Create a status post for this upload'),0,''), 
 			'$albums' => $albums['albums'],
 			'$selname' => $selname,
 			'$permissions' => t('Permissions'),
diff --git a/mod/wall_upload.php b/mod/wall_upload.php
index 81ca643df..31a497f0f 100644
--- a/mod/wall_upload.php
+++ b/mod/wall_upload.php
@@ -32,7 +32,7 @@ function wall_upload_post(&$a) {
 
 	$observer = $a->get_observer();
 
-	$args = array( 'source' => 'editor', 'not_visible' => 1, 'contact_allow' => array($channel['channel_hash']));
+	$args = array( 'source' => 'editor', 'visible' => 0, 'contact_allow' => array($channel['channel_hash']));
 
  	$ret = photo_upload($channel,$observer,$args);