This is the simple plugin to fetch data from Google Sheets and display it on your website.

Home Page: https://wordpress.org/plugins/wp-simple-spreadsheet-fetcher-for-google/

License: GNU General Public License v3.0

PHP 10.20% CSS 1.89% JavaScript 85.96% Shell 1.88% SCSS 0.06%

wp-simple-spreadsheet-fetcher-for-google's People

Contributors

Stargazers

Watchers

Forkers

mzdebo farookbe paulocr

wp-simple-spreadsheet-fetcher-for-google's Issues

シート名と範囲の入力を切り分け

Multiple Charts cause error

If having multiple charts on same page an error saying that props already exists happens and any additional charts are not drawn

CSRF

概要

CSRFの脆弱性が存在します。

再現手順

POSTツールでログイン状態のクッキーとともに /wp-admin/plugins.php?page=wsgsf_settings に POST
api_key=任意の文字列

期待する動作

無関係のページ等から更新されないこと

スクリーンショット

発生した環境

WP v5.3.2
Chrome v79.0.3945.88
WP Simple Spreadsheet Fetcher for Google v0.3.6

補足

add_submenu_page は権限のチェックはありますが送信元のチェック等はありません。

form に token を埋め込み更新前にチェックするようにします。

wp-simple-spreadsheet-fetcher-for-google/wp-simple-spreadsheet-fetcher-for-google.php

Lines 66 to 70 in 041cd62

 $html .= '<form id="wp2s2fg_api_spreadsheetId_form" action="' . htmlspecialchars( $_SERVER["PHP_SELF"] . '?' . $_SERVER["QUERY_STRING"] ) . '" method="POST" >'; 

 $html .= '<div class="wp2s2fg_api_spreadsheetId_form_label">' . __( "API Key : ", 'wp-simple-spreadsheet-fetcher-for-google' ) .'</div><input type="text" name="api_key" placeholder="API-Key" value="' . esc_html( $api_key ) . '" required />'; 

 $html .= '<br>'; 

 $html .= '<input type="submit" value="Set Configuration Info" />'; 

 $html .= '</form >';

wp-simple-spreadsheet-fetcher-for-google/wp-simple-spreadsheet-fetcher-for-google.php

Lines 96 to 100 in 041cd62

 public function render_settings() { 

 if ( isset( $_POST['api_key'] )) { 

 wp2s2fg_set_api_key( sanitize_text_field( $_POST['api_key'] ) ); 

 }

もしくはSettings APIを使用します。
https://developer.wordpress.org/plugins/settings/settings-api/
https://nskw-style.com/2014/wordpress/settings-api.html

フェッチャーアイテムの「範囲(range)」は「セル(cell)」の方がよいのでは?

有用なプラグインです。ありがとうございます。
遊んでいてちょっととまどった点。アドバンスドフェッチャーは単一の値を取得しますが、「範囲」となっていて少し動作に戸惑いました。「セル」で良いのでは?
実際範囲を指定してもコードでは最初のセルのみを使用していますね。

build/get-value-query.php

$data = '<p class="wp2s2fg_fetcher-advanced_number">' . esc_html( $values[0][0] ) . '</p>';

style.css 
.wp2s2fg_fetcher-advanced_price{
	font-size: 0.8rem;
}

.wp2s2fg_fetcher-advanced_number_container{
	font-size: 1.1rem;
}

ただ削除すると「/」とベースラインが合いませんがこれは <p> と <span> の違いから。

<span> で統一すれば解決する?

	$html .= '<form id="wp2s2fg_api_spreadsheetId_form" action="' . htmlspecialchars( $_SERVER["PHP_SELF"] . '?' . $_SERVER["QUERY_STRING"] ) . '" method="POST" >';
	$html .= '<div class="wp2s2fg_api_spreadsheetId_form_label">' . __( "API Key : ", 'wp-simple-spreadsheet-fetcher-for-google' ) .'</div><input type="text" name="api_key" placeholder="API-Key" value="' . esc_html( $api_key ) . '" required />';
	$html .= '<br>';
	$html .= '<input type="submit" value="Set Configuration Info" />';
	$html .= '</form >';

	public function render_settings() {

	if ( isset( $_POST['api_key'] )) {
	wp2s2fg_set_api_key( sanitize_text_field( $_POST['api_key'] ) );
	}

naogify / wp-simple-spreadsheet-fetcher-for-google Goto Github PK