@@ -21,13 +21,13 @@ function listAccounts() {
2121 let pageToken ;
2222 do {
2323 const response = AdSense . Accounts . list ( { pageToken : pageToken } ) ;
24- if ( response . accounts ) {
25- for ( const account of response . accounts ) {
26- Logger . log ( 'Found account with resource name "%s" and display name "%s".' ,
27- account . name , account . displayName ) ;
28- }
29- } else {
24+ if ( ! response . accounts ) {
3025 Logger . log ( 'No accounts found.' ) ;
26+ return ;
27+ }
28+ for ( const account of response . accounts ) {
29+ Logger . log ( 'Found account with resource name "%s" and display name "%s".' ,
30+ account . name , account . displayName ) ;
3131 }
3232 pageToken = response . nextPageToken ;
3333 } while ( pageToken ) ;
@@ -47,15 +47,15 @@ function listAdClients(accountName) {
4747 const response = AdSense . Accounts . Adclients . list ( accountName , {
4848 pageToken : pageToken
4949 } ) ;
50- if ( response . adClients ) {
51- for ( const adClient of response . adClients ) {
52- Logger . log ( 'Found ad client for product "%s" with resource name "%s".' ,
53- adClient . productCode , adClient . name ) ;
54- Logger . log ( 'Reporting dimension ID: %s' ,
55- adClient . reportingDimensionId ?? 'None' ) ;
56- }
57- } else {
50+ if ( ! response . adClients ) {
5851 Logger . log ( 'No ad clients found for this account.' ) ;
52+ return ;
53+ }
54+ for ( const adClient of response . adClients ) {
55+ Logger . log ( 'Found ad client for product "%s" with resource name "%s".' ,
56+ adClient . productCode , adClient . name ) ;
57+ Logger . log ( 'Reporting dimension ID: %s' ,
58+ adClient . reportingDimensionId ?? 'None' ) ;
5959 }
6060 pageToken = response . nextPageToken ;
6161 } while ( pageToken ) ;
@@ -75,13 +75,13 @@ function listAdUnits(adClientName) {
7575 pageSize : 50 ,
7676 pageToken : pageToken
7777 } ) ;
78- if ( response . adUnits ) {
79- for ( const adUnit of response . adUnits ) {
80- Logger . log ( 'Found ad unit with resource name "%s" and display name "%s".' ,
81- adUnit . name , adUnit . displayName ) ;
82- }
83- } else {
78+ if ( ! response . adUnits ) {
8479 Logger . log ( 'No ad units found for this ad client.' ) ;
80+ return ;
81+ }
82+ for ( const adUnit of response . adUnits ) {
83+ Logger . log ( 'Found ad unit with resource name "%s" and display name "%s".' ,
84+ adUnit . name , adUnit . displayName ) ;
8585 }
8686
8787 pageToken = response . nextPageToken ;
@@ -114,22 +114,22 @@ function generateReport(accountName, adClientReportingDimensionId) {
114114 orderBy : [ '+DATE' ]
115115 } ) ;
116116
117- if ( report . rows ) {
118- const spreadsheet = SpreadsheetApp . create ( 'AdSense Report' ) ;
119- const sheet = spreadsheet . getActiveSheet ( ) ;
117+ if ( ! report . rows ) {
118+ Logger . log ( 'No rows returned.' ) ;
119+ return ;
120+ }
121+ const spreadsheet = SpreadsheetApp . create ( 'AdSense Report' ) ;
122+ const sheet = spreadsheet . getActiveSheet ( ) ;
120123
121- // Append the headers.
122- sheet . appendRow ( report . headers . map ( ( header ) => header . name ) ) ;
124+ // Append the headers.
125+ sheet . appendRow ( report . headers . map ( ( header ) => header . name ) ) ;
123126
124- // Append the results.
125- sheet . getRange ( 2 , 1 , report . rows . length , report . headers . length )
126- . setValues ( report . rows . map ( ( row ) => row . cells . map ( ( cell ) => cell . value ) ) ) ;
127+ // Append the results.
128+ sheet . getRange ( 2 , 1 , report . rows . length , report . headers . length )
129+ . setValues ( report . rows . map ( ( row ) => row . cells . map ( ( cell ) => cell . value ) ) ) ;
127130
128- Logger . log ( 'Report spreadsheet created: %s' ,
129- spreadsheet . getUrl ( ) ) ;
130- } else {
131- Logger . log ( 'No rows returned.' ) ;
132- }
131+ Logger . log ( 'Report spreadsheet created: %s' ,
132+ spreadsheet . getUrl ( ) ) ;
133133}
134134
135135/**
0 commit comments